Skip to content

fix: use text widget instead of custom tab_text#2131

Closed
hojjatabdollahi wants to merge 2 commits intopop-os:masterfrom
hojjatabdollahi:hojjat/fix-headerbar
Closed

fix: use text widget instead of custom tab_text#2131
hojjatabdollahi wants to merge 2 commits intopop-os:masterfrom
hojjatabdollahi:hojjat/fix-headerbar

Conversation

@hojjatabdollahi
Copy link
Contributor

@hojjatabdollahi hojjatabdollahi commented Feb 23, 2026

Draft while waiting for pop-os/libcosmic#1140

Ellipsizes the window title in SSD and tab text for stacked windows.

image
  • I have disclosed use of any AI generated code in my commit messages.
    • If you are using an LLM, and do not fully understand the changes it is making to the code base, do not create a PR.
    • In our experience, AI generated code often results in overly complex code that lacks enough context for a proper fix or feature inclusion. This results in considerably longer code reviews. Due to this, AI authored or partially authored PRs may be closed without comment.
  • I understand these changes in full and will be able to respond to review comments.
  • My change is accurately described in the commit message.
  • My contribution is tested and working as described.
  • I have read the Developer Certificate of Origin and certify my contribution under its conditions.

@hojjatabdollahi hojjatabdollahi marked this pull request as ready for review March 2, 2026 19:13
@git-f0x
Copy link
Contributor

git-f0x commented Mar 2, 2026

Actually, specifying the font with .font(self.font) might be needed, since it's used in tabs.rs to set the font to semibold. Sorry about that.

Copy link
Contributor

@git-f0x git-f0x left a comment

Choose a reason for hiding this comment

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

Seems to match previous behavior (other than having ellipsize), with the text of the active tab becoming semibold and accent-colored, and the close button disappearing for non-active tabs when the tab is narrow enough.
A potential future improvement could be to not show any text (only icon) for non-active tabs if it would end up being only "..." (at really narrow widths, only ".." is visible), using that extra width for the active tab. But that's out of scope of this PR.

@hojjatabdollahi hojjatabdollahi mentioned this pull request Mar 10, 2026
5 tasks
@hojjatabdollahi
Copy link
Contributor Author

superseded by #2180

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