stock/tango icons for folder pane

RESOLVED FIXED in Thunderbird 3.0b3

Status

Thunderbird
Folder and Message Lists
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: andreasn, Assigned: clarkbw)

Tracking

unspecified
Thunderbird 3.0b3
All
Linux
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(7 attachments, 4 obsolete attachments)

(Reporter)

Description

9 years ago
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.7) Gecko/2009030422 Ubuntu/8.10 (intrepid) Firefox/3.0.7 Ubiquity/0.1.5
Build Identifier: 

Thunderbird don't currently pick up the folder icons from the system, making them looking out of place. As for the other icons, we should use tango styled icons.

Reproducible: Always
(Reporter)

Comment 1

9 years ago
Created attachment 367743 [details]
modified folder pane css
(Reporter)

Comment 2

9 years ago
Created attachment 367744 [details]
additional icons for the folder pane

Updated

9 years ago
Assignee: nobody → nisses.mail
Status: UNCONFIRMED → NEW
Ever confirmed: true
Hardware: x86 → All
(Assignee)

Updated

9 years ago
Blocks: 415415
(Assignee)

Comment 3

9 years ago
we need to figure out how we're going to deal with the (new) indications.

Just to clarify upfront, Thunderbird keeps track of "new mail" which is different than "unread mail"; which it also keeps track of.  Unread mail is the (often) bold count of messages inside a folder which are not marked read.  New messages are ones that have arrived in a folder during the Thunderbird session.

Thunderbird uses alternate folder/mailbox/account icons with a (*) orange flash to indicate new messages.  

If we're going to use stock folder icons we need a strategy for marking those folders as new.
(Reporter)

Comment 4

9 years ago
Created attachment 368381 [details]
updated css

realized "treechildren::-moz-tree-line" needs to be set to "visibility: hidden;" in order to look good.
Attachment #367743 - Attachment is obsolete: true
(Reporter)

Comment 5

9 years ago
Created attachment 369287 [details]
updated folder pane

Realized we can call the icon as a background image and then use list-style-image as an emblem on top as the new indicator. As far as I can tell, there are no regressions of this approach, but I need to test it out some more.
Attachment #368381 - Attachment is obsolete: true
(Reporter)

Comment 6

9 years ago
Created attachment 369288 [details]
blank image emblem, to put in icons
(Reporter)

Comment 7

9 years ago
Created attachment 369289 [details]
"new" emblem, to put in icons
This makes the icon "move" in some conditions (focussed selection). I was able to work around this with

background-position: center center;
background-repeat: no-repeat;

for every note type (inbox, sent, drafts, ...). Maybe there's a better way to do this?
(Assignee)

Comment 9

9 years ago
there is a folder-unsent icon in here that I'm not sure what to do with.  seems like it was a mixup with folder-outbox... not sure.  I'll have the patch ready in a bit.
Created attachment 370776 [details] [diff] [review]
wip for the folder pane

here's the WIP that I have right now.

one thing that I see that's left to do is decide about the secure mail account icon.  I don't really see the point in differentiating the secure mail accounts from the non-secure ones w/ icons.  That only really makes sense to me if I had two copies of the same account, one secure and one insecure; but then the point of security would seem to be lost.  Perhaps there's just something I'm missing.

also I think we could use some height spacing between the rows.

I'll put up a screenshot of what it looks like right now so we can have something to look at.
Created attachment 370779 [details]
screenshot of patched folder tree
At least in theory, the secure account icon should answer the question "do I want to touch this account in a way that will send my password over the wire while I'm here at (Starbucks|CanSecWest)?"
I don't believe the % of people who would understand the implications of the difference makes sense, especially given the high likelihood that decisions about password sending are done at "remember this password" time.  

To increase security, it feels like making "remember this password" on insecure accounts req
sigh.

To increase security, it feels like making "remember this password" on insecure
accounts require more conscious understanding of the issues might make sense.  A different icon doesn't, to me.
My steps for going from a "safe" connection to an unsafe one would be "uncheck the boxes for check automatically and check at startup, hibernate, walk" though for people who don't automatically check for new, it would be "hibernate, walk." Would yours be "prefs, forget password, shutdown the program so the currently cached password is forgotten, walk, restart the program, cancel password prompt, repeat"?

Do you also have an alternate method for reminding me to check every few weeks on whether my remaining insecure server has become sensible enough to have a secure way to connect, which is primary effect the icons have for me?
I'm not sure I understand your first question.

My only point is that I think simply having a different icon doesn't actually provide enough information to 99.99% of users to result in actually safer behavior.  It might work for some like you with deep understanding of the security risks involved, but I think it's unrealistic to think that just an icon increases our effective security.  So if it takes Bryan & Andreas any time at all to deal with this, my advice would be that the time would be better spent working on other areas, in terms of teaching security implications.

If there is somewhere that I haven't seen some UI that explains the meaning of these icons at account creation time, then my point is somewhat negated.  I am operating on the assumption (because I haven't noticed UI like that) that the icon is the only indication of the difference in account types).

I think I'm getting an idea of your question about my point about account types & passwords -- I don't know what the right UX should be.  I would actually want to explore a lot richer UI for explaining MITM, identifying network hops between TB and the server, etc., but I think that's a lot harder, and definitely out of scope for this bug.

Comment 17

9 years ago
So what is the point of *not* having different icons? Other than the relatively small work to put a secure indicator on a couple of icons.
The point wasn't a size of work required to make it happen, be it small or large.  I already spent my evening making the patch and am working on others now.  I tried to fix each piece and understood what it was doing and this piece didn't make sense, so I didn't fix it.
(Reporter)

Comment 19

9 years ago
Magnus: it's doable, but as Bryan and David pointed out, the too tiny icons feels a bit too close to a secret handshake language.

Bryan: in your screenshot, I see a news.mozilla.org. Is that a newsgroup or a chat server? Will look into creating updated icons for those.

Comment 20

9 years ago
Probably doesn't matter much for one or two accounts, but there are people who have 10s of accounts, and then it may be a useful reminder when to connect.

news.mozilla.org is a news server. (Yeah, the current icons look a bit like a chat!)
Created attachment 371500 [details] [diff] [review]
updated WIP, this uses the account for every account type

this just updates the secure server account icons to to use the normal account icons.  I think this is ready for review.  We could open new bugs about a secure account icon and a newsgroup icon.
Assignee: nisses.mail → clarkbw
Attachment #370776 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Created attachment 372161 [details] [diff] [review]
[checked in] folder pane update

here's a final version

we're using folder icons as the background now with a blank/new icon for the list-style-image which meant I had to do some attrs of background-position in order to use this single image file that we've changed over to from the individual images.

thanks for looking at this!
Attachment #371500 - Attachment is obsolete: true
Attachment #372161 - Flags: review?(mkmelin+mozilla)

Updated

9 years ago
Summary: no stock icons and tango icons for sidebar → stock/tango icons for folder pane

Comment 23

9 years ago
Comment on attachment 372161 [details] [diff] [review]
[checked in] folder pane update

Nice! r=mkmelin

I notice message threading don't get connectors anymore. Haven't been good (properly connected) since before tb2 anyway so I'm fine with that.


> .folderSummary-previewText {
>-  color: grey; 
>+  color: GrayText; 

Tiny nit: trailing space ^^^

The "new" indicator on top of the inbox does look a bit messy. Will post a small screen shot.
Attachment #372161 - Flags: review?(mkmelin+mozilla) → review+

Comment 24

9 years ago
Created attachment 372217 [details]
screenshot - "new mail"

May depend on OS theme, I suppose...

Comment 25

9 years ago
Comment on attachment 372161 [details] [diff] [review]
[checked in] folder pane update

Checked in with the nit fixed, and using more specific .tab-icon-image selectors
changeset:   2377:a79528b03c85
http://hg.mozilla.org/comm-central/rev/a79528b03c85

Updated

9 years ago
Attachment #372161 - Attachment description: folder pane update → [checked in] folder pane update

Updated

9 years ago
Target Milestone: --- → Thunderbird 3.0b3
We could try looking at the new mail icon in the bottom right hand side so it wouldn't cover the inbox icon.  I opened bug 489377 for that
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED

Updated

9 years ago
Depends on: 507580
You need to log in before you can comment on or make changes to this bug.