Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ui: Fix keybind sizing for non-default UI font sizes #24708

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MrSubidubi
Copy link
Contributor

@MrSubidubi MrSubidubi commented Feb 12, 2025

Closes #24597

This fixes the regression from 00971fb which removed the text_ui(cx) - call from the keybinding render. The removal caused improperly scaled font rendering as shown in the images below.

This PR reintroduces this behaviour for all cases where size is not set.

main With this patch Parent of 00971fb
Small font size (10px) cur_small pr_small prev_small
Large font size cur_big pr_big prev_big

Release Notes:

  • Fixed keybind hints being improperly scaled for custom ui font sizes

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Feb 12, 2025
@maxdeviant maxdeviant changed the title keybinds: Fix sizing for non-default ui font sizes keybinds: Fix sizing for non-default UI font sizes Feb 12, 2025
@maxdeviant maxdeviant changed the title keybinds: Fix sizing for non-default UI font sizes ui: Fix keybind sizing for non-default UI font sizes Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Hotkey Text Font Size Too Large (0.173.5-pre regression)
1 participant