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

Current Folder Toolbar display - text is always black #81

Closed
Vacoas opened this issue Nov 30, 2020 · 16 comments
Closed

Current Folder Toolbar display - text is always black #81

Vacoas opened this issue Nov 30, 2020 · 16 comments
Assignees

Comments

@Vacoas
Copy link

Vacoas commented Nov 30, 2020

Hi,
I am currently using Thunderbird 78.5.0 and QuickFolders Pro 5.0.
In the main toolbar, the font used to display the tab varies depending on the tab colour (e.g. White for dark blue, black for yellow).

image

The same does not apply to the Current Folder toolbar. The tab name is always in black, making it difficult to read in some cases.

image
image
image

Regards,
MP

@RealRaven2000
Copy link
Owner

RealRaven2000 commented Nov 30, 2020

I did some testing with dark themes on Tb68 vs 78. apparently the selector for the button is not specific enough. Please try out this version:

QuickFolders-wx-5.1pre9.zip

  • I also added a rule for the (svg - based) icon color to harmonise with the selected text for "uncolored" tabs.
  • I think I may need to modify the rules for hovered colors as well to make them "override" the default color when you hold the mouse pointer over an uncolored tab.

To install, download the zip file and then drag into Add-ons Manager!

@RealRaven2000
Copy link
Owner

I added an additional selector for label and icon on the "hovered" state, as well:

Uploading QuickFolders-wx-5.1pre10.zip…

To install, download the zip file and then drag into Add-ons Manager!

@RealRaven2000 RealRaven2000 changed the title QuicKFolders Current Folder Toolbar display Current Folder Toolbar display - font color always black Nov 30, 2020
@RealRaven2000 RealRaven2000 pinned this issue Nov 30, 2020
@RealRaven2000 RealRaven2000 changed the title Current Folder Toolbar display - font color always black Current Folder Toolbar display - text is always black Nov 30, 2020
@Vacoas
Copy link
Author

Vacoas commented Dec 2, 2020

Just tested 5.1pre24.
Folders for which there is NO tab are displayed correctly in the current folder toolbar :
image
Folders for which there IS a tab are not correctly displayed :
image

I am also running into a problem installing the version which seems to be Thunderbird-related and I had to copy the 'chrome' folder and others manually. More about that below.

@Vacoas Vacoas closed this as completed Dec 2, 2020
@Vacoas Vacoas reopened this Dec 2, 2020
@RealRaven2000
Copy link
Owner

Just tested 5.1pre24.
Folders for which there IS a tab are not correctly displayed :
image

I can't reproduce this [coloring works always even if tabs of the current folder exist] - did you restart Thunderbird just in case old CSS rules were "hanging around"? What color settings have you set for "Uncolored tabs" - it would be best if you could post a screenshot of your color settings (bling my tabs) in the QuickFolders options dialog.

As example, here are my settings for testing with a dark theme:
image

please leave ths issue open until I officially release 5.1 on atn.

@Vacoas
Copy link
Author

Vacoas commented Dec 2, 2020

image
I did restart Thunderbird every time I loaded something new.

@RealRaven2000
Copy link
Owner

RealRaven2000 commented Dec 2, 2020

I just tried your color settings on my system still can't reproduce - I believe it may be theme colors specific to your Operating System - which one are you on?

You could inspect the current folder element via tools / developer tools / developer toolbox and the inspector:
image

once you activate the element picker, click on the current folder panel:
image

Expand the toolbarbutton element in the left hand pane and click on label:
image

on the right hand side select "Computed" (Styles) and expand the color rule:
image

you should see the "winning" rule / selector on top. just paste a screenshot of this set of rules and we can go forward. It is interesting to know the exact selector with the black color, the css file name and line number.

PS I forgot - if you can't find the rule that causes the dark text color, it may help to see the "full picture" if you activate the [x] Browser Styles checkbox on the top right:

image

this will include all "ground-rules" from Thunderbird.

@Vacoas
Copy link
Author

