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

CustomGradientPicker: glitchy interactions while adding color stops #41390

Closed
ciampo opened this issue May 27, 2022 · 2 comments
Closed

CustomGradientPicker: glitchy interactions while adding color stops #41390

ciampo opened this issue May 27, 2022 · 2 comments
Assignees
Labels
[Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Package] Components /packages/components [Type] Bug An existing feature does not function as intended

Comments

@ciampo
Copy link
Contributor

ciampo commented May 27, 2022

When clicking on the gradient bar to add a color stop:

  • it's impossible to add a color stop in proximity of the pre-existing start/end color stops — the "+" simply doesn't appear and clicking doesn't do anything (although I'm not sure if this is by design)
  • when the "+" icon appears on hover the gradient bar, I noticed that the icon's position doesn't track perfectly the cursor. This behaviour is more visible as the cursor moves away from the middle and towards the left or the right of the gradient bar. It gets to a point where the "+" icon appears, but clicking with the mouse doesn't fire on the icon (because of its offset)
customgradientpicker-clicking-color-stops.mp4

(originally reported while reviewing #41361 (review))

@ciampo ciampo added [Type] Bug An existing feature does not function as intended [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Package] Components /packages/components labels May 27, 2022
@andrewserong
Copy link
Contributor

Fix for the icon positioning in #41492 — it doesn't address the logic of when to hide the inserter as in real world usage, that appears to be slightly less of a problem with interacting with the component. Happy to look at that one separately if we get the time 🙂 (it'll be slightly more complex to deal with because the threshold currently deals in percentages).

@andrewserong andrewserong self-assigned this Jun 2, 2022
@ciampo
Copy link
Contributor Author

ciampo commented Aug 16, 2022

Closing in favour of #42994

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Package] Components /packages/components [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

2 participants