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
BUG: Fix sample data set labels not appearing on large desktop monitors #6731
Conversation
# two columns of buttons fit into the size of the Welcome module's minimum width | ||
# on screens with a various resolution and scaling (see qt.QDesktopWidget().size, | ||
# desktop.devicePixelRatioF(), qt.QDesktopWidget().physicalDpiX()) | ||
iconSize = qt.QSize(int(mainWindow.iconSize.width() * 6), int(mainWindow.iconSize.height() * 4)) |
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.
Thanks for fixing this 👌🙏
Are the multipliers 6
and 4
required?
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.
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.
I understand now. Thanks for clarifying.
# two columns of buttons fit into the size of the Welcome module's minimum width | ||
# on screens with a various resolution and scaling (see qt.QDesktopWidget().size, | ||
# desktop.devicePixelRatioF(), qt.QDesktopWidget().physicalDpiX()) | ||
iconSize = qt.QSize(int(mainWindow.iconSize.width() * 6), int(mainWindow.iconSize.height() * 4)) |
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.
I understand now. Thanks for clarifying.
Thanks for testing! |
Fixes the issue of clipped labels in Sample Data module by using the default icon size as a basis, which is more robust than computing it relative to window size.
Before the fix:
After the fix:
Tested on both large desktop and small laptop screens.