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

Context Menu should use menu element #1199

Open
pano9000 opened this issue Feb 15, 2025 · 0 comments
Open

Context Menu should use menu element #1199

pano9000 opened this issue Feb 15, 2025 · 0 comments

Comments

@pano9000
Copy link
Contributor

pano9000 commented Feb 15, 2025

Description

Hi,

the context menu currently technically is against HTML specs:
It uses a div element with li children, which is technically not allowed per HTML specs (but obviously works anyways):
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/li#technical_summary

Instead I think we just should use the more semantic menu element, which allows li elements as direct children:
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/menu

TriliumNext Version

git

What operating system are you using?

Ubuntu

What is your setup?

Local (no sync)

Operating System Version

git

Error logs

No response

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

1 participant