Vacoas commented Dec 3, 2020 via email

@Vacoas
Copy link
Author

Vacoas commented Dec 5, 2020

Here are the steps I performed.
Starting Thunderbird with a current folder such as 'Contrepoints'.
Selecting 'Figaro' in the folder pane.
This seems to generate two actions as I have 2 entries for 'toolbarbutton-text' and 'Figaro'.
The first entry has the right color for the text: rgb(220,235,245).
The second entry comes back with rgb(0,0,0) which seems to overwrite the previous entry.

Figaro 01_cr
Figaro 02_cr
Figaro 03_cr
Figaro 04_cr

@RealRaven2000
Copy link
Owner

Hi, I am running Windows 10 20H2 with all the most recent updates 19042.662. To perform the testing I removed all extensions except QuickFolders to make sure they do not interfere. Especially, the DeepDark extension that uses specific files in a chrome directory. The installation of 5.1pre24 went fine. I first tested the Current Folder Toolbar for a folder which does not have a tab Then I did the same thing with a folder that has a tab I find the toolbarbutton-text entry for 'Atlantico' twice : Regards, Maurice Le 02/12/2020 à 15:30, RealRaven2000 a écrit :

I just tried your color settings on my system still can't reproduce - I believe it may be theme colors specific to your Operating System - which one are you on? You could inspect the current folder element via tools / developer tools / developer toolbox and the inspector: once you activate the element picker, click on the current folder panel: Expand the toolbarbutton element in the left hand pane and click on label: on the right hand side select "Computed" (Styles) and expand the color rule: you should see the "winning" rule / selector on top. just paste a screenshot of this set of rules and we can go forward. It is interesting to know the exact selector with the black color, the css file name and line number.

If you want to send screenshot, please do it from github directly. Directly replied mails are very hard to read. (I edited your first post to make it a little easier, but screens were missing)

@RealRaven2000
Copy link
Owner

RealRaven2000 commented Dec 5, 2020

If this image is your "Computed styles":

image

.. you may have to expand the first two lines so the full rule is shown. We need to find out which rule has higher specificity and why.

@Vacoas
Copy link
Author

Vacoas commented Dec 5, 2020 via email

@RealRaven2000
Copy link
Owner

RealRaven2000 commented Dec 5, 2020

I have taken out the named rule for now - even though it is inherited from the parent element (the button) it may give problems because of higher specificity - see:
https://specifishity.com/

New version below:
QuickFolders-wx-5.1pre35.zip

I am hoping the missing colors on that element won't matter, but we will see. I can add a class-based rule later.

Installation: download the zip file above. Then drag the file into Thunderbird Add-ons Manager.

@Vacoas
Copy link
Author

Vacoas commented Dec 5, 2020 via email

@RealRaven2000
Copy link
Owner

Version 5.1pre35 does fix the issue. The chart to illustrate the concept of specificity is interesting and funny. I am afraid it is a bit beyond my grasp but I get the general idea. Would like me to provide some more debugging information ? Many thanks for your help. Regards, Maurice

No, it's fine. I had to do a few more changes for dark mode to make the standard button (uncolored) text and icons color work.
image
Here is the latest version for testing:

QuickFolders-wx-5.1pre42.zip

Installation: download the zip file above. Then drag the file into Thunderbird Add-ons Manager.

PS: Remember if you reply vie email, just cut out all the quoted text at the bottom because otherwise I have edit all your posts which is extra work. I would prefer if you could go to github directly (you can even paste screenshots here) but it's ok if you can't do that.

@Vacoas
Copy link
Author

Vacoas commented Dec 7, 2020

I have installed 5.1pre42 and performed a number of tests.
So far, I have not encountered any problem.

@RealRaven2000
Copy link
Owner

fixed in release 5.1 on 10/Dec/2020

@RealRaven2000 RealRaven2000 self-assigned this Dec 13, 2020
@RealRaven2000 RealRaven2000 unpinned this issue Dec 13, 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