-
Notifications
You must be signed in to change notification settings - Fork 202
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
MAYA-111084 - LE save icon missing when scaling is 125% #1408
MAYA-111084 - LE save icon missing when scaling is 125% #1408
Conversation
* We don't provide icons at 125% so make sure to account for that.
#ifdef Q_OS_DARWIN | ||
return baseName + "_100.png"; | ||
#else | ||
const auto scale = utils->dpiScale(); | ||
if (scale >= 2.0) | ||
return baseName + "_200.png"; | ||
else if (scale >= 1.5) | ||
return baseName + "_150.png"; | ||
return baseName + "_100.png"; | ||
#endif |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When running at custom scaling 125%, the base name would become "_125.png" - but we do not provide at 125% icon file. So reworked the code to make sure we always return either _100/_150/_100 - which are the three sizes we support.
background-position: center center; | ||
background-repeat: no-repeat; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At 125 the icon used is _100 so there was extra space and the icon was centered and actually repeated. So this centers it (vert/horz) and turns off repeat. I checked all supported scaling factors (100/125/150/200).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
look good to me.
MAYA-111084 - LE save icon missing when scaling is 125%