Skip to content

fixed an issue where “100%” was displayed as “10…”#40

Merged
vaxerski merged 3 commits intohyprwm:mainfrom
FelipeFMA:main
Feb 6, 2026
Merged

fixed an issue where “100%” was displayed as “10…”#40
vaxerski merged 3 commits intohyprwm:mainfrom
FelipeFMA:main

Conversation

@FelipeFMA
Copy link
Contributor

image

Copy link
Member

@vaxerski vaxerski left a comment

Choose a reason for hiding this comment

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

a better way could be to precalc the size of 100%. with a wider font still it could grow even more

@FelipeFMA
Copy link
Contributor Author

I will try to work on that tomorrow

@FelipeFMA
Copy link
Contributor Author

@vaxerski I tried to precalc the size of "100%" but preferredSize() is private and size() returns 0 until the element is in the widget tree. Is there a way to measure text size before layout that I'm missing, or would this need an API addition to hyprtoolkit?

@vaxerski
Copy link
Member

vaxerski commented Feb 6, 2026

yes, check hyprland-guiutils specifically the dialog, it does layout size precalc

@FelipeFMA
Copy link
Contributor Author

@vaxerski What do you think?

@vaxerski
Copy link
Member

vaxerski commented Feb 6, 2026

I think hamburgers are tasty

oh you mean the code, I'll review later today

@FelipeFMA
Copy link
Contributor Author

Pizza is better

Copy link
Member

@vaxerski vaxerski left a comment

Choose a reason for hiding this comment

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

yup lgtm, thanks!

@vaxerski vaxerski merged commit 05c8b4b into hyprwm:main Feb 6, 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