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

NEXT Focus ring/border flashes white when switching focus #2665

Closed
TheEisbaer opened this issue May 17, 2024 · 3 comments
Closed

NEXT Focus ring/border flashes white when switching focus #2665

TheEisbaer opened this issue May 17, 2024 · 3 comments
Labels
bug Something isn't working help wanted Extra attention is needed
Milestone

Comments

@TheEisbaer
Copy link

Current Behavior

Focus ring/border flashes white, see video

Aufzeichnung.2024-05-16.075708.mp4

Expected Behavior

Focus ring/border does not flash

Steps To Reproduce

  1. go to next skeleton docs
  2. have cerberus theme selected (there its the most obvious)
  3. switch focus on form elements

Link to Reproduction / Stackblitz

No response

More Information

Chrome Version 125.0.6422.61 (64-Bit)
Windows 11 23H2

@TheEisbaer TheEisbaer added the bug Something isn't working label May 17, 2024
@endigo9740 endigo9740 added this to the v3.0 (Next) milestone May 17, 2024
@endigo9740 endigo9740 added the help wanted Extra attention is needed label May 24, 2024
@endigo9740
Copy link
Contributor

Thanks, we had a similar issue in v2 - but I can't recall how we resolved it. We should be able to compare the v2 stylesheet to v3 and figure it out though. I'we welcome help doing this if someone has time!

@jaakkonakaza
Copy link

Screen.Recording.2024-05-25.at.21.05.37.mov

This seems to be caused by the animation being way too fast. In this video, I changed it to 300ms instead of 100ms. Thoughts? Would it be better to get rid of the whole animation or to make it less noticeable?

/packages/skeleton/src/plugin/components/forms.css

/* Standard Fields --- */

.input,
.textarea,
.select {
	@apply field-outter field-inner;
	@apply border-none;
- 	@apply transition-[all] duration-100;
+       @apply transition-[all] duration-300;
}

@jaakkonakaza
Copy link

Screen.Recording.2024-05-25.at.21.11.57.mov

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants