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)

x86_64
All
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: Paenglab, Assigned: Paenglab)

Details

Attachments

(1 file)

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.
Attached patch PatchSplinter Review
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)
Attachment #588618 - Attachment is patch: true
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)
(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).
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.
With Bug 719006 the Lightning buttons are now in the tabbar-toolbar.
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.

Attachment

General

Created:
Updated:
Size: