Closed
Bug 728309
Opened 11 years ago
Closed 11 years ago
Re-enable mail toolbar in message tab and Gloda search result list
Categories
(Thunderbird :: Toolbars and Tabs, defect)
Tracking
(thunderbird11+ fixed, thunderbird12 fixed)
VERIFIED
FIXED
Thunderbird 13.0
People
(Reporter: mconley, Assigned: mconley)
References
(Blocks 1 open bug)
Details
Attachments
(9 files)
1.02 KB,
patch
|
Bienvenu
:
review+
bwinton
:
ui-review+
standard8
:
approval-comm-aurora+
standard8
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
68.81 KB,
image/png
|
Details | |
135.07 KB,
image/png
|
Details | |
113.06 KB,
image/png
|
Details | |
118.32 KB,
image/png
|
Details | |
48.68 KB,
image/png
|
Details | |
47.52 KB,
image/png
|
Details | |
133.45 KB,
image/png
|
Details | |
154.18 KB,
image/png
|
Details |
We'd like to re-enable the below-tabs mail toolbar for the message pane and for the Gloda search result list.
Assignee | ||
Updated•11 years ago
|
Summary: Re-enable mail toolbar in Message tab → Re-enable mail toolbar in message tab and Gloda search result list
Assignee | ||
Updated•11 years ago
|
Blocks: tb-tabsontop
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → mconley
Assignee | ||
Updated•11 years ago
|
tracking-thunderbird11:
--- → ?
Assignee | ||
Comment 1•11 years ago
|
||
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)
Assignee | ||
Comment 2•11 years ago
|
||
Assignee | ||
Comment 3•11 years ago
|
||
Assignee | ||
Comment 4•11 years ago
|
||
Assignee | ||
Comment 5•11 years ago
|
||
Assignee | ||
Comment 6•11 years ago
|
||
Assignee | ||
Comment 7•11 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?
Assignee | ||
Comment 10•11 years ago
|
||
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
Assignee | ||
Comment 11•11 years ago
|
||
Assignee | ||
Comment 12•11 years ago
|
||
Comment 13•11 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•11 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.
Assignee | ||
Comment 15•11 years ago
|
||
(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! :)
![]() |
||
Comment 16•11 years ago
|
||
So done and marked as a dependent bug to this one.
Comment 17•11 years ago
|
||
Comment on attachment 598292 [details] [diff] [review] Patch v1 toolbars are present and functional, r=me
Attachment #598292 -
Flags: review?(dbienvenu) → review+
Comment 18•11 years ago
|
||
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•11 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•11 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 21•11 years ago
|
||
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 22•11 years ago
|
||
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-
Assignee | ||
Comment 23•11 years ago
|
||
(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 24•11 years ago
|
||
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+
Updated•11 years ago
|
Assignee | ||
Comment 25•11 years ago
|
||
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?
Assignee | ||
Comment 26•11 years ago
|
||
Committed to comm-central as http://hg.mozilla.org/comm-central/rev/5c3f99ded9d7
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Attachment #598292 -
Flags: approval-comm-beta?
Attachment #598292 -
Flags: approval-comm-beta+
Attachment #598292 -
Flags: approval-comm-aurora?
Attachment #598292 -
Flags: approval-comm-aurora+
Assignee | ||
Comment 27•11 years ago
|
||
Committed to comm-aurora as http://hg.mozilla.org/releases/comm-aurora/rev/a97e6cafdc03
status-thunderbird12:
--- → fixed
Assignee | ||
Comment 28•11 years ago
|
||
Committed to comm-beta as http://hg.mozilla.org/releases/comm-beta/rev/5b08357336fb
status-thunderbird11:
--- → fixed
![]() |
||
Comment 29•11 years ago
|
||
Verified fixed in 11.0 beta 3 Linux x86_64 build1, thanks Mike!
Status: RESOLVED → VERIFIED
Assignee | ||
Comment 30•11 years ago
|
||
rsx11m: No problem - thanks for verifying! -Mike
Updated•11 years ago
|
Target Milestone: --- → Thunderbird 13.0
You need to log in
before you can comment on or make changes to this bug.
Description
•