Menu bar has too much space left on it's right side

VERIFIED FIXED in Thunderbird 25.0

Status

Thunderbird
Theme
VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: MikkCZ, Assigned: Paenglab)

Tracking

17 Branch
Thunderbird 25.0
x86_64
Windows 7

Thunderbird Tracking Flags

(thunderbird24+ fixed)

Details

Attachments

(3 attachments)

Created attachment 764124 [details]
screenshot

There is a plenty of space left on the right side of the main toolbar in TB. Look at the screenshot. When tabs are not drawn in titlebar (mail.tabs.drawInTitlebar = false), it's OK. 

This code (using Stylish) fixes that in Aero with Glass effect (=>tabs above the main toolbar):
#navigation-toolbox {
  margin-right: -110px;
}
#messengerWindow[sizemode="maximized"] #navigation-toolbox #tabs-toolbar {
  margin-right: 110px !important;
}

For Aero without Glass effect works (=>tabs under the main toolbar):
#messengerWindow #navigation-toolbox #tabs-toolbar {
  margin-right: -110px;
}

For Classic (WinXP-like) or High contrast (=>tabs under the main toolbar):
#messengerWindow #navigation-toolbox #tabs-toolbar {
  margin-right: -60px;
}
(Assignee)

Comment 1

4 years ago
Is this the main toolbar or do you mean the menubar used as main toolbar? The main toolbar shouldn't have this space. If it's the menubar, I can add styles to let the space only on the topmost toolbar.
The toolbar with the menu (File, ...).

IMHO it seems to be more UI than theme issue. The menu/tabbar (which is below) shouldn't be right margined by the minimize/close buttons.

For WinXP it's the same.

Updated

4 years ago
Summary: Main toolbar has too much space left on it's right side → Menu bar has too much space left on it's right side

Comment 3

4 years ago
Yes, I see this on Win XP too. I think linux too, but will have to check.

It does make sense when the Menu Bar is not shown as the buttons fit just before the main window buttons (minimize, close). But when the Menu Bar is shown, it is not necessary to reserve the space.
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Assignee)

Comment 4

4 years ago
Created attachment 764817 [details] [diff] [review]
patch

This patch removes the space on all Windows systems except Aero from tabbar when the menubar above is always shown. When the menubar is autohiding then the space is left to not let the items on the tabbar jump.

On Aero, where the menubar is below the tabbar, the menubar has it's spacer always removed.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #764817 - Flags: ui-review?(bwinton)
Attachment #764817 - Flags: review?(bwinton)
Can I ask, in which version will be the patch applied?
(Assignee)

Comment 6

4 years ago
After review I can ask for approval for TB 17 ESR. Depending the release driver's decision it can land there or only in a later version.
(In reply to Richard Marti [:Paenglab] from comment #6)
> After review I can ask for approval for TB 17 ESR. Depending the release
> driver's decision it can land there or only in a later version.

Perhaps not much point, with TB24 coming out in a couple months.
Comment on attachment 764817 [details] [diff] [review]
patch

In classic/high-contrast mode, I think we could do with a little less space:
https://dl.dropboxusercontent.com/u/2301433/Screenshots/MenuSpace.png
but ui-r=me with that tightened up a little.  (If it's reasonably easy to do.  If it's hard, then skip it.)

And the code seems okay, so r=me, too.
Attachment #764817 - Flags: ui-review?(bwinton)
Attachment #764817 - Flags: ui-review+
Attachment #764817 - Flags: review?(bwinton)
Attachment #764817 - Flags: review+
(Assignee)

Comment 9

4 years ago
Created attachment 777161 [details]
pacing after TB restart

(In reply to Blake Winton (:bwinton) from comment #8)
> Comment on attachment 764817 [details] [diff] [review]
> patch
> 
> In classic/high-contrast mode, I think we could do with a little less space:
> https://dl.dropboxusercontent.com/u/2301433/Screenshots/MenuSpace.png
> but ui-r=me with that tightened up a little.  (If it's reasonably easy to
> do.  If it's hard, then skip it.)

You switched directly from Basic/Aero theme to classic/high-contrast? Then I see the same. After TB restart the spacing is like in my screenshot.

TB calculates only at startup the (placeholder)spacer width of the caption buttons. When you started TB with Basic/Aero then the buttons are wider and by changing the theme the spacer stays at this width also with the smaller classic buttons. Restarting TB let it recalculate the correct width.
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/17f6b3acfc77
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 25.0
So, will be this patch applied in next TB release (24), or only from 25?
(Assignee)

Comment 12

4 years ago
Thanks to ask and remind me to set the approval request for TB 24.
(Assignee)

Comment 13

4 years ago
Comment on attachment 764817 [details] [diff] [review]
patch

[Approval Request Comment]
This would help save space for buttons etc. also for ESR-24 users.
This is only a CSS patch so it should be really safe to not regress in a bad way.
Attachment #764817 - Flags: approval-comm-aurora?
Attachment #764817 - Flags: approval-comm-aurora? → approval-comm-aurora+
https://hg.mozilla.org/releases/comm-aurora/rev/512db0e46549
status-thunderbird24: --- → fixed
tracking-thunderbird24: --- → +

Comment 15

4 years ago
Works very nice on Win XP, TB25.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.