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

ToggleGroupControl: apply animation only when interacting with toggle #57668

Closed
simison opened this issue Jan 9, 2024 · 5 comments
Closed

ToggleGroupControl: apply animation only when interacting with toggle #57668

simison opened this issue Jan 9, 2024 · 5 comments
Labels
[Package] Components /packages/components [Status] Needs More Info Follow-up required in order to be actionable. [Status] Stale Gives the original author opportunity to update before closing. Can be reopened as needed. [Type] Enhancement A suggestion for improvement.

Comments

@simison
Copy link
Member

simison commented Jan 9, 2024

What problem does this address?

Toggle button has a nice animation when toggling, but the animation gets applied also when something else in the surrounding UI changes location of the toggle on the screen:

Screen.Recording.2024-01-09.at.13.12.43.mov

Note how changing other settings causes toggle animation to lag. That said, collapsing panels doesn't cause the same effect, so I'm not entirely sure what's in play here.

What is your proposed solution?

Apply animation only onChange, and disable otherwise.

@simison simison added [Type] Enhancement A suggestion for improvement. [Feature] UI Components Impacts or related to the UI component system labels Jan 9, 2024
@mirka mirka changed the title Toggle group: apply animation only when interacting with toggle ToggleGroupControl: apply animation only when interacting with toggle Jan 9, 2024
@mirka mirka added [Package] Components /packages/components and removed [Feature] UI Components Impacts or related to the UI component system labels Jan 9, 2024
@mirka
Copy link
Member

mirka commented Jan 9, 2024

Are you able to provide some minimal repro code? At some point we fixed the issue of unwanted animations like this, so what you're seeing is probably a particular case. Like you mentioned, it doesn't usually happen on normal layout shifts, for example when toggling the hideLabelFromVision prop.

@mirka mirka added the [Status] Needs More Info Follow-up required in order to be actionable. label Feb 2, 2024
Copy link

Help us move this issue forward. This issue is being marked stale since it has no activity after 15 days of requesting more information. Please add info requested so we can help move the issue forward. Note: The triage policy is to close stale issues that need more info and no response after 2 weeks.

@github-actions github-actions bot added the [Status] Stale Gives the original author opportunity to update before closing. Can be reopened as needed. label Feb 18, 2024
@Mamaduka Mamaduka removed the [Status] Stale Gives the original author opportunity to update before closing. Can be reopened as needed. label Feb 18, 2024
Copy link

github-actions bot commented Mar 5, 2024

Help us move this issue forward. This issue is being marked stale since it has no activity after 15 days of requesting more information. Please add info requested so we can help move the issue forward. Note: The triage policy is to close stale issues that need more info and no response after 2 weeks.

@github-actions github-actions bot added the [Status] Stale Gives the original author opportunity to update before closing. Can be reopened as needed. label Mar 5, 2024
@mirka
Copy link
Member

mirka commented Mar 7, 2024

I'm assuming this will be fixed by #59642. Please re-open if not!

@mirka mirka closed this as completed Mar 7, 2024
@tyxla
Copy link
Member

tyxla commented Mar 8, 2024

Thanks for connecting the dots, @mirka, I didn't realize there was an open issue about it. And yes, that's exactly the one we were fixing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Package] Components /packages/components [Status] Needs More Info Follow-up required in order to be actionable. [Status] Stale Gives the original author opportunity to update before closing. Can be reopened as needed. [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests

4 participants