Skip to content

Fix window controls when horizontal tab bar is hidden - #1187

Merged
black7375 merged 1 commit into
black7375:masterfrom
hdk5:fix-menubar-window-control
Oct 5, 2026
Merged

black7375 merged 1 commit into
black7375:masterfrom
hdk5:fix-menubar-window-control

Conversation

@hdk5

@hdk5 hdk5 commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Describe the PR

Firefox now provides window controls and drag spacers in the navigation toolbar for configurations where the horizontal tab strip is hidden. Lepton previously continued using the menubar/tabbar control layout and reserving a fixed window-control gap, which caused:

  • Empty space in the navigation toolbar
  • Missing controls when the menubar was hidden

PR Type

  • Add: Add feature or enhanced.
  • Fix: Bug fix or change default values.
  • Clean: Refactoring.
  • Doc: Update docs.

Related Issue
#1145

Comment on lines +11 to +12
@include WindowControl_Right {
:root[sizemode="normal"]

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems like using a different option might cause problems due to the size mode.
I will review it over the weekend, make corrections, and merge it.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was intentional, to match the behavior when the pre-spacer also appears only when the browser window is not maximized - which, if I remember correctly, was Lepton's behavior a several versions back.
Admittedly, I tested it only on my Windows configuration with right-side window controls, as left-sided variations are out of scope of my personal interest.

@hdk5
hdk5 force-pushed the fix-menubar-window-control branch from fae8394 to fd557fe Compare September 15, 2026 15:09
@hdk5
hdk5 force-pushed the fix-menubar-window-control branch from fd557fe to 45cef89 Compare September 15, 2026 15:20
@black7375

Copy link
Copy Markdown
Owner

A bit late, but thank you for the contribution!

@black7375
black7375 merged commit 77e742e into black7375:master Oct 5, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants