Bug 582801 introduced a real toolbar on the tab-bar and marked the tabmail-buttons as deprecated. We should investigate what we are doing with the two buttons in this box (calendar-tab-button and task-tab-button). Possible solutions are: 1. Move the two buttons from #tabmail-buttons to #tabbar-toolbar and let them not removable as they are now in this toolbar. 2. As 1. but make the buttons removable. 3. Remove this two buttons and use the two already existing toolbar-buttons #lightning-button-calendar and #lightning-button-tasks instead. Solution 1 is the easiest and for the user this would be no markable change. Solution 2 is flexible but we could irritate the user with two sets of movable buttons which are doing the same. Solution 3 can work for new users but what is with old users which have customized their toolbars with this buttons? Do you have other solutions?
I added you to CC for a first feedback. What do you mean?
I think option 1 makes most sense. Its too bad the tabmail-buttons are going away. I liked the fact that the calendar icons were large and visible. How large are the #tabbar-toolbar icons?
There is no difference in size. The #tabbar-toolbar is fixed to small size because big size makes no sense in the tab-bar.
I'd remove the obsolete tabmail-buttons and add two new buttons to the new tab toolbar. Buttons should be displayed by default after installation. But the user should be allowed to remove them.
This is solution 1, move the buttons from tabmail-buttons to tabbar-toolbar. The only difference would be the two buttons are now on the right side of the Quick Filter button instead of at the left side. I could add a insertbefore="QFB-button" but what happens when this button is removed or in a other toolbar? And the TB devs are thinking about removing this button from this bar (Bug 717264).
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #590456 - Flags: review?(philipp)
Comment on attachment 590456 [details] [diff] [review] Move the buttons to tabbar-toolbar Review of attachment 590456 [details] [diff] [review]: ----------------------------------------------------------------- If you add in insertbefore="invalidbuttonId" then it will just ignore it and append to the end as usual, so go ahead and do that. What about the icons, can we use the old icons? Also, what about new users? Are the buttons automatically in the default set, or do they need to be added on first run? Given the above are non-issues, r=philipp
Attachment #590456 - Flags: review?(philipp) → review+
(In reply to Philipp Kewisch [:Fallen] from comment #6) > If you add in insertbefore="invalidbuttonId" then it will just ignore it and > append to the end as usual, so go ahead and do that. Okay, then I'll do it. > What about the icons, can we use the old icons? Also, what about new users? > Are the buttons automatically in the default set, or do they need to be > added on first run? It is only a move the icons are still the same and also for new users they are immediately shown.
It looks Calendar doesn't have a approval-comm-aurora, so I'm asking here: Is it okay to land this patch on aurora? The Tabs-on-top patch is also on aurora.
Comment on attachment 590456 [details] [diff] [review] Move the buttons to tabbar-toolbar a=philipp for aurora
Pushed to comm-central: <http://hg.mozilla.org/comm-central/rev/e828b319282e> and comm-aurora: <http://hg.mozilla.org/releases/comm-aurora/rev/da70672b466a> With insertbefore= it didn't worked but with defaultset= for the toolbar it worked to position the buttons. So I used this.
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.3
Hmm not sure modifying the default set in an overlay is the right thing to do. If another extension adds something to the default set, then this overlay might override it. I could be wrong though.
You need to log in before you can comment on or make changes to this bug.