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

Text-decoration, hover and transition on Accordion on Safari not working #2951

Closed
georgibakken opened this issue Jun 13, 2024 · 3 comments
Closed

Comments

@georgibakken
Copy link

Bug report

Current Behavior

Using text-decoration, hover and transition with Safari does not work as intended on the accordion, and the hover effect seems to be triggered when the item is not hovered.

Note that the background-color can update on hover as expected.

Screen.Recording.2024-06-13.at.10.44.04.mov

Removing

text-decoration: underline;
text-decoration-color: transparent;

from .AccordionTrigger and adding text-decoration: underline; to .AccordionTrigger:hover makes the hover work, however for the transition to work, I need to have them defined in .AccordionTrigger.

Expected behavior

Screen recording of the expected behaviour, on Chrome

Screen.Recording.2024-06-13.at.10.47.33.mov

Here is a simple html/css code example to show that text-decoration, transition and hover works with Safari

Reproducible example

CodeSandbox Template

Suggested solution

Additional context

Your environment

Software Name(s) Version
Radix Package(s) radix-ui/react-accordion 1.1.2
React n/a 18.2.0
Browser Safari Version 17.5
Assistive tech
Node n/a v22.2.0
npm/yarn pnpm 8.9.0
Operating System macOS 14.5 (23F79)
@georgibakken georgibakken changed the title Text-decoration, hover and transition on Accordion on Safari acts funky Text-decoration, hover and transition on Accordion on Safari not working Jun 14, 2024
@vladmoroz
Copy link
Collaborator

This is a bug with <button> styling in Safari and has nothing to do with the Accordion component itself

@vladmoroz vladmoroz closed this as not planned Won't fix, can't repro, duplicate, stale Jul 9, 2024
@georgibakken
Copy link
Author

Thanks for looking at it. Do you have any reference to this bug so I know where to look ?

@vladmoroz
Copy link
Collaborator

No idea, I just swapped p for button in your basic example to confirm it's a browser bug

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants