Skip to content

gh-14168: Fix collapsed workspace toggle hitbox - #14795

Open
bnquon wants to merge 8 commits into
zen-browser:devfrom
bnquon:fix/collapsed-workspace-actions
Open

gh-14168: Fix collapsed workspace toggle hitbox#14795
bnquon wants to merge 8 commits into
zen-browser:devfrom
bnquon:fix/collapsed-workspace-actions

Conversation

@bnquon

@bnquon bnquon commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

follows the suggested approach from the original issue by preventing the workspace actions button from appearing on hover in the collapsed sidebar. this keeps the space icon visible and clickable for collapsing pinned tabs, while the actions menu remains right-clickable.

demo showing sidebar toggling and the style fixes for the hover

zen-issue-14168-demo.mp4

closes #14168

@bnquon
bnquon requested a review from mr-cheffy as a code owner July 31, 2026 03:33
@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. Improvement labels Jul 31, 2026
@bnquon

bnquon commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

while testing these changes, i noticed that the workspace action in a collapsed unsynced/blank window has no visible icon or label in the current release. the tooltip appears and the action still works when clicked, but the button itself looks blank, as shown while hovering in the attached image.

this is unrelated to the changes in this pr, so i preserved the existing behavior. should this be addressed separately, or would you prefer that i include some suggested change in this PR?

from the current release, red arrow added to show where my cursor is hovering during SS, on top of the blank window workspace collapsed icon
image

@mr-cheffy

Copy link
Copy Markdown
Member

Sure, you can address it here

@dosubot dosubot Bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Jul 31, 2026
@bnquon

bnquon commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

Sure, you can address it here

added a dedicated icon class for the move to space action in blank windows. contemplated between this icon and the open icon, but went with the corner-down-right arrow since it’s less commonly used elsewhere and felt clearer for moving something. the tooltip and expanded sidebar behavior still work too

zen-issue-14168-followup-demo.mov

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Jul 31, 2026

This comment was marked as resolved.

@mr-cheffy

Copy link
Copy Markdown
Member

Only concern I have so far is that maybe people are used to simply click on the three dots for the workspace actions, but thats not possible anymore...

@bnquon

bnquon commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

yeah fair, although i do think this is cleaner than adding another button or finding a separate place for the collapsed action, and it better matches the click behavior of the expanded sidebar

@mr-cheffy

Copy link
Copy Markdown
Member

What about making it float to the top left? Like the close buttons on tabs

@bnquon

bnquon commented Aug 1, 2026

Copy link
Copy Markdown
Contributor Author

What about making it float to the top left? Like the close buttons on tabs

i was able to get it looking like this by adding the same Firefox css from the collapsed tab close button. i checked the left alignment using Shottr’s (screenshot app) ruler, and it looks to have the same gap as the buttons on tabs. the only difference is that i used 1px less padding to make the three dots look a little bigger and clearer. this only applies to normal workspaces, the blank windows keep the move to space arrow from the earlier change.

if this looks fine and you want to go with this lmk and i can commit my local changes

(hovered workspace icon in sidebar view)
image

@mr-cheffy

Copy link
Copy Markdown
Member

That looks great!

@dosubot dosubot Bot added size:M This PR changes 30-99 lines, ignoring generated files. and removed size:S This PR changes 10-29 lines, ignoring generated files. labels Aug 1, 2026
@bnquon

bnquon commented Aug 1, 2026

Copy link
Copy Markdown
Contributor Author

okay sounds good! committed those changes

Comment thread src/zen/tabs/zen-tabs/vertical-tabs.css
Comment thread src/zen/tabs/zen-tabs/vertical-tabs.css Outdated
Comment thread src/zen/tabs/zen-tabs/vertical-tabs.css
@dosubot dosubot Bot removed the lgtm This PR has been approved by a maintainer label Aug 1, 2026
@bnquon
bnquon requested a review from mr-cheffy August 4, 2026 07:33
--close-button-extra-background: transparent;
width: 15px;
height: 15px;
box-sizing: border-box;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What's up with these weird attributes being added? Seems really vibecoded. color-scheme as well, etc...

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.

these are taken from Firefox’s base tab close button styles and collapsed vertical tab close button styles for consistency

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Improvement size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Small hitbox for hiding space

3 participants