Closed
Bug 483759
Opened 16 years ago
Closed 16 years ago
stock/tango icons for folder pane
Categories
(Thunderbird :: Folder and Message Lists, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.0b3
People
(Reporter: andreasn, Assigned: clarkbw)
References
Details
Attachments
(7 files, 4 obsolete files)
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•16 years ago
|
||
Reporter | ||
Comment 2•16 years ago
|
||
Updated•16 years ago
|
Assignee: nobody → nisses.mail
Status: UNCONFIRMED → NEW
Ever confirmed: true
Hardware: x86 → All
Assignee | ||
Comment 3•16 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•16 years ago
|
||
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•16 years ago
|
||
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•16 years ago
|
||
Reporter | ||
Comment 7•16 years ago
|
||
Comment 8•16 years ago
|
||
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•16 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.
Assignee | ||
Comment 10•16 years ago
|
||
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.
Assignee | ||
Comment 11•16 years ago
|
||
Comment 12•16 years ago
|
||
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)?"
Comment 13•16 years ago
|
||
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
Comment 14•16 years ago
|
||
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.
Comment 15•16 years ago
|
||
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?
Comment 16•16 years ago
|
||
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•16 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.
Assignee | ||
Comment 18•16 years ago
|
||
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•16 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•16 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!)
Assignee | ||
Comment 21•16 years ago
|
||
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
Assignee | ||
Comment 22•16 years ago
|
||
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•16 years ago
|
Summary: no stock icons and tango icons for sidebar → stock/tango icons for folder pane
Comment 23•16 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•16 years ago
|
||
May depend on OS theme, I suppose...
Comment 25•16 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•16 years ago
|
Attachment #372161 -
Attachment description: folder pane update → [checked in] folder pane update
Updated•16 years ago
|
Target Milestone: --- → Thunderbird 3.0b3
Assignee | ||
Comment 26•16 years ago
|
||
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
Closed: 16 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•