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

iconized_main_menu.css #390

Open
wincmd64 opened this issue Jul 29, 2024 · 9 comments
Open

iconized_main_menu.css #390

wincmd64 opened this issue Jul 29, 2024 · 9 comments

Comments

@wincmd64
Copy link

can you expand iconized_main_menu.css with more icons?
like: https://gist.github.com/qaz69wsx/83a90423163b65a8344b2e60fc356a58

Thx

@MrOtherGuy
Copy link
Owner

What icons are missing currently from it? But I'm not going to expand it by adding icons for items in other menus, if that's what you mean. I'm open to creating separate style for that purpose though.

@wincmd64
Copy link
Author

wincmd64 commented Jul 29, 2024

I'm open to creating separate style for that purpose though.

Okey, it would be great!
iconized_tabs_menu.css
iconized_context_menu.css

@MrOtherGuy
Copy link
Owner

MrOtherGuy commented Jul 29, 2024

Tabs menu would be somewhat reasonable to implement, but the "normal" content area context menu is quite a task since it has like 80 possible menu items, and it would be pretty annoying if some had icons while others didn't.

@MrOtherGuy
Copy link
Owner

I have iconized_content_context_menu.css in progress. I won't add unique icons for everything at once, instead a bunch of items use some "generic" icon fitting for the item category e.g. most image related menuitems use a generic canvas icon.

@MrOtherGuy
Copy link
Owner

I've added iconized_content_context_menu.css in ba712db - there's also some items that could reasonably be expected to have an icon in sub-menus (inside Frame notably) but they are not included in this patch yet.

@wincmd64
Copy link
Author

wincmd64 commented Aug 1, 2024

👍 iconized_context_menu.css
🙄 iconized_tabs_menu.css
🙄 iconized_favs_menu.css

@MrOtherGuy
Copy link
Owner

I've added styles for:

Iconized content context menu also received a few minor updates.

@MrOtherGuy
Copy link
Owner

I suspect that the icon alignment is probably wrong on Linux, as well as for anyone on macOS if someone would use these with macOS native context menus disabled. But it should be simple to add platform specific alignment rules.

@wincmd64
Copy link
Author

wincmd64 commented Aug 4, 2024

I like it. It looks great!

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