Closed
Bug 718173
Opened 13 years ago
Closed 11 years ago
Move the #toolbar-buttons to the right of the #tabbar-toolbar
Categories
(Thunderbird :: Toolbars and Tabs, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: Paenglab, Assigned: Paenglab)
Details
Attachments
(1 file)
1.09 KB,
patch
|
bwinton
:
ui-review-
|
Details | Diff | Splinter Review |
With Lightning or an other Add-on installed with buttons in #toolbar-buttons could make the behavior of customization a bit strange. When customizing you can put the item on the right of the tabs (here specially between tabs and Lightning buttons). But then the items jumps on the right of the Lightning buttons. It is also possible to let fall the items directly over the other buttons of the #tabbar-toolbar. Then the items stay where they are put. With Bug 717264 it could be no item is in this toolbar initially then it isn't possible to put the items directly in this toolbar and the behavior is more obvious.
Assignee | ||
Comment 1•13 years ago
|
||
This patch moves the #toolbar-buttons to the right of the #tabbar-toolbar. This makes now the QFB button isn't directly beside the alltabs-button with a Add-on installed on the default configuration. Blake, what do you mean is this okay or would you leave the toolbars as they are?
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #588618 -
Flags: ui-review?(bwinton)
Attachment #588618 -
Flags: review?(bwinton)
Assignee | ||
Updated•13 years ago
|
Attachment #588618 -
Attachment is patch: true
Comment 2•12 years ago
|
||
Comment on attachment 588618 [details] [diff] [review] Patch So, as I understand it, Lightning is going to switch back to toolbar buttons, so shouldn't have this issue anymore. (Cc-ing Fallen for confirmation.) I think what I'ld really like to see is for us just to remove tabmail-buttons, since anything that went there can now go in tabbar-toolbar instead, and be more customizable, too! Given that, I think I'm going to ui-r- this patch, and clear the review request. (Cc-ing Sancus to see how many extensions currently use the "tabmail-buttons" element, and how easy it would be to add this to the compatibility bump check.) Thanks, Blake.
Attachment #588618 -
Flags: ui-review?(bwinton)
Attachment #588618 -
Flags: ui-review-
Attachment #588618 -
Flags: review?(bwinton)
Comment 3•12 years ago
|
||
(In reply to Blake Winton (:bwinton - Thunderbird UX) from comment #2) > I think what I'ld really like to see is for us just to remove > tabmail-buttons, since anything that went there can now go in tabbar-toolbar > instead, and be more customizable, too! I'd recommend keeping tabmail-buttons around for a little while for backwards compatibility (or checking to make sure no popular add-ons are using it).
Comment 4•12 years ago
|
||
I'm hoping Sancus could run that check for us. If not, then I agree we probably want to keep it around, and then putting it to the right seems reasonable.
Assignee | ||
Comment 5•12 years ago
|
||
With Bug 719006 the Lightning buttons are now in the tabbar-toolbar.
Assignee | ||
Updated•11 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•