Skip to content

Scaling is a bit off #694

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

Open
CelticMinstrel opened this issue Mar 9, 2025 · 2 comments
Open

Scaling is a bit off #694

CelticMinstrel opened this issue Mar 9, 2025 · 2 comments
Labels
bug dialogxml Affects the dialog system

Comments

@CelticMinstrel
Copy link
Member

I don't know what's happening exactly, but elements are positioned a little bit differently when you set a higher scaling factor. One particularly notable example is the Force Default Party option in the scenario editor preferences – at a scale of 2, it overlaps the 3 LED. At a scale of 3 it's pushed even further down, overlapping the Cancel and OK buttons. I've only noticed it in dialogs so I don't think it affects the main window. I wonder if it could be that relative positioning is not accounting for scale somehow?

@CelticMinstrel CelticMinstrel added bug dialogxml Affects the dialog system labels Mar 9, 2025
@NQNStudios
Copy link
Collaborator

NQNStudios commented Mar 14, 2025

separate bug, fixed nowBecause I've just discovered that implicitly calling recalcRect() on controls in setText() causes a lot of problems. Every frame drifts a little in relative positioning, and when you page back and forth in a stack you get the frames warping more and more every time:

Image

It seemed like a good idea, and I don't know what you were fixing in your branch when you changed that, but for my branch I'm just going to comment that out for now.

@CelticMinstrel
Copy link
Member Author

The rect needs to be recalculated for the pattern name labels. I think that was the reason I made that change, and I made it general because (as you say) it seemed like a good idea. The alternative would have been to make racalcRect public so it can be called directly by the pattern picker.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug dialogxml Affects the dialog system
Projects
None yet
Development

No branches or pull requests

2 participants