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

Scale challenge icons based on screen height #12858

Merged

Conversation

LillyJadeKatrin
Copy link
Contributor

No description provided.

@JosJuice
Copy link
Member

JosJuice commented Jun 14, 2024

This will be shown next to text, right? If so, you should use DisplayFramebufferScale so you get the same scale factor as for text.

(Or is imgui already scaling image sizes by this? Is this change being done for another reason than HiDPI support?)

@LillyJadeKatrin
Copy link
Contributor Author

The change is actually kinda the opposite of HiDPI; there were complaints that at smaller screen sizes the icons were taking up too much space. Looked up what That Other Emulator Platform does and for 64x64 icons (same ones as here) they were scaled to 1/16 of the window height, which at 1024 is full size, hence the calculation I perform here.

@iwubcode
Copy link
Contributor

iwubcode commented Jun 15, 2024

I appreciate you bringing that up Jos. I questioned it too but forgot to post about it. We do already scale for hidpi (just to mention).

It feels a bit strange to me to scale just the icons. I'd almost prefer if we came up with what visual look we want the UI to have at our max resolution (or some specific resolution) and then if users still want to scale, we could have a general purpose scale element that increases or decreases the UI size to user taste.

But at the same time, I don't want to block this over something so silly. If this needs to get pushed, we can figure things out later.

@LillyJadeKatrin LillyJadeKatrin force-pushed the retroachievements-challenge-scale branch from 9b58e8d to 11ad587 Compare June 15, 2024 20:33
@LillyJadeKatrin LillyJadeKatrin force-pushed the retroachievements-challenge-scale branch from 11ad587 to 45563a8 Compare June 19, 2024 18:32
Copy link
Member

@JosJuice JosJuice left a comment

Choose a reason for hiding this comment

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

It's fine I guess.

@JMC47 JMC47 merged commit 0cc1d4c into dolphin-emu:master Jun 20, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants