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

Fix #535 - Combobox offset when using fractional scaling #544

Merged
merged 5 commits into from
Jun 8, 2022

Conversation

jmarrec
Copy link
Collaborator

@jmarrec jmarrec commented Jun 7, 2022

TODO: investigate side effects / display glitches this brings?

This only happens when you use Fractional Scaling, eg 250%. It works at 200% or 300%.

See this MCVE: https://github.com/jmarrec/QtTestbed/tree/main/Test-Combobox-Popup-Issue

And the bug report I filed at Qt's issue tracker: https://bugreports.qt.io/browse/QTBUG-104073

@jmarrec jmarrec requested a review from macumber June 7, 2022 15:13
@jmarrec jmarrec self-assigned this Jun 7, 2022
@jmarrec
Copy link
Collaborator Author

jmarrec commented Jun 7, 2022

The only glitch I can see is this:

image

Versus with RoundPreferDown

image

…earGradient, icon and text to void antialiasing issues
@jmarrec
Copy link
Collaborator Author

jmarrec commented Jun 7, 2022

I replaced the gradient image (1px wide) + image header (icon + pixelised text) by the icon, a QLinearGradient and some text placed by the QPainter:

Before:

image

After:

image

@jmarrec
Copy link
Collaborator Author

jmarrec commented Jun 7, 2022

Final result with Muli-Regular embedded (that's what's used on the https://openstudiocoalition.org/)

image

@jmarrec jmarrec merged commit 6d6e2e4 into develop Jun 8, 2022
@jmarrec jmarrec deleted the 535_Combobox_Offset branch June 8, 2022 00:04
@github-actions github-actions bot locked and limited conversation to collaborators Jun 8, 2022
@@ -159,14 +161,29 @@ void OSDialog::paintEvent(QPaintEvent* event) {

painter.setRenderHint(QPainter::Antialiasing);

QImage centerHeader = QImage(":/images/header-backgnd-1px-wide.png");
Copy link
Collaborator

Choose a reason for hiding this comment

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

Awesome, we should nuke this png at some point

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Offset drop downs for measure arguments on the measure tab
2 participants