conema / monterail-fulldark

Monterail Full Dark, a Thunderbird theme inspired by the mockup of Monterail
Mozilla Public License 2.0
60 stars 12 forks source link

Light grey color on the Mail Toolbar... #22

Closed DXNWO closed 4 years ago

DXNWO commented 6 years ago

Can the light grey color on the Mail Toolbar and the light grey color when composing a new message be changed to a darker color? Such as, the dark color in the built-in dark theme (see screenshots). Thanks.

image

image

Cri85 commented 5 years ago

I agree. Also the tab "today", "yesterday" etc. in the mail list are too light, the white text can not be read

conema commented 5 years ago

Hi @Cri85, I saw that you are a graphic designer, I would thank you if you could find some time for create a new palette for the theme!

Cri85 commented 5 years ago

Hi @Cri85, I saw that you are a graphic designer, I would thank you if you could find some time for create a new palette for the theme!

Yes, I can do it, but I have no idea how works the theme customization...

Using Photoshop I can see that the primary palette is this: immagine

I like it, but some text need more contrast because the background is too light or dark.

conema commented 5 years ago

The real problem is that with the new version of TB, full themes will stop working (see #23), so, at the moment, I don't know if it's a clever thing to continue to develop this theme.

CountQuadula commented 5 years ago

Monterail Full Dark + TB 60.3.0 This is a nice design except for the "light grey" header. I've seen hints that this can be edited, but no tutorial. Please link detail instruction on how to change the color to darker grey. Thanks!

To help others

Go to the "extensions" folder, here there should be a file named "MonterailFullDark@conema.me.xpi" While Thunderbird is opened in background, delete the "MonterailFullDark@conema.me.xpi" file.

This file would not delete while TB was open. A workaround is to Drag the file to the Desktop, then Delete it.

conema commented 5 years ago

Monterail Full Dark + TB 60.3.0 This is a nice design except for the "light grey" header. I've seen hints that this can be edited, but no tutorial. Please link detail instruction on how to change the color to darker grey. Thanks!

Open with a zip/rar tool the file named "MonterailFullDark@conema.me.xpi", that is located into "extensions" folder (TB must be closed). Open chrome/MonterailOverlay/css/root.css and edit the 17th row that should have --background-color-light: #838689;, change #838689 with the color that you want. Save the file & start TB.

To help others

Go to the "extensions" folder, here there should be a file named "MonterailFullDark@conema.me.xpi" While Thunderbird is opened in background, delete the "MonterailFullDark@conema.me.xpi" file.

This file would not delete while TB was open. A workaround is to Drag the file to the Desktop, then Delete it.

That depends from the OS, but yes, in Windows the file can't be deleted while TB is opened.

DXNWO commented 5 years ago

Hello Conema, thanks for the information. I was able to change the the light grey on the Mail Toolbar to a darker color with the instructions you provided (see below).

mont1

DXNWO commented 5 years ago

Once again, thanks for the information. I noticed an odd font color change after making the change from earlier. I need to elaborate, the font color changes in the “To” and “Subject” fields. If either those fields are selected the text color is white but if neither are selected then the text turns black. This makes it hard to see with the dark color change previously described. See screenshots below, thanks again.

mont2

mont3

conema commented 5 years ago

Once again, thanks for the information. I noticed an odd font color change after making the change from earlier. I need to elaborate, the font color changes in the “To” and “Subject” fields. If either those fields are selected the text color is white but if neither are selected then the text turns black. This makes it hard to see with the dark color change previously described. See screenshots below, thanks again.

Resolved with the last commit, remember that if you update the theme, you should re-edit the root.css file. Let me know if all is okay.

PS: Open a new issue if you have any other issues.

CountQuadula commented 5 years ago

I have the same issue. How to fix the black text? "Resolved with the last commit" what does that mean?

conema commented 5 years ago

I have the same issue. How to fix the black text? "Resolved with the last commit" what does that mean?

That means that the files on this repository are been updated. These files may/may not be ready for automatic updates, for this one, it's not ready and you should update the theme with the manual installation mode (Download the file from the repo, zip the file -not the monterail-fulldark directory-, rename the zip as .xpi and install it through TB).

DXNWO commented 5 years ago

Hello Conema, thank you for updating the compose.css. Font/text color is good now in the To & Subject fields, great job.

CountQuadula commented 5 years ago

Download is 617kb, but TB is giving error"won't install because the file is corrupt".

conema commented 5 years ago

Download is 617kb, but TB is giving error"won't install because the file is corrupt".

May you send me a screenshot of the .xpi content?

CountQuadula commented 5 years ago

Screenshot of TB tb_addon

conema commented 5 years ago

As I said in my last comment, you should make a zip with the content of monterail-fulldark-master. So you should take the files and folder in the screenshot, create a zip file with only them into and rename it to .xpi. (Now you have .xpi->monterail-fulldark-master->files/folder (chrome/chrome.manifest/icon.png/etc...), you should have .xpi->files/folder) immagine

conema commented 5 years ago

@CountQuadula The update is being released with automatic update, so you can wait for it, just one/two day!

PS: Update is now available

CountQuadula commented 5 years ago

Thanks Conema, Apologies, I clearly misunderstood the instructions. This is not something I typically work on. Thanks for your patience. Now that i've edited the color to: #090909, it looks great! Thanks for the fix.

CountQuadula commented 5 years ago

The main (root) window is still blaring white, what controls that color? tb_main

conema commented 5 years ago

Thanks Conema, Apologies, I clearly misunderstood the instructions. This is not something I typically work on. Thanks for your patience. Now that i've edited the color to: #090909, it looks great! Thanks for the fix.

No problem!

The main (root) window is still blaring white, what controls that color?

This can't be done by changing some theme variable like last time, you should change the background/text color of the #acctCentralGrid container with CSS. Something like #acctCentralGrid { background-color: #some-color; color: #some-color; }

CountQuadula commented 5 years ago

Haha, Yea, I'm lost already. Where is "#acctCentralGrid container "?

conema commented 5 years ago

Haha, Yea, I'm lost already. Where is "#acctCentralGrid container "?

Open the same file as before, root.css, go at the end of the file, after the last } and paste this: #acctCentralGrid { background-color: #some-color; color: #some-color; }. Remember to change #some-color, with the color that you want. Save the file and do the same thing that you did for changing the other color (zip, xpi, etc...).

PS: If you want more help to learn of to tweak things, you should go to mozillazine, a forum about this sort of things

CountQuadula commented 5 years ago

There are two "#some-color;" Do they need to be the same Color? e.g. `}

acctCentralGrid { background-color: #090909r; color: #090909;

}`

conema commented 5 years ago

There are two "#some-color;" Do they need to be the same Color? e.g. } #acctCentralGrid { background-color: #090909r; color: #090909; }

Nope, "background-color" is for the background, "color" is for the text color

CountQuadula commented 5 years ago

Here is the last two code lines, I've tried it with "} ." and without "." with no change. `#facet-date text { fill: var(--headers-background-color); }

/ TB Mail Main Screen /

acctCentralGrid {

background-color: #363636; color: #838689;

}`

conema commented 5 years ago

Here is the last two code lines, I've tried it with "} ." and without "." with no change. `#facet-date text { fill: var(--headers-background-color); }

/ TB Mail Main Screen /

acctCentralGrid {

background-color: #363636; color: #838689;

}`

Mhmh, I think that it's better to put this code into userChrome.css, but it's a thing that doesn't concern the theme, so you would find more useful to open a thread on mozillazine to ask for help!

conema commented 4 years ago

I changed the color in the last commit (after a long time)