Skip to content

UI scaling fixes for 516 (+ command bar typing indicators) #6280

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

Merged
merged 27 commits into from
Apr 13, 2025

Conversation

Absolucy
Copy link
Member

@Absolucy Absolucy commented Apr 9, 2025

About The Pull Request

Ports the following PRs:

Changelog

🆑
add: (MTO, Lilah Novi) Say commands typed in the command bar now trigger typing indicators.
qol: (hry-gh) Under the chat settings, you can now press "Adjust UI Sizes" to drag around some of the panel sizes.
fix: (hry-gh) The input at the bottom right is no longer so teeny tiny on 516 if you use a window scaling that is greater than 100%
qol: (hry-gh) There's a new UI preference called UI scale which allows people that use windows scaling to have their UIs original size with the contents zoomed out, instead of the default, which is the UIs being larger with the contents "normal" size.
fix: (hry-gh) Various UIs did not respect windows scaling, they now do.
fix: Fixed the white border around the tgui-say input on 516 when it was focused.
/:cl:

san7890 and others added 11 commits April 9, 2025 16:33
This was another boolean that was used to just manage stuff codeside
that really was not accessed _too_ much and is ultimately not useful as
a variable on `/mob`. This just moves it to a trait because it's only
really used in a few spots for a similar intent+purpose.

Less stuff to deal with in the average view variables window whenever
looking at a mob, which is really nice and welcome.
Doesn't concern players.
## About The Pull Request
It was very specifically broken by [this
commit](tgstation/tgstation@94d00aa),
because it wrongfully changed the logic. Now the typing indicator is
back, it's no longer just the thinking indicator.

## Why It's Good For The Game
Typing indicator is meant to be there, now it's back! No more awkward
flickering!

## Changelog

:cl: GoldenAlpharex
fix: The typing indicator has overcome its shyness and is now back to
its usual form.
/:cl:
changes a whole load of crap in the skin to make it a bit more reactive
to sizing changes, and thus to high dpi monitors

this required adding a bunch of splitters - these have a sane preset for
most monitors but if you ever need to edit it, it's hidden under the
chat settings

1920x1080, 100% scaling

![image](https://github.com/user-attachments/assets/d2b54186-2163-4a77-8580-59008bb7ccc6)

4k, 200% scaling
before:

![vraQH3PvmD8nAGcZ@2x](https://github.com/user-attachments/assets/7f66cfe4-a656-447e-a9b3-d8305638f13d)

after:

![1sc22SAu7x8AOU7D@2x](https://github.com/user-attachments/assets/c2c6e90f-e884-41b3-b80a-191d2aee1f8f)

the sliders:

[user-attachments/assets/776730a7-a325-4b3b-8e27-15636828c6d4](https://github.com/user-attachments/assets/776730a7-a325-4b3b-8e27-15636828c6d4)

it makes the game a bit more playable on my laptop

:cl:
qol: under the chat settings, you can now press "Adjust UI Sizes" to
drag around some of the panel sizes
fix: the input at the bottom right is no longer so teeny tiny on 516 if
you use a window scaling that is greater than 100%
/:cl:

---------

Co-authored-by: harryob <[email protected]>
…erence for smaller windows (#90418)

various uis (tgui-say, vv, player panel, anything using
`/datum/browser`) would not correct for dpi. they now do

we also have a preference to allow you to have smaller windows that are
zoomed out, instead of larger windows that are not zoomed

viewport units, like `vh` and `vw`. they need to be fed through the
`vp(100vw)` function first. there was only one such unit in the codebase

on 4k monitor with 200% scaling:

fixed (pref on default)

![BiygmRan0QqxOMqL@2x](https://github.com/user-attachments/assets/1bdbc93c-1cd4-4a17-bf18-6cca7a5df032)

pref disabled

![DUWSVJNXt3xuQJEy@2x](https://github.com/user-attachments/assets/1b87ed9e-0498-44c4-ab1b-2c0321e70ce0)

516 fucked with uis again and this unfucks them a bit

:cl:
qol: there's a new UI preference called UI scale which allows people
that use windows scaling to have their UIs original size with the
contents zoomed out, instead of the default, which is the UIs being
larger with the contents "normal" size
fix: various UIs did not respect windows scaling, they now do
/:cl:

---------

Co-authored-by: harryob <[email protected]>
@Absolucy Absolucy added Code: Fix fix da bug QoL improves quality of life Code: TGUI Involves TGUI in some way, shape or form labels Apr 9, 2025
@github-actions github-actions bot added the Merge Conflict DO NOT RENAME THIS LABEL label Apr 11, 2025
@Absolucy Absolucy removed the Merge Conflict DO NOT RENAME THIS LABEL label Apr 11, 2025
@github-actions github-actions bot added the Merge Conflict DO NOT RENAME THIS LABEL label Apr 12, 2025
@Absolucy Absolucy removed the Merge Conflict DO NOT RENAME THIS LABEL label Apr 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code: Fix fix da bug Code: TGUI Involves TGUI in some way, shape or form QoL improves quality of life
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants