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

"(private tab)" is unreadable on systems with light-on-dark tooltip themes #38

Closed
ssokolow opened this issue Apr 21, 2013 · 7 comments
Closed

Comments

@ssokolow
Copy link

I run Lubuntu Linux and, like the main Ubuntu distribution, the default theme uses tooltips with a dark background and white text.

Because you use hyperlink blue for your "(private tab)" text, that means I can barely see that it's there... let alone read it.

It'd be better to use the system-provided colors and just make "(private tab)" stand out by making it bold and possibly underlined.

@Infocatcher
Copy link
Owner

Unfortunately system colors aren't good too: only gray variants on Windows and may be not compatible with non-default themes.

Infocatcher added a commit that referenced this issue Apr 21, 2013
@Infocatcher
Copy link
Owner

With 17afdcf commit for bright default text will be used following style:

#privateTab-tooltip-isPrivateTabLabel {
    color: currentColor;
    text-decoration: underline;
    -moz-text-decoration-color: currentColor;
    -moz-text-decoration-style: dashed;
}

Test version: private_tab-0.1.1pre15-fx-sm.xpi

@ssokolow
Copy link
Author

My mistake. I didn't mean to specifically say that system colours should always be used. What I meant but didn't communicate properly was that, if the theme decides to use system-provided colors, the tooltips should follow along to avoid this kind of breakage.

The test version is perfectly readable though the "(private tab)" label isn't as eye-grabbing with only a dashed underline.

(Hence why I suggested using bold to replace the blue colour. It'd look good and be simpler than the alternative: Using colour theory to generate candidate foreground colors that complement the background color, calculating the luminosity contrast ratio of each, and then picking one with a ratio of at least 5:1.)

@Infocatcher
Copy link
Owner

I'm not sure about bold text... Some theme may use it. And bold text appearance is font-dependent.
Also I think, that underline is mere specific and more likely will not be used in themes.

About alternative. I really won't use complex things for simple features. Here may be background images or background gradients...

So, I think, this fix is good approach. :)
And may be easily corrected using userChrome.css – not so easily as using some options, but better than nothing.

Really here is another issue: missing options for private tabs appearance. But many users asks about "something instead of underline". And this is problem with themes compatibility. Only underline and changed icons from example styles looks good in most themes. But I prefer original icons.

@Infocatcher
Copy link
Owner

So, can I close this issue? :)

@ssokolow
Copy link
Author

Yeah. My only concern with the underline is that it might not be as eye-grabbing as intended. If you're OK with that, then the test version is perfect.

@Infocatcher
Copy link
Owner

OK, I tested, and bold + underline looks not so bad. :)

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