The default bug view has changed. See this FAQ.

Re-enable mail toolbar in message tab and Gloda search result list

VERIFIED FIXED in Thunderbird 13.0

Status

Thunderbird
Toolbars and Tabs
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: mconley, Assigned: mconley)

Tracking

(Blocks: 1 bug)

Trunk
Thunderbird 13.0
x86
All
Dependency tree / graph

Thunderbird Tracking Flags

(thunderbird11+ fixed, thunderbird12 fixed)

Details

Attachments

(9 attachments)

We'd like to re-enable the below-tabs mail toolbar for the message pane and for the Gloda search result list.
Summary: Re-enable mail toolbar in Message tab → Re-enable mail toolbar in message tab and Gloda search result list
Blocks: 644169
Assignee: nobody → mconley
tracking-thunderbird11: --- → ?
Created attachment 598292 [details] [diff] [review]
Patch v1

This seems pretty simple - but did I forget any cases?

Screenshots of how this looks with our current styling forthcoming...
Attachment #598292 - Flags: ui-review?(bwinton)
Attachment #598292 - Flags: review?(dbienvenu)
Created attachment 598293 [details]
Linux - mail toolbar in message tab
Created attachment 598294 [details]
Linux - mail toolbar in search result tab
Created attachment 598295 [details]
Windows 7 - toolbar in message tab
Created attachment 598296 [details]
Windows 7 - toolbar in search tab
Created attachment 598300 [details]
Windows XP - toolbar in message tab
Created attachment 598302 [details]
Windows XP - toolbar in search result tab

Comment 8

5 years ago
Hi Mike, looks good but how does this relate to bug 725507?
This looks like my proposal (3) over there...

Is this a short-term/interim solution for 11.0 only?
Duplicate of this bug: 725507
Hey rsx11m,

Thanks for pointing that one out! Resolving as duplicate.  I'll do development on this bug #.

> Is this a short-term/interim solution for 11.0 only?

Unsure - bwinton might know?

-Mike
Created attachment 598310 [details]
OSX- toolbar in search result tab
Created attachment 598311 [details]
OSX- toolbar in message tab

Comment 13

5 years ago
(In reply to Mike Conley (:mconley) from comment #10)
> > Is this a short-term/interim solution for 11.0 only?
> 
> Unsure - bwinton might know?

I sure hope so.

Comment 14

5 years ago
(In reply to Mike Conley (:mconley) from comment #10)
> Resolving as duplicate.  I'll do development on this bug #.

Usually it's a good idea to look for existing bugs before filing a new one, and I thought that this specific one was high-profile enough not to be missed... ;-)

> > Is this a short-term/interim solution for 11.0 only?
> Unsure - bwinton might know?

If not, we can keep bug 725507 to further investigate into how to improve the message-tab and search-tabs toolbars from here.
(In reply to rsx11m from comment #14)
> Usually it's a good idea to look for existing bugs before filing a new one,
> and I thought that this specific one was high-profile enough not to be
> missed... ;-)
> 

Whoops - I had looked around, but I guess I didn't notice 725507.  If you'd like to re-open and use that for investigating UI improvements in those areas, be my guest! :)

Updated

5 years ago
Blocks: 725507

Comment 16

5 years ago
So done and marked as a dependent bug to this one.

Comment 17

5 years ago
Comment on attachment 598292 [details] [diff] [review]
Patch v1

toolbars are present and functional, r=me
Attachment #598292 - Flags: review?(dbienvenu) → review+
Things to consider:

1) Bug 725507 Comment 6 by rsx11m:
> FWIW, the stand-alone message viewer (Open Message in New Window) has a full
> main-like toolbar as well, which can be customized separately from the
> 3-pane window toolbar. So, this toolbar could be "recycled" for the message
> tab as a variant of option (3).

So currently, there are at least two independently customizable "mail" toolbars, one in the main 3pane, and another one for message windows. Now we have tabs on top, so in a message tab, the "mail" toolbar there is no longer an application-wide toolbar, but a message-specific toolbar (ux-natural-mapping). So I'm pretty sure it should *not* be the same as the main mail toolbar. At least, we should use the same separately customizable toolbar that we have in the stand-alone msg win, so that we have a msg-related toolbar (vs. a general purpose toolbar for the main 3pane). Which of the two toolbars did you use in your patch?

2) Consider giving the "open as list"-style global search results tab it's own (3rd) toolbar again. User might want to see things there that are not needed for a single message. And we could share that 3rd multiple-messages-related toolbar with "open in conversation" which is pretty much the same as "open as list".

3) Ensure that the "Quick filter" button is disabled or even hidden where it cannot be applied, e.g. in msg tab. Again, a good argument for giving the single msg tab it's own toolbar (shared with single msg windows), and having a separate toolbar for search tab & open in conversation, both with multiple messages where filtering makes sense.

With tabs on top, it doesn't make sense to put the very same main mail toolbar everywhere. Tabs on top is a paradigm shift and whatever toolbar is inside the tab should relate to that type of tab.

Another question: Having a set of independently configurable toolbars is useful, but having separate sets of selected buttons which I can only put on one toolbar but not on another seems very restrictive. Is it technically possible to arrange the customization dialogue in such a way that the user can put *any* button on *any* toolbar? That's how applications like Word handle toolbar customization, and it's a lot more flexible than we are (and would solve a lot of problems where certain buttons are missing in the palettes for that very toolbar).

Comment 19

5 years ago
Thomas: AFAICT, this patch reintroduces the main toolbar ("mail-bar3") for the message and search tabs, thus the same toolbar as before. Apparently that box is just hidden in the current implementation, and the patch simply removes hiding it. This should be the simplest version for a mitigating fix with target 11.0, and not much more may be possible considering that it's in the beta phase already and any changes are restricted. Also, there is still bug 725507 for more comprehensive modifications on current trunk following damage control for 11.0 here.

Comment 20

5 years ago
(In reply to Thomas D. from comment #18)
> Things to consider:
> 
> 1) Bug 725507 Comment 6 by rsx11m:
> > FWIW, the stand-alone message viewer (Open Message in New Window) has a full
> > main-like toolbar as well, which can be customized separately from the
> > 3-pane window toolbar. So, this toolbar could be "recycled" for the message
> > tab as a variant of option (3).
> 
> So currently, there are at least two independently customizable "mail"
> toolbars, one in the main 3pane, and another one for message windows.

Not really. It's the same toolbar, but in a different window, so it's persisted in a different spot. If you put that toolbar in the main window for a message tab, it would automatically become the same toolbar as in the 3pane.
Comment on attachment 598292 [details] [diff] [review]
Patch v1

Seems like a reasonable thing to do until we get a different toolbar for those screens…  ;)

ui-r=me!
Attachment #598292 - Flags: ui-review?(bwinton) → ui-review+
Comment on attachment 598292 [details] [diff] [review]
Patch v1

D'oh!  I actually take this ui-r+ back, because I just noticed that the QFB button doesn't work in the View-As-List mode, and I really think it should…
Attachment #598292 - Flags: ui-review+ → ui-review-
(In reply to Blake Winton (:bwinton - Thunderbird UX) from comment #22)
> Comment on attachment 598292 [details] [diff] [review]
> Patch v1
> 
> D'oh!  I actually take this ui-r+ back, because I just noticed that the QFB
> button doesn't work in the View-As-List mode, and I really think it should…

Hm - this bug appears to exist regardless of my change.  I think it's a separate issue.  Would you mind greatly if I opened up a separate bug for it?

-Mike
Comment on attachment 598292 [details] [diff] [review]
Patch v1

(In reply to Mike Conley (:mconley) from comment #23)
> (In reply to Blake Winton (:bwinton - Thunderbird UX) from comment #22)
> > Comment on attachment 598292 [details] [diff] [review]
> > Patch v1
> > 
> > D'oh!  I actually take this ui-r+ back, because I just noticed that the QFB
> > button doesn't work in the View-As-List mode, and I really think it should…
> 
> Hm - this bug appears to exist regardless of my change.  I think it's a
> separate issue.  Would you mind greatly if I opened up a separate bug for it?

Okay, but I really do think we should fix it before we land.

Thanks,
Blake.
Attachment #598292 - Flags: ui-review- → ui-review+
tracking-thunderbird11: ? → +
Depends on: 729197
Comment on attachment 598292 [details] [diff] [review]
Patch v1

We want this for TB 11.
Attachment #598292 - Flags: approval-comm-beta?
Attachment #598292 - Flags: approval-comm-aurora?
Committed to comm-central as http://hg.mozilla.org/comm-central/rev/5c3f99ded9d7
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Attachment #598292 - Flags: approval-comm-beta?
Attachment #598292 - Flags: approval-comm-beta+
Attachment #598292 - Flags: approval-comm-aurora?
Attachment #598292 - Flags: approval-comm-aurora+
Committed to comm-aurora as http://hg.mozilla.org/releases/comm-aurora/rev/a97e6cafdc03
status-thunderbird12: --- → fixed
Committed to comm-beta as http://hg.mozilla.org/releases/comm-beta/rev/5b08357336fb
status-thunderbird11: --- → fixed

Comment 29

5 years ago
Verified fixed in 11.0 beta 3 Linux x86_64 build1, thanks Mike!
Status: RESOLVED → VERIFIED
rsx11m:

No problem - thanks for verifying!

-Mike
Target Milestone: --- → Thunderbird 13.0
You need to log in before you can comment on or make changes to this bug.