overdodactyl / ShadowFox

A universal dark theme for Firefox
https://overdodactyl.github.io/ShadowFox/
MIT License
1.32k stars 58 forks source link

Tabs Bar Color #233

Closed GTK66 closed 5 years ago

GTK66 commented 5 years ago

In Firefox 65 The Tabs Bar takes on the color that is set in windows 10.. In order to make it dark you either have to change the color settings in Windows 10 or leave out the check mark in Windows 10 Colors in Title Bars and Windows borders as shown in the image below. This will make the Tabs bar dark again, but it messes up Windows 10's Programs. If I leave the the check mark out, all programs in Windows 10 will have a bright tool bar. I use ShadowFox to get away from bright colors. This issue became a problem with Firefox 65 as Mozilla changed something with the bars. This did not happen in Firefox 64. See the attached image link. This is not an issue with the default dark theme it is in ShadowFox. A Simple CSS script may fix this.

https://i.imgur.com/GuZWQ7i.png

overdodactyl commented 5 years ago

This is not an issue with the default dark theme it is in ShadowFox

Are you sure of this? ShadowFox doesn't have any styling for the tabs bar, so its color should be entirely dependent on Firefox/your Windows setup

GTK66 commented 5 years ago

Yes

Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10


From: Pat Johnson notifications@github.com Sent: Saturday, December 29, 2018 3:27:19 PM To: overdodactyl/ShadowFox Cc: GTK66; Author Subject: Re: [overdodactyl/ShadowFox] Tabs Bar Color (#233)

This is not an issue with the default dark theme it is in ShadowFox

Are you sure of this? ShadowFox doesn't have any styling for the tabs bar, so it's color will be entirely dependent on Firefox/your Windows setup

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/overdodactyl/ShadowFox/issues/233#issuecomment-450517813, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ANSRmt37DIBQmdAsBO75woLzXfCldAm2ks5u99AngaJpZM4ZkmH-.

overdodactyl commented 5 years ago

I'm not entirely sure how that could be - as mentioned above, ShadowFox doesn't touch the tab bar, nor take into account any Windows based colors/themes. The styling for that is entirely based on Firefox (how they utilize your Windows theme, I am unsure).

Based on your screen shots in https://github.com/SrKomodo/shadowfox-updater/issues/31, however, it doesn't look like you have the FF dark theme in use (the address bar, for example, is white).

Whether Firefox changed something to no longer allow you to use the default theme without a certain Windows config, I am not sure.

If that's the case I may need to look into overriding the tab color, but as it stands it is not ShadowFox causing bright colors in your tab region.

overdodactyl commented 5 years ago

@loopy750, any insights here by chance?

GTK66 commented 5 years ago

I will put this as simple as I can. Beginning with Firefox 65 Firefox uses the color scheme that the user has set in Windows 10. If you have it set to Blue, The Tabs Bar at the top of Firefox will be Blue. To fix this the Windows 10 user must change his color scheme to a dark shade to get the total dark view in Firefox. This works but it makes Windows 10 look even worse than it does. I corrected my issue by Installing That Black Theme by Madonna. All of the above being said, other users will have the same problem. In Firefox 64 this was not an issue. It is in any release greater than Firefox 64.

Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10


From: Pat Johnson notifications@github.com Sent: Saturday, December 29, 2018 4:10:33 PM To: overdodactyl/ShadowFox Cc: GTK66; Author Subject: Re: [overdodactyl/ShadowFox] Tabs Bar Color (#233)

I'm not entirely sure how that could be - as mentioned above, ShadowFox doesn't touch the tab bar, nor take into account any Windows based colors/themes. The styling for that is entirely based on Firefox (how they utilize your Windows theme, I am unsure).

Based on your screen shots in SrKomodo/shadowfox-updater#31https://github.com/SrKomodo/shadowfox-updater/issues/31, however, it doesn't look like you have the FF dark theme in use (the address bar, for example, is white).

Whether Firefox changed something to no longer allow you to use the default theme without a certain Windows config, I am not sure.

If that's the case I may need to look into overriding the tab color, but as it stands it is not ShadowFox causing bright colors in your tab region.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/overdodactyl/ShadowFox/issues/233#issuecomment-450520337, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ANSRmqLXi-Cmx9Tx6kl83vPU5Tchs8kcks5u99pJgaJpZM4ZkmH-.

overdodactyl commented 5 years ago

Gotcha, thanks

So generally speaking, in the past I've opted to stay as consistant with Firefox's look as possible, and make ShadowFox simply a complementary feature to the available dark theme rather than res-style/change things that are designed with intention for the dark theme.

I would tend to agree though that having a colored tab area doesn't fit great with ShadowFox, so maybe this is a case worth making an exception for.

I don't use Windows though, so I'm not sure how this theming fits in with their ecosystem/how users would feel about me overriding it.

I guess another possibility would be to add an option in the updater that could append something to the effect of

#TabsToolbar {
    background: var(--tone-9) !important;
}

to userChrome.css (pinging @SrKomodo in case you have any thoughts/opinions on this).

Feedback from other Windows users would be greatly appreciated as well.

Relevant reddit thread with a screenshot of how Firefox is effected:

https://old.reddit.com/r/firefox/comments/9ukhed/nightly_dark_theme_menu_bar_no_longer_dark/

loopy750 commented 5 years ago

It doesn't look too bad with the light theme, but does look horrible with the dark theme.

/* Dark theme */

.tab-background:not([selected]):-moz-lwtheme-brighttext,
.browser-toolbar.titlebar-color:-moz-lwtheme-brighttext {
  background-color: #0C0C0D !important;
  color: #F9F9FA !important;
}

/* Light theme */

.tab-background:not([selected]):-moz-lwtheme-darktext,
.browser-toolbar.titlebar-color:-moz-lwtheme-darktext {
  background-color: #E3E4E6 !important;
  color: #18191A !important;
}
overdodactyl commented 5 years ago

Thanks @loopy750 and @GTK66!

I added the code above (dark theme) with the following variables:

--tab-background-color
--tab-color

which default to var(--tone-9) and var(--in-content-selected-text) respectively.

Let me know if this doesn't fix things and I'll re-open!

GTK66 commented 5 years ago

No this did not work unless I placed in the wrong place. I placed it at user name\AppData\Roaming\Mozilla\Firefox\Profiles\bjaip11i.default\chrome

Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10


From: Pat Johnson notifications@github.com Sent: Saturday, December 29, 2018 8:12:53 PM To: overdodactyl/ShadowFox Cc: GTK66; Mention Subject: Re: [overdodactyl/ShadowFox] Tabs Bar Color (#233)

Thanks @loopy750https://github.com/loopy750 and @GTK66https://github.com/GTK66!

I added the code above (dark theme) with the following variables:

--tab-background-color --tab-color

which default to var(--tone-9) and var(--in-content-selected-text) respectively.

Let me know if this doesn't fix things and I'll re-open!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/overdodactyl/ShadowFox/issues/233#issuecomment-450532324, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ANSRmvAby5CQTdZm9Cq1fFQphbn_TBf1ks5u-BMVgaJpZM4ZkmH-.

GTK66 commented 5 years ago

This is what I was trying to tell you. So in order for SgadowFox to work properly from Fx 65 on out the code needs changed. See Below:

https://i.imgur.com/gXRtifc.png

Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10


From: Pat Johnson notifications@github.com Sent: Saturday, December 29, 2018 8:12:53 PM To: overdodactyl/ShadowFox Cc: GTK66; Mention Subject: Re: [overdodactyl/ShadowFox] Tabs Bar Color (#233)

Thanks @loopy750https://github.com/loopy750 and @GTK66https://github.com/GTK66!

I added the code above (dark theme) with the following variables:

--tab-background-color --tab-color

which default to var(--tone-9) and var(--in-content-selected-text) respectively.

Let me know if this doesn't fix things and I'll re-open!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/overdodactyl/ShadowFox/issues/233#issuecomment-450532324, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ANSRmvAby5CQTdZm9Cq1fFQphbn_TBf1ks5u-BMVgaJpZM4ZkmH-.

overdodactyl commented 5 years ago

No this did not work unless I placed in the wrong place. I placed it at user name\AppData\Roaming\Mozilla\Firefox\Profiles\bjaip11i.default\chrome

It's impossible for me to know if that is the correct location for you, but it should be in your userChrome.css file.

It was added to ShadowFox in 9d59a01 though, so if you use the updater/update your ShadowFox code, you will have it.

The code works on macOS to change the tab bar area... @loopy750 is the current version of SF working for you to adjust the tab area in Windows?

This is what I was trying to tell you. So in order for SgadowFox to work properly from Fx 65 on out the code needs changed. See Below:

That change is not relevant to ShadowFox; the significant change was that Firefox now styles the tab area based on your Windows theme. It didn't do this before, and as such I did not include any code to style that area (because the default dark theme handled it).

GTK66 commented 5 years ago

Then why did this work fine with Firefox 64, It does NOT work with 65. So Firefox made a change so in order for ShadowFox to work users will have to stay on Firefox 64 or change their Color settings in Windows 10. If this is what ShadowFox calls customer service then I do not need your program.

Don’t worry you will be getting many complaintrs.

Please do not respond as I am done with your POS product.

Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10


From: Pat Johnson notifications@github.com Sent: Sunday, December 30, 2018 3:29:38 PM To: overdodactyl/ShadowFox Cc: GTK66; Mention Subject: Re: [overdodactyl/ShadowFox] Tabs Bar Color (#233)

No this did not work unless I placed in the wrong place. I placed it at user name\AppData\Roaming\Mozilla\Firefox\Profiles\bjaip11i.default\chrome

It's impossible for me to know if that is the correct location for you, but it should be in your userChrome.css file.

It was added to ShadowFox in 9d59a01https://github.com/overdodactyl/ShadowFox/commit/9d59a013c712abcaa6ef8d9732f1ef2ef5181774 though, so if you use the updater/update your ShadowFox code, you will have it.

The code works on macOS to change the tab bar area... @loopy750https://github.com/loopy750 is the current version of SF working for you to adjust the tab area in Windows?

This is what I was trying to tell you. So in order for SgadowFox to work properly from Fx 65 on out the code needs changed. See Below:

That change is not relevant to ShadowFox; the significant change was that Firefox now styles the tab area based on your Windows theme. It didn't do this before, and as such I did not include any code to style that area (because the default dark theme handled it).

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/overdodactyl/ShadowFox/issues/233#issuecomment-450584678, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ANSRmiEq619ZC3Eh9A54A0jWBps15_n9ks5u-SIygaJpZM4ZkmH-.

overdodactyl commented 5 years ago

Then why did this work fine with Firefox 64, It does NOT work with 65

Because, as said before, Mozilla just started styling Firefox with Windows' colors. It has nothing to do with the other reasons you mentioned and ShadowFox has never styled the tab area.

users will have to stay on Firefox 64 or change their Color settings in Windows 10

No, I made changes (as stated above) to address it.

If this is what ShadowFox calls customer service then I do not need your program.

I directed you to the right location to submit an issue when you opened one that didn't even make sense in the wrong repository, and I subsequently made changes to the project to address the problem; I'm not sure what else you want.

I am maintaining this project in my free time along with the help of some others. I am not "customer service," and you should never expect that from any open-source project that does not seek or accept any form of payment.

I am done with your POS product

¯\(ツ)

Again, I believe the issue addressed here has been fixed. If anyone else finds that this is not the case, please open up a new issue and we will get it resolved there as I am locking this conversation.

loopy750 commented 5 years ago

The code works on macOS to change the tab bar area... @loopy750 is the current version of SF working for you to adjust the tab area in Windows?

Hi, works fine for me...

With Shadowfox: https://i.imgur.com/RPbhxxm.png Without shadowfox: https://i.imgur.com/4VWpugR.png

overdodactyl commented 5 years ago

Perfect, thanks for the help and the update!