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.

Sign up for GitHub

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

Jump to bottom

themes: use popover API for dropdowns etc when it isn't really new anymore #138

Open
icxes opened this issue May 12, 2024 · 1 comment
Open

themes: use popover API for dropdowns etc when it isn't really new anymore #138

icxes opened this issue May 12, 2024 · 1 comment
Assignees
@icxes
Labels
front-end issue is related to the structure of the website and doesn't require backend work low-priority issue is not important and likely won't get resolved soon post-1.0 issue won't be resolved until after launch

Comments

@icxes
Copy link
Collaborator

icxes commented May 12, 2024

https://developer.mozilla.org/en-US/docs/Web/API/Popover_API
@Wessie Wessie transferred this issue from R-a-dio/valkyrie-themes May 21, 2024
@Wessie Wessie changed the title use popover API for dropdowns etc when it isn't really new anymore themes: use popover API for dropdowns etc when it isn't really new anymore May 21, 2024
@Wessie Wessie added the front-end issue is related to the structure of the website and doesn't require backend work label May 21, 2024
@icxes
Copy link
Collaborator Author

icxes commented Dec 25, 2024 •
edited
Loading

This has now been supported for over a year on Chromium-based browsers and for about half a year on Firefox, with a certain bug that would've been problematic now also resolved on Safari, so this should be fine for 99% of users. The ones using browsers without support for this probably also have javascript disabled, so the site won't work for them anyway.

@icxes icxes self-assigned this Dec 25, 2024
@icxes icxes added post-1.0 issue won't be resolved until after launch low-priority issue is not important and likely won't get resolved soon labels Dec 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@icxes icxes

Labels
front-end issue is related to the structure of the website and doesn't require backend work low-priority issue is not important and likely won't get resolved soon post-1.0 issue won't be resolved until after launch
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Wessie @icxes