Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Awkward space to the left and right of the sort handle in List #11125

Open
2 of 6 tasks
ashetland opened this issue Dec 20, 2024 · 0 comments
Open
2 of 6 tasks

Awkward space to the left and right of the sort handle in List #11125

ashetland opened this issue Dec 20, 2024 · 0 comments
Assignees
Labels
1 - assigned Issues that are assigned to a sprint and a team member. bug Bug reports for broken functionality. Issues should include a reproduction of the bug. Calcite (design) Issues logged by Calcite designers. calcite-components Issues specific to the @esri/calcite-components package. estimate - 3 A day or two of work, likely requires updates to tests. impact - p1 - need for current milestone User set priority impact status of p1 - need for current milestone p - high Issue should be addressed in the current milestone, impacts component or core functionality

Comments

@ashetland
Copy link

Check existing issues

Actual Behavior

There is space to the left of the sort handle in the v3 List that creates an awkward 4px where the handle cannot be grabbed. Also, without space to the right of the handle, the hover and focus states butt against the item content.
CleanShot 2024-12-20 at 10 53 11@2x

Expected Behavior

The space to the left is removed and space to the right is added.

Reproduction Sample

https://esri.github.io/calcite-design-system/?path=/story/components-list--scales

Reproduction Steps

Scroll to view the drag-enabled stories.

Reproduction Version

v3.0.0-next.80

Relevant Info

See this Figma file for spec changes to each scale.

cc @eriklharper

Regression?

No response

Priority impact

impact - p1 - need for current milestone

Impact

No response

Calcite package

  • @esri/calcite-components
  • @esri/calcite-components-react
  • @esri/calcite-design-tokens
  • @esri/calcite-ui-icons
  • @esri/eslint-plugin-calcite-components

Esri team

Calcite (design)

@ashetland ashetland added bug Bug reports for broken functionality. Issues should include a reproduction of the bug. 0 - new New issues that need assignment. needs triage Planning workflow - pending design/dev review. labels Dec 20, 2024
@github-actions github-actions bot added Calcite (design) Issues logged by Calcite designers. calcite-components Issues specific to the @esri/calcite-components package. impact - p1 - need for current milestone User set priority impact status of p1 - need for current milestone labels Dec 20, 2024
@ashetland ashetland changed the title Space to the left and right of the sort handle in List Awkward space to the left and right of the sort handle in List Dec 20, 2024
@geospatialem geospatialem added p - high Issue should be addressed in the current milestone, impacts component or core functionality 1 - assigned Issues that are assigned to a sprint and a team member. estimate - 3 A day or two of work, likely requires updates to tests. and removed 0 - new New issues that need assignment. needs triage Planning workflow - pending design/dev review. labels Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1 - assigned Issues that are assigned to a sprint and a team member. bug Bug reports for broken functionality. Issues should include a reproduction of the bug. Calcite (design) Issues logged by Calcite designers. calcite-components Issues specific to the @esri/calcite-components package. estimate - 3 A day or two of work, likely requires updates to tests. impact - p1 - need for current milestone User set priority impact status of p1 - need for current milestone p - high Issue should be addressed in the current milestone, impacts component or core functionality
Projects
None yet
Development

No branches or pull requests

3 participants