Closed Bug 263255 Opened 16 years ago Closed 16 years ago

Improve thread pane UI for message grouping

Categories

(Thunderbird :: Mail Window Front End, defect)

x86
Windows XP
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird0.9

People

(Reporter: mscott, Assigned: mscott)

Details

(Keywords: fixed-aviary1.0)

Attachments

(6 files, 1 obsolete file)

Tracking bug for some CSS improvements I hope to make to build off of David's
great grouping feature.
Status: NEW → ASSIGNED
Target Milestone: --- → Thunderbird0.9
Attached image some possible ideas
Here are a couple of different looks...
Attached image another idea
Attachment #161385 - Flags: superreview?(bienvenu)
Comment on attachment 161385 [details] [diff] [review]
patch for some UI improvements for grouping

I'm a little torn about removing the thread column, since it could be used to
switch to normal threaded mode, if it worked, but it does look cleaner without
it...
Attachment #161385 - Flags: superreview?(bienvenu) → superreview+
Other things I'd like to try to address here:

1) Adding a customizeable toolbar button for popping you in and out of grouped
mode, making this feature more visible.

2) I need to do some mac osx theme work to make this work for Mac

3) For messages from yesterday, maybe just showing the time instead of a date
stamp on the msg.

4) double clicking a dummy group row causes us to open the next msg in a stand
alone window....
Kevin will probably want to style this in a nice OS X friendly fashion but this
gets us going.
Comment on attachment 162136 [details] [diff] [review]
some CSS to get grouping going on Mac OS X

wrong patch
Attachment #162136 - Attachment is obsolete: true
the OS X style rules have been checked into the branch and the trunk but they
could use some more OS X specific polish.
Comment on attachment 162668 [details] [diff] [review]
patch to get rid of column re-ordering, make all attributes group with dummy rows except for subject

David, what do you think of this patch?

1) No more column re-ordering

2) as we discussed all sort attributes except for the subject now use dummy
rows

3) Some of our sort attributes like priority, lable and status have states with
no title text so I had to make up some entries such as "No Status" or "No
Priority" to show up in the UI for that particular container.

4) I'd like to get rid of the code that shows/hides the thread icon next but am
still mulling that over. 

I'm sure I broke something in here when toggling back and forth between group
by sorted, threaded mode, etc. with my threadpane.js changes :)
Attachment #162668 - Flags: superreview?(bienvenu)
Comment on attachment 162668 [details] [diff] [review]
patch to get rid of column re-ordering, make all attributes group with dummy rows except for subject

looks good, thx, Scott!
Attachment #162668 - Flags: superreview?(bienvenu) → superreview+
screen shot to show off how grouping now looks when you are sorted by
attributes like Label and Date.

Subject is now the only one that won't use these dummy rows.
I think we've gotten enough mileage out of this bug. Further improvements can be
handled in separate bugs.

Closing this out. All of the patches are in the branch and trunk. 
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Keywords: fixed-aviary1.0
Minor nit: The "mark as deleted" icon (envelope with red X) doesn't appear in
grouped view.
Hmm that worked for me Peter using todays build. I went into grouped mode by
date. Deleted a message and saw the red X next to the icon. Are you using the
default theme? I"m sure most themes haven't been updated to work right with
groups yet. 
(In reply to comment #16)
version 0.8 (20041021), winXP, default theme (plus "purge" button extension <-
should be in default theme, BTW), IMAP, setting: mark as deleted, sort by date,
send e-mail to self, select message, press g, re-select message, click delete
button -> message's envelope icon is *not* marked with red X.
(In reply to comment #17)
> version 0.8 (20041021), winXP, default theme (plus "purge" button extension <-
> should be in default theme, BTW), IMAP, setting: mark as deleted, sort by date,
> send e-mail to self, select message, press g, re-select message, click delete
> button -> message's envelope icon is *not* marked with red X.

I repeated your steps with version 1.0 (20041206) and all behaves well. Peter
can you test again with a recent version? Perhaps this could be fixed by the
theme update before 1.0.

(In reply to comment #18)
> Peter can you test again with a recent version? 

OK, it works now - Thunderbird "version 1.0 (20041212)"

BTW: It would be nice if the selected message *stayed selected* when toggling
between grouped and listed views. (I never use grouped view)
You need to log in before you can comment on or make changes to this bug.