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

Add a Cast icon for the context menu #131

Closed
rodrigoswz opened this issue Jul 17, 2020 · 3 comments
Closed

Add a Cast icon for the context menu #131

rodrigoswz opened this issue Jul 17, 2020 · 3 comments

Comments

@rodrigoswz
Copy link

Perhaps the minor and least important issue of this amazing project...

I just wanted to request to add an icon in the context menu because currently the option mixes with the other extensions and causes a certain strangeness. It's it.

image

@hensm
Copy link
Owner

hensm commented Jul 21, 2020

Late response, sorry.

Yeah, the only reason I didn't add the icon already was because it looked weird against dark themes. The menu icon itself isn't such a big problem, since only Firefox on Linux has dark menus, but it can't be set independently of the extension icon which appears against multiple dark backgrounds.

Could use a different colour until they support theme-specific icons, I suppose?

menu
addons

@rodrigoswz
Copy link
Author

I understand, for now, a different color looks good to me, if you agree ofc.

But, Open in Sidebar is an extension on my browser that can do this, note my first comment with light theme and now see how the icon of this extension looks in the dark theme:

image

I haven't found the source code for that extension though (if that is open source).

@hensm
Copy link
Owner

hensm commented Jul 22, 2020

Seems to inherit its fill colour from the browser stylesheet, but this is only enabled for internal URLs and Mozilla extensions by default. Do you have svg.context-properties.content.enabled flipped, by any chance? I could make it so it falls back to the blue, so anyone with it enabled would see the correct colours.

Turns out prefers-color-scheme works in icon SVGs now, so that's also an option. But it breaks on anything but the default theme and probably doesn't work on Linux (e: or in fact macOS/Windows, since the context menus don't match the dark theme yet, so it's not really an option except for the add-ons page).

Just a matter of picking the least worst solution,

@hensm hensm closed this as completed in 84c99ba Jul 24, 2020
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