-
Notifications
You must be signed in to change notification settings - Fork 515
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
Some additions for user.js that are worth. #1408
Comments
Lines 645 to 649 in b4225ba
|
I'd have to look up the reasoning for the removed color, but IIRC its part of a longer term strategy on UX. Color doesn't mean anything IMO Also, since we use HTTPS-Only mode, every site that is not HTTPS is going to have an interstitial (for those that are excepted, then at least on the first visit), so I fail to see what coloring up the icon will do declining |
IIRC the reason was/is to mark unsafe sites as unsafe instead of safe sites as safe as safe (https) is/should be the default in the web. Therefore the color moved from green/safe to red/unsafe. |
right .. i'm not going to bother digging up the comments from bugzilla, but that's it from memory - i.e instead of checking for green (which is over half the internet's top sites and something like 80% of traffic), you should be noticing red (was less frequent) - i.e the change is more noticeable |
And FF 100 is going to remove it. |
yeah I noticed that and was going to find this issue to mention it https://bugzilla.mozilla.org/show_bug.cgi?id=1756609 (emphasis mine)
|
No, I need it. Just Updated Firefox and noticed things. Need to fix Overrides. |
you can most likely use userChrome.css and base64 if you want to - don;t ask me, try at r/firefoxcss |
security.secure_connection_icon_color_gray
Changes the colour of the secure connection padlock icon in the address bar to white/black (depending on if your theme is light ordark), or green. Enabled by default.Set to true for white/black, and false for green.
Set as Green in AF.
For Autoplay -
media.autoplay.default
= 5 [0:allow; 1:blockAudible; 5:blockAll]The text was updated successfully, but these errors were encountered: