Skip to content

workspaces: make button width scale to fit names#740

Open
LukeCarrier wants to merge 2 commits into
MalpenZibo:mainfrom
LukeCarrier:workspaces-var-width
Open

workspaces: make button width scale to fit names#740
LukeCarrier wants to merge 2 commits into
MalpenZibo:mainfrom
LukeCarrier:workspaces-var-width

Conversation

@LukeCarrier
Copy link
Copy Markdown

@LukeCarrier LukeCarrier commented May 7, 2026

I bind a keyboard shortcut Super + R to set the name of each space, and I generally set the names to those of the projects I'm working on. In current ashell releases this results in overlapping text as soon as names are over 1-2 characters in length.

Make the workspace buttons width shrink to fit the workspace names, and differentiate between them with colours rather than borders.

image

Now that workspaces' buttons are of variable width due to scaling to the
fit the workspace name, we need to visually distinguish between active
and inactive workspaces through some other means. Inverting colors
should let us preserve the existing workspace color options.
Copy link
Copy Markdown
Owner

@MalpenZibo MalpenZibo left a comment

Choose a reason for hiding this comment

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

I recently added the workspace animation that works with the fixed width of the button so I need to rethink the solution to address this bug

@LukeCarrier
Copy link
Copy Markdown
Author

Would it make sense to use colour in place of item width to differentiate between active and inactive workspaces, and those which have contents and those which don't? That's what my second commit was doing here, but I wasn't really sure if you'd want changes to the theme.

No rush, I am happy to run this patch in the meantime. ashell is an awesome piece of software and is a major upgrade over my janky Waybar configuration -- thank you for maintaining it!

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