Closed
Bug 74357
Opened 24 years ago
Closed 15 years ago
text styles for collapsed threads with unread children
Categories
(SeaMonkey :: MailNews: Message Display, defect)
SeaMonkey
MailNews: Message Display
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: sspitzer, Unassigned)
References
(Depends on 1 open bug, Blocks 1 open bug)
Details
(Whiteboard: [Halloween2011Bug])
4.x mac would underline the subjects (maybe more) of collapsed threads that had
unread messages.
we should do this in the 6.x classic mac skin. should we do it in modern?
Comment 1•24 years ago
|
||
I think this should belong to andreww or hewitt?
Comment 2•24 years ago
|
||
No, this isn't a themes issue, it requires marking some attribute in content to
indicate a descendant message of a thread is unread. Not sure if this is being
done yet. If it is, then it becomes a themes issue.
64477 touched on this topic. Using underline for collapsed threads with unread
children seems like a good idea for all platforms/skins. /And doing away with
the green arrow since the green arrow is used everywhere else to mean "newest"
(not "unread"). Thoughts?
From 64477:
------- Additional Comments From mpt 2001-01-18 10:03 -------
Underlining doesn't look unread, but gets the `this thread contains unread
messages' idea across much better than just the green arrow does. Try 4.x for
Mac sometime -- it's groovy.
------- Additional Comments From jglick@netscape.com 2001-01-18 10:42 -------
I think using the bolding is bad because the parent message isn't unread. I also
think changing the icon to have the green arrow is bad because we are using the
green arrow in Mail to mean "Newest" messages (which is also unread, but unread
doesn't always equal "newest"). Underlining seems to work ok.
Comment 4•24 years ago
|
||
I like the 4.x mac style of:
text fields (subject, date, sender, etc) of closed threads with unread are
underlined.
the fix will take me a minute, please update the spec (and add a link to it) if
you want me to do it.
I'll make modern and mac classic do it.
are you sure you want this for unix / win classic? I'm ok with that decision,
but I'm not sure if the goal of classic is to exactly match 4.x or not.
Status: NEW → ASSIGNED
Comment 5•24 years ago
|
||
note, right now we don;t bold the parent if the parent is unread (and has unread
children).
so we are doing everything right except for the underline.
fix on the way.
>are you sure you want this for unix / win classic? I'm ok with that decision,
>but I'm not sure if the goal of classic is to exactly match 4.x or not.
Would like to hear others opinion on this one. If we did something on one
platform for 4.x that people think was a good idea, I don't think its a problem
to incorporate it on all platforms for classic (as well as other skins).
Joe, is classic trying to duplicate 4.x exactly for each platform?
i'd like the underline although i think it does conflict w/ the definition of
classic.
Comment 8•24 years ago
|
||
The original intent of Classic wasn't to mimic 4.x in every regard, its
original designer (Ben) has said so himself. It was more of an "embrace and
extend" technique.
That said, Outlook Express just bolds the thread titles, and I like that best
(I can see lots of underlines making it hard to read, and the bolding is
consistent with read/unread email messages...). I can understand the arguments
against that, though (the parent message has been read, but is still bold).
cc'ing other usability people for their input
so, let's just go w/ a class that can be styled in userChrome.css.
Comment 10•24 years ago
|
||
The fact that 1% of users can change this if they know how to edit their
userChrome.css file shouldn't play a role in this usability decision at all...
Updated•24 years ago
|
QA Contact: esther → nbaca
Reporter | ||
Comment 11•24 years ago
|
||
my part of the fix for this is in.
blocked by #74663 (underlines don't show up in the outliner)
Depends on: 74663
Reporter | ||
Comment 12•24 years ago
|
||
since "underlined" will indicate if a thread has unread children or not, should
the icon in the thread column only show the green arrow if the thread contains
any "new" messages?
note, 4.x linux showed the green arrow if the thread contained any unread.
Comment 13•24 years ago
|
||
>since "underlined" will indicate if a thread has unread children or not, should
>the icon in the thread column only show the green arrow if the thread contains
>any "new" messages?
The "green arrow" is used everywhere else in the app to mean "newest" mail, not "unread".
If we want to be consistent with that the threaded icon should follow this also.
>note, 4.x linux showed the green arrow if the thread contained any unread.
I believe 4.x Mac and Win did also (but that doesn't mean it was right).
Comment 14•24 years ago
|
||
I agree that getting rid of the green arrows in the message pane would be a good
thing.
Does anyone have a screen shot of 4.x mac in threaded mode. As a Windows user
who hasn't seen this yet, I feel like all of the underlines might make the
screen even busier than it already is.
Reporter | ||
Comment 15•24 years ago
|
||
the underlines are a nice touch, I think you'll like them.
once we have them, it will be a snap to fix it so the green only shows in the
thread column thread pane if there are new messages in the thread.
Comment 16•24 years ago
|
||
ok, no need for a screenshot.
This appears to be working already.
At first glance it seems a little strange to me, but I could get used to it.
Reporter | ||
Comment 17•24 years ago
|
||
getting a mac 4.x screen shot now...
Comment 18•24 years ago
|
||
Actually, I think this looks really good for threads that aren't consecutive.
It only looks strange to me now when you have a whole bunch of collapsed threads
next to each other.
Anyway, as I said before, this is working for me in modern on 20010404 on Win 2000
Reporter | ||
Comment 19•24 years ago
|
||
it wasn't working for me on linux. but I'll wait until my build is done to make
sure.
it could be related to the other possible issues we have with outliner and fonts
(with desenders not showing up.)
since it works somewhere, we can mark this fixed and then change 74663 to be a
linux issue.
Reporter | ||
Comment 20•24 years ago
|
||
since this works in w2k, I'll mark this fixed.
I'll start a new bug to discuss not showing the green arrow in the thread icon
column unless the thread has new messages.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 21•24 years ago
|
||
see #74663 for the platform specific issues with "underline" not working.
Comment 22•24 years ago
|
||
SS: Where was that new bug?
Comment 23•24 years ago
|
||
skewer, the bug you want is bug 58788 [#74663 was marked a up of that].
Comment 24•24 years ago
|
||
Er, not that one. The one about the new icon and underline appearing on threads
that didn't contain any new messages.
Comment 25•23 years ago
|
||
I'm not seeing the underlines anymore in 2001060504 Win98. I won't reopen this
unless someone else can confirm (mainly because I think these are kind of ugly
and may have intentionally been removed, and are not needed with the green "new"
arrow icon).
Comment 26•23 years ago
|
||
Note that I know why underlining only worked on Windows, and I actually patched
the text decoration painting code for nsTextBox when I landed 78695. Outliner
needs to have the same patch applied, and then underline and strikethru and
overline would all work on all platforms.
Comment 27•23 years ago
|
||
Build 2001-06-18-04: WinMe
Build 2001-06-18-08: Linux RH 6.2
Build 2001-06-18-05: Mac 9.04
Reopening since the underline is not appearing for threads with unread children.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 28•23 years ago
|
||
nbaca, had this ever worked for you on win32? just curious...
anyhow, the [other] platform issues are covered in bug 58788...
Reporter | ||
Comment 29•23 years ago
|
||
note, this works on win2k (or did at one point.) I think it still does, but
that might be display specific.
I need to follow up with hyatt about how to fix it.
hyatt wrote: "Note that I know why underlining only worked on Windows, and I
actually patched the text decoration painting code for nsTextBox when I landed
78695. Outliner needs to have the same patch applied, and then underline and
strikethru and overline would all work on all platforms."
accepting until it is fixed in a way that works for all displays.
Status: REOPENED → ASSIGNED
Comment 30•23 years ago
|
||
Using a build from a couple of days ago on Win 2000, this no longer worked.
Comment 31•23 years ago
|
||
I saw this working at one point too, on Windows.
(This would explain why I tried to use text-decoration: underline in another
outliner and it didn't work...)
OS: Mac System 8.5 → All
Hardware: PC → All
Comment 32•23 years ago
|
||
*** Bug 90038 has been marked as a duplicate of this bug. ***
Comment 33•23 years ago
|
||
*** Bug 97095 has been marked as a duplicate of this bug. ***
Updated•23 years ago
|
Updated•23 years ago
|
QA Contact: nbaca → olgam
Comment 34•23 years ago
|
||
If we're at it (underline has problems), maybe it would be better to use bolding
instead.
If you argue that the parent message is read and only some children are unread,
then we might use bolded text of dark grey color (so that it looks like "dimmed"
black bolded text). It means "some but not all children included" in similar
contexts - for example in different installers that allow hierarchical package
selection (MS Office, Linux distros, OpenOffice...).
Given that underlining doesn't currently work it's _very_ hard to tell what
threads are unread.
The green-arrow thing is something counter-intuitive. I consider myself a power
user, and I didn't notice it meant "unread thread". I discovered the green arrow
only when I started to search for this bug on bugzilla... I was actually looking
for a bug that would say "Collapsed unread thread isn't bold"...
Anyway, since:
1. that bug has been unresolved for such a long time because of decorations not
quite working on all significant platforms
2. it still can be argued whether the underline stuff won't just confuse the
majority of both Outlooks users
...maybe we should do it the 95%-of-users-expected way, that is the way it is in
Outlooks. At least temporarily. This should be simple, shouldn't it?
Updated•23 years ago
|
Keywords: mozilla0.9.7
Updated•23 years ago
|
Whiteboard: se-radar
Comment 35•23 years ago
|
||
Gray is universally recognized as the "disabled" color in Windows. Furthermore,
you have to be careful that in such an instance you choose an OS- or
skin-appropriate color. Obviously if all my windows have gray backgrounds with
white text, gray text is a bad idea.
Comment 36•23 years ago
|
||
I vote for bold. I think that when we implement the labels feature, the bolding
might make more sense to a user as opposed to underlining.
Comment 37•23 years ago
|
||
FYI - Labels: 81292
Updated•23 years ago
|
Comment 38•23 years ago
|
||
*** Bug 120673 has been marked as a duplicate of this bug. ***
Comment 39•23 years ago
|
||
win32 build 2002011703, win98se
Up until sometime around this build, collapsed threads with unread children had
no special styling, at least on Win98. They are now underlined. This pretty
much sucks when the view is Threads with Unread - virtually every thread is
underlined and, IMO, seriously impairs readability of the threadpane.
Some folks must think this styling is beneficial, or it wouldn't have been
implemented in the first place (whoever put the comments in threadPane.css
thought it was "slick", but that's a matter of opinion). I ended up turning it
off completely in userChrome.css because most ngs I frequent are in Threads with
Unread, and I had serious difficulty reading the threadpane with all the
underlines. A color property would be much easier on the eyes. Underlines stink.
Comment 40•23 years ago
|
||
Is this really a nsbeta1+, P2? If yes, then we need to try and targeted to a
milestone M1.0 or earlier to make the beta.
Comment 41•23 years ago
|
||
<quote src="news://news.mozilla.org/3C486ADA.5050507@beonex.com" author="me">
Arg! Please don't use underlines *anywhere*. underlines are an ugly
leftover from typewriting machines. Please use italics, bold, color,
whatever, but let us forget underlines forever.
</quote>
Bold usually means unread. For me, it would make sense, if collapsed threads
with unread were bold, because that top-level post then stands for the whole
thread. As soon as followups are visible, the bolding of the top-level post
depends on if that post itself has been read. If we would allow subthread to be
collapsed, the same would apply to them.
Comment 42•23 years ago
|
||
There is one problem with my suggestion: The user can't see anymore, which
threads he started to read (but didn't finish or which have new posts). Maybe
Aleksander's suggestion in comment 34 is a good idea.
> Gray is universally recognized as the "disabled" color in Windows
"There are different shades of grey". ;-P Seriously, disabled is some mid-grey,
while Aleksander spoke about dark grey. It is quite possible to differentiate them.
Comment 43•23 years ago
|
||
I like Ben's idea for using bold.
As for how to indicate which threads have unread children...
There's an icon next to the subject, which indicates newly downloaded messages
with a little green arrow.
Why not add a third state to that icon to indicate whether or not a message has
been read? (in addition to going from bold to non-bold). A collapsed thread
with unread children will have an icon that indicates that the message is read
(if the parent is indeed read), but it will be bold, indicating that there are
unread children.
I realize that it's very Outlook Express-ish, but the concept works well.
Comment 44•23 years ago
|
||
You can test this easily:
- create <profiledir>/chrome/userChrome.css
- add the following lines:
-----------
outlinerchildren:-moz-outliner-cell-text(container, closed, hasUnread, read) {
text-decoration: none !important;
color: darkgrey !important;
font-weight: bold !important;
-----------
To my eye there does not seem to be a good shade of dark gray for this: either it is
- too dark to be distinguishable from black & bold
- too medium so that it is not too distingushable from black & normal-weight
- too light so it's, well, light, and rather disabled-looking perhaps
Anyway, the underline is not nearly as messy as I'd thought it would be, it
actually looks neat, and is symbolically logical (stuff [unread] _under_ this one).
Comment 45•23 years ago
|
||
I found that a color property does not always work well.
outlinerchildren:-moz-outliner-cell-text(container, closed, hasUnread, read) {
text-decoration: none !important;
color: darkgrey !important; (or any other color of your choosing)
When the top-level message in the thread is selected and there are still unread
children, the subject line in the threadpane is no longer readable against the
background color of the selected message. If the threadpane loses focus, i.e.
click in the message view pane, the selected message background color changes,
but it may still be tough to read depending on the text color (gray on gray is
not very good). This is likely to be more of a problem in Classic than Modern,
due to the unknown color combinations used in system settings. If colors are
used, may need some more styles to account for selected message in a collapsed
thread with unread children.
Comment 46•23 years ago
|
||
That can be enhanced by adding new CSS rule:
outlinerchildren:-moz-outliner-cell-text(selected, container, closed, hasUnread,
read) {
color: whatever !important;
}
Anyway, I vote for underlines.
Comment 47•23 years ago
|
||
IIRC Collapsed folders with unread subfolders are bold. This is a similar
pattern to collapsed threads with new messages. So, bold for both please?
Comment 48•23 years ago
|
||
What about creating two meta bugs, first being "Collapsed threads with unread
children should be bold", second being "Collapsed threads with unread children
should be underlined", state in thoes bugs that voting will be over after, say,
3 weeks, and let people vote on one of those bugs to see which option wins?
Comment 49•23 years ago
|
||
The underlines look pretty good.
Comment 50•23 years ago
|
||
The underlines may look OK when the view is All threads, but, IMO, present a
readability problem when the view is just Threads With Unread.
Comment 51•23 years ago
|
||
Could this bug depend on bug 59681? I mean, that Mozilla displays underlines
although there are no unread children?
Updated•23 years ago
|
Keywords: mozilla0.9.7
Updated•20 years ago
|
Product: Browser → Seamonkey
Comment 52•20 years ago
|
||
The CSS selector that currently works (in the thread pane, at least) is:
treechildren::-moz-tree-cell-text(container, closed, hasUnread, read)
which is slightly different from the selector presented in comment 46; I don't
know whether that one has been obsoleted. The 'selected' attribute seen in that
older one may be added to the parenthesized list if a different appearance is
desired when the node is selected.
I turned the testcase from bug 58788 (which this bug depends on) into an
attachment there, making it a little easier to verify whether it's been fixed;
the outstanding problem being mentioned had been (three years ago!) for XUL
text-decorations under Linux.
Other than that, the use of underlines has become pervasive; all the themes I've
tried in Mozilla, and the default TB theme, use them. Is making these bold
instead still an issue? (Maybe so -- there was just recently a request in the
newsgroups to use bolding, which is what led me to this bug.)
(In reply to comment #47 [Neil])
> IIRC Collapsed folders with unread subfolders are bold. This is a similar
> pattern to collapsed threads with new messages. So, bold for both please?
True about the folders, but: folders also show a count of unread items, while a
collapsed folder with nothing unread on its own simply shows bold. So, there's
still a simple way to distinguish between "visible item is/has unread" and
"only visible item's subitems are/have unread".
Updated•20 years ago
|
Assignee: sspitzer → mail
Status: ASSIGNED → NEW
Updated•16 years ago
|
Assignee: mail → nobody
Priority: P2 → --
QA Contact: laurel → message-display
Target Milestone: Future → ---
Comment 53•15 years ago
|
||
This bug report is registered in the SeaMonkey product, but has been without a comment since the inception of the SeaMonkey project. This means that it was logged against the old Mozilla suite and we cannot determine that it's still valid for the current SeaMonkey suite. Because of this, we are setting it to an UNCONFIRMED state.
If you can confirm that this report still applies to current SeaMonkey 2.x nightly builds, please set it back to the NEW state along with a comment on how you reproduced it on what Build ID, or if it's an enhancement request, why it's still worth implementing and in what way.
If you can confirm that the report doesn't apply to current SeaMonkey 2.x nightly builds, please set it to the appropriate RESOLVED state (WORKSFORME, INVALID, WONTFIX, or similar).
If no action happens within the next few months, we move this bug report to an EXPIRED state.
Query tag for this change: mass-UNCONFIRM-20090614
Status: NEW → UNCONFIRMED
Comment 54•15 years ago
|
||
This bug report is registered in the SeaMonkey product, but has been without a comment since the inception of the SeaMonkey project. This means that it was logged against the old Mozilla suite and we cannot determine that it's still valid for the current SeaMonkey suite. Because of this, we are setting it to an UNCONFIRMED state.
If you can confirm that this report still applies to current SeaMonkey 2.x nightly builds, please set it back to the NEW state along with a comment on how you reproduced it on what Build ID, or if it's an enhancement request, why it's still worth implementing and in what way.
If you can confirm that the report doesn't apply to current SeaMonkey 2.x nightly builds, please set it to the appropriate RESOLVED state (WORKSFORME, INVALID, WONTFIX, or similar).
If no action happens within the next few months, we move this bug report to an EXPIRED state.
Query tag for this change: mass-UNCONFIRM-20090614
Comment 55•15 years ago
|
||
This bug report is registered in the SeaMonkey product, but has been without a comment since the inception of the SeaMonkey project. This means that it was logged against the old Mozilla suite and we cannot determine that it's still valid for the current SeaMonkey suite. Because of this, we are setting it to an UNCONFIRMED state.
If you can confirm that this report still applies to current SeaMonkey 2.x nightly builds, please set it back to the NEW state along with a comment on how you reproduced it on what Build ID, or if it's an enhancement request, why it's still worth implementing and in what way.
If you can confirm that the report doesn't apply to current SeaMonkey 2.x nightly builds, please set it to the appropriate RESOLVED state (WORKSFORME, INVALID, WONTFIX, or similar).
If no action happens within the next few months, we move this bug report to an EXPIRED state.
Query tag for this change: mass-UNCONFIRM-20090614
Comment 56•15 years ago
|
||
This bug report is registered in the SeaMonkey product, but has been without a comment since the inception of the SeaMonkey project. This means that it was logged against the old Mozilla suite and we cannot determine that it's still valid for the current SeaMonkey suite. Because of this, we are setting it to an UNCONFIRMED state.
If you can confirm that this report still applies to current SeaMonkey 2.x nightly builds, please set it back to the NEW state along with a comment on how you reproduced it on what Build ID, or if it's an enhancement request, why it's still worth implementing and in what way.
If you can confirm that the report doesn't apply to current SeaMonkey 2.x nightly builds, please set it to the appropriate RESOLVED state (WORKSFORME, INVALID, WONTFIX, or similar).
If no action happens within the next few months, we move this bug report to an EXPIRED state.
Query tag for this change: mass-UNCONFIRM-20090614
Comment 57•15 years ago
|
||
This bug report is registered in the SeaMonkey product, but has been without a comment since the inception of the SeaMonkey project. This means that it was logged against the old Mozilla suite and we cannot determine that it's still valid for the current SeaMonkey suite. Because of this, we are setting it to an UNCONFIRMED state.
If you can confirm that this report still applies to current SeaMonkey 2.x nightly builds, please set it back to the NEW state along with a comment on how you reproduced it on what Build ID, or if it's an enhancement request, why it's still worth implementing and in what way.
If you can confirm that the report doesn't apply to current SeaMonkey 2.x nightly builds, please set it to the appropriate RESOLVED state (WORKSFORME, INVALID, WONTFIX, or similar).
If no action happens within the next few months, we move this bug report to an EXPIRED state.
Query tag for this change: mass-UNCONFIRM-20090614
Comment 58•15 years ago
|
||
This bug report is registered in the SeaMonkey product, but has been without a comment since the inception of the SeaMonkey project. This means that it was logged against the old Mozilla suite and we cannot determine that it's still valid for the current SeaMonkey suite. Because of this, we are setting it to an UNCONFIRMED state.
If you can confirm that this report still applies to current SeaMonkey 2.x nightly builds, please set it back to the NEW state along with a comment on how you reproduced it on what Build ID, or if it's an enhancement request, why it's still worth implementing and in what way.
If you can confirm that the report doesn't apply to current SeaMonkey 2.x nightly builds, please set it to the appropriate RESOLVED state (WORKSFORME, INVALID, WONTFIX, or similar).
If no action happens within the next few months, we move this bug report to an EXPIRED state.
Query tag for this change: mass-UNCONFIRM-20090614
Comment 59•15 years ago
|
||
MASS-CHANGE:
This bug report is registered in the SeaMonkey product, but still has no comment since the inception of the SeaMonkey project 5 years ago.
Because of this, we're resolving the bug as EXPIRED.
If you still can reproduce the bug on SeaMonkey 2 or otherwise think it's still valid, please REOPEN it and if it is a platform or toolkit issue, move it to the according component.
Query tag for this change: EXPIRED-20100420
Status: UNCONFIRMED → RESOLVED
Closed: 24 years ago → 15 years ago
Resolution: --- → EXPIRED
Comment 60•13 years ago
|
||
This is WFM for a long time now.
Resolution: EXPIRED → WORKSFORME
Whiteboard: se-radar → [Halloween2011Bug]
You need to log in
before you can comment on or make changes to this bug.
Description
•