Move the #toolbar-buttons to the right of the #tabbar-toolbar

RESOLVED WONTFIX

Status

Thunderbird
Toolbars and Tabs
RESOLVED WONTFIX
6 years ago
5 years ago

People

(Reporter: Paenglab, Assigned: Paenglab)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
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

6 years ago
Created attachment 588618 [details] [diff] [review]
Patch

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

6 years ago
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)

Comment 3

6 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).
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

6 years ago
With Bug 719006 the Lightning buttons are now in the tabbar-toolbar.
(Assignee)

Updated

5 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.