Overlay grain.png on the Folder Pane

RESOLVED FIXED in Thunderbird 22.0

Status

Thunderbird
Folder and Message Lists
--
enhancement
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: JosiahOne, Assigned: JosiahOne)

Tracking

({ux-consistency})

unspecified
Thunderbird 22.0
ux-consistency
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(9 attachments, 8 obsolete attachments)

59.84 KB, image/png
Details
189.53 KB, image/png
Details
188.19 KB, patch
Paenglab
: review+
bwinton
: ui-review+
Details | Diff | Splinter Review
348.44 KB, image/png
Details
552.37 KB, image/png
Details
323.71 KB, image/png
Details
306.22 KB, image/png
Details
33.71 KB, patch
Details | Diff | Splinter Review
227.34 KB, image/png
Details
Note: Part of a blog post we are pushing to update TB UI.
======================

http://infinite-josiah.blogspot.com/2013/02/thunderbird-ui-concept.html

See point 6 there for more details on this.
======================

Basically in order to update the UI and be more consistent with our current style, we should overlay the grain.png image over the blue color, and that would make it look more like our website and new tab page on Firefox.

Not very important, but a welcome change.

Grain is available at: https://assets.mozillalabs.com/Websites/Sandstone/Backgrounds/
Assignee: nobody → josiah
Created attachment 719105 [details] [diff] [review]
Patch.

Patch which overlays the grain.png image.
Created attachment 719106 [details]
Current Folder Pane

Screenshot showing the default FolderPane.
Created attachment 719107 [details]
New "grain" folderpane

Image showing the folder pane with grain. Requesting UI review from Blake.
Attachment #719107 - Flags: ui-review?(bwinton)
Created attachment 719556 [details] [diff] [review]
Patch.

Updated patch SHOULD bring grain texture to all platforms. However, I am not able to verify any platform except for OS X. If someone could verify that would be great.
Attachment #719105 - Attachment is obsolete: true
Attachment #719556 - Flags: ui-review?(bwinton)
Created attachment 719563 [details] [diff] [review]
Patch.

Fixed oversight.
Attachment #719556 - Attachment is obsolete: true
Attachment #719556 - Flags: ui-review?(bwinton)
Attachment #719563 - Flags: ui-review?(bwinton)
Created attachment 719615 [details] [diff] [review]
Patch.

Excluding Windows Classic theme from the grain texture. Also removed unnecessary code concerning Aero.
Attachment #719563 - Attachment is obsolete: true
Attachment #719563 - Flags: ui-review?(bwinton)
Attachment #719615 - Flags: ui-review?(bwinton)
Created attachment 719619 [details] [diff] [review]
Patch.

Added grain to Linux.
Attachment #719615 - Attachment is obsolete: true
Attachment #719615 - Flags: ui-review?(bwinton)
Attachment #719619 - Flags: ui-review?(bwinton)
Attachment #719619 - Flags: review?(richard.marti)

Comment 8

4 years ago
(In reply to Josiah [:JosiahOne] from comment #6)
> Excluding Windows Classic theme from the grain texture.

Yes, that sounds like a good idea. The effect is subtle but nevertheless may appear a bit distracting with the Windows Classic and especially the High Contrast desktop themes.
Comment on attachment 719619 [details] [diff] [review]
Patch.

Code looks good r+
Attachment #719619 - Flags: review?(richard.marti) → review+
Comment on attachment 719619 [details] [diff] [review]
Patch.

Subtle, but nice.  ui-r=me.

(Thanks for the ping!)
Attachment #719619 - Flags: ui-review?(bwinton) → ui-review+
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/f24a3796b2f4
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 22.0

Comment 12

4 years ago
Comment on attachment 719107 [details]
New "grain" folderpane

I think that you can cancel the ui-review? request here now that the patch has been checked in.
Comment on attachment 719107 [details]
New "grain" folderpane

Canceling unneeded ui review.
Attachment #719107 - Flags: ui-review?(bwinton)
Shouldn't the chat's contact list also become this grain to be consistent?

Comment 15

4 years ago
(that was spun off as bug 849215.)

Comment 16

4 years ago
I can't say I see any difference in the screenshots. I wonder how much memory this invisible effect takes.
(In reply to :aceman from comment #16)
> I can't say I see any difference in the screenshots. I wonder how much
> memory this invisible effect takes.

It's actually quite visible, but perhaps if you have a weaker monitor it does not appear visible. However, the file itself is 47 KB. Of course memory increase is not significant, but is there.

Comment 18

4 years ago
I don't think the majority of TB users are some graphics professionals with "better than weak" monitors.

I can barely see the grain on a 24" IPS monitor (not calibrated), but only because you told me to look closely. If I didn't know what to look for in the shots, I'd consider it noise in the monitor.
(In reply to :aceman from comment #18)
> I don't think the majority of TB users are some graphics professionals with
> "better than weak" monitors.
> 
> I can barely see the grain on a 24" IPS monitor (not calibrated), but only
> because you told me to look closely. If I didn't know what to look for in
> the shots, I'd consider it noise in the monitor.

My monitor is not meant for any graphic professionals. My monitor only cost $200, so that says a lot.

As for your issue, I realized on Linux the Folderpane is gray, so it will not show up well. Currently pondering on what to do about that...

Comment 20

4 years ago
Created attachment 723997 [details]
Screenshot of the grain on Linux
Created attachment 724000 [details]
Full Window on OS X with grain.

Here's a screenshot of the effect on a full window. You must make sure zoom is at 100% for it to be pretty clear.
Created attachment 724383 [details] [diff] [review]
Linux Grain Patch, Concept 1

Alright, this should work as a darker version of the grain that affects Linux only. Ace, what do you think of this? Does it show up better or worse?
Flags: needinfo?(acelists)
Created attachment 724511 [details] [diff] [review]
Linux Grain Patch, Concept 2

Talked to Ace on IRC. That patch did not work, hopefully this one contrasts better.
Attachment #724383 - Attachment is obsolete: true
Flags: needinfo?(acelists)

Comment 24

4 years ago
Created attachment 724518 [details]
screenshot of Linux with concept 2

It is visible now, but probably too much.
Created attachment 724529 [details] [diff] [review]
Linux Grain Patch, Concept 3

This should tame down the sharpness. If not, I can do it once more. Hopefully this will be better.
Attachment #724511 - Attachment is obsolete: true
Flags: needinfo?(acelists)

Comment 26

4 years ago
Created attachment 724562 [details]
screenshot of Linux with concept 3
Flags: needinfo?(acelists)
(In reply to :aceman from comment #26)
> Created attachment 724562 [details]
> screenshot of Linux with concept 3

Ace, how does this look for you? Is it to grainy? If it looks fine then we could land this one for Linux.

Comment 28

4 years ago
I've attached the screenshot with concept 3. I do see the grain there :)
(In reply to :aceman from comment #28)
> I've attached the screenshot with concept 3. I do see the grain there :)

Yes, but to me that looks a little to grainy still? Do you agree?

Comment 30

4 years ago
Can you make one in the middle between the windows version and concept 3 ?
Created attachment 728277 [details] [diff] [review]
Linux Grain Patch, Concept 4

Well, my previous image was somehow deleted. But I created a new one. Hopefully this is in between the two.  Try it out and let me know how it is.
Attachment #724529 - Attachment is obsolete: true
Flags: needinfo?(acelists)

Comment 32

4 years ago
That is invisible again.
Flags: needinfo?(acelists)
Created attachment 728296 [details] [diff] [review]
Linux Grain Patch, Concept 5

And again. Here's another.
Attachment #728277 - Attachment is obsolete: true
Flags: needinfo?(acelists)
Attachment #728296 - Attachment is patch: true

Comment 34

4 years ago
Created attachment 728307 [details]
screenshot of Linux with concept 5
Flags: needinfo?(acelists)
(In reply to :aceman from comment #34)
> Created attachment 728307 [details]
> screenshot of Linux with concept 5

Thanks, this looks pretty good. I think we could land this.
Bug 853926

Comment 37

4 years ago
This was a bad surprise when starting thunderbird 24.

It looks really bad with a dark theme, for it predates the background color of the theme.
http://www.kirikoo.net/images/14Anonyme-20131031-202254.jpg

Not everyone is using a default gray / bright theme! Thunderbird supported perfectly dark themes until this was added.
Flags: needinfo?(josiah)
(In reply to calimeroteknik from comment #37)
> This was a bad surprise when starting thunderbird 24.
> 
> It looks really bad with a dark theme, for it predates the background color
> of the theme.
> http://www.kirikoo.net/images/14Anonyme-20131031-202254.jpg
> 
> Not everyone is using a default gray / bright theme! Thunderbird supported
> perfectly dark themes until this was added.

Sorry for not getting back to you sooner... 

But yes, that is definitely a severe problem and I am currently investigating a fix. I will file a bug on Monday and add those details to this bug. 

Thanks for the report, 
Josiah Bruner
Flags: needinfo?(josiah)
Depends on: 935023

Comment 39

4 years ago
Many thanks to you for the fix!

Comment 40

4 years ago
NO NO NO NO NO -- THERE SHOULD BE NO OVERLAY ON THE FOLDER PAGE. This kind of garbage drives users nuts. I use a dark background and the botched overlay in 24 and 25 rendered my light text unreadable and there was no way to turn it off (without hacking css files)

This whole line of thinking is flawed. If you want something added cosmetically, then DO IT TO YOUR OWN thunderbird install - DO NOT SCREW MINE UP. I don't want your setup, I want mine.
Flags: needinfo?(josiah)

Comment 41

4 years ago
David, please bug 935023.

Comment 42

4 years ago
When you do things like this, you cause bug reports to be written and scarce user and admin time to be spent trying to solve the problems one user choice has created: see, eg.:

https://bugs.archlinux.org/task/36979

Seriously, if you like the overlay, then add it to your own userChrome.css, do not make millions have to find a way to hack userChrome.css to remove what you think looks good. I understand progress and also what a "default" theme should be, and above all:

The default theme for anything should remain a simple, clean and plain "default" that is consistent between versions

We in opensource development would all do well to remember this. It is frustrating to spend hours trying to make tbird usable again, just because somebody decided to impose his thought of what "he likes" on the entire userbase without consideration that he might be screwing up the usability for everyone with a dark OS desktop theme involve.

You do great work, but a "default" should remain a consistent "default" between versions from an aesthetics standpoint. Enough said, this is just a very frustrating situation for all loyal tbird users that have dark OS themes.

Comment 43

4 years ago
That's bug 949389 now which you've opened, thus please let's keep the discussion there given that this bug is closed (and has been obsoleted by bug 935023 anyway, at least on trunk).
Flags: needinfo?(josiah)

Updated

4 years ago
Depends on: 949389
(In reply to David Rankin from comment #40)
> NO NO NO NO NO -- THERE SHOULD BE NO OVERLAY ON THE FOLDER PAGE. This kind
> of garbage drives users nuts. I use a dark background and the botched
> overlay in 24 and 25 rendered my light text unreadable and there was no way
> to turn it off (without hacking css files)
> 
> This whole line of thinking is flawed. If you want something added
> cosmetically, then DO IT TO YOUR OWN thunderbird install - DO NOT SCREW MINE
> UP. I don't want your setup, I want mine.

That's has been fixed in version 28 by bug 935023. I have just requested uplift there, so your issue should be fixed in the next dot release.

Also, do *not* needinfo me just because you have an issue with a change. It interrupts workflow and is frankly just pretty rude. Bugzilla is not really the place for that kind of discussion, however I am always open to email (just be respectful about it).

Thanks,
Josiah
You need to log in before you can comment on or make changes to this bug.