Closed Bug 728309 Opened 12 years ago Closed 12 years ago

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

Categories

(Thunderbird :: Toolbars and Tabs, defect)

x86
All
defect
Not set
normal

Tracking

(thunderbird11+ fixed, thunderbird12 fixed)

VERIFIED FIXED
Thunderbird 13.0
Tracking Status
thunderbird11 + fixed
thunderbird12 --- fixed

People

(Reporter: mconley, Assigned: mconley)

References

(Blocks 1 open bug)

Details

Attachments

(9 files)

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: tb-tabsontop
Assignee: nobody → mconley
Attached patch Patch v1Splinter Review
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)
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?
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
(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.
(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! :)
Blocks: 725507
So done and marked as a dependent bug to this one.
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).
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.
(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+
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
Closed: 12 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+
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.

Attachment

General

Created:
Updated:
Size: