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

Suggestion: Make yad settings window scale dynamically #81

Closed
Antiquete opened this issue Sep 27, 2020 · 3 comments · Fixed by #82
Closed

Suggestion: Make yad settings window scale dynamically #81

Antiquete opened this issue Sep 27, 2020 · 3 comments · Fixed by #82

Comments

@Antiquete
Copy link
Contributor

Antiquete commented Sep 27, 2020

Problem

Right now the size of yad settings window is static. It is either fetched from global.conf or if settings are missing there then directly from stl script. This causes issues for people with different resolutions #73

Solution

Make STLSETWIDTH and STLSETHEIGHT scale dynamically based on a percentage of user's resolution. This will make ui scale consistently across resolutions.

How to implement this?

I am thinking of using xrandr in some form to fetch resolution and then simply setting height and width based on that. Xrandr is preferred since that generally comes preinstalled with most linux distros, however, any other alternative is also good enough.

@Antiquete
Copy link
Contributor Author

Just now noticed that xrandr also comes in as a dependency for steam itself, so it can be used without adding any extra dependency on stl's side.

@Antiquete
Copy link
Contributor Author

I will branch this and see if I can get this to work on my end.

@frostworx
Copy link
Collaborator

This looks like a very nice idea and implementation. Thanks for the contribution!

frostworx added a commit that referenced this issue Sep 27, 2020
Add dynamic scaling to yad ui based on resolution
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 a pull request may close this issue.

2 participants