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

Revert "Fix flicker in control nodes due to pivot offset" #46891

Merged
merged 1 commit into from
Mar 11, 2021

Conversation

akien-mga
Copy link
Member

@akien-mga akien-mga commented Mar 11, 2021

Reverts #46409

Fixes #46850.

See #46409 (comment) for details.

Users who need this fix can try to disable gui/common/snap_controls_to_pixels which should be equivalent. It may however cause other regressions, which is why we're reverting this for now and we'll investigate further for a more thorough fix in 3.2.5 (hopefully!).

@akien-mga akien-mga added this to the 3.2 milestone Mar 11, 2021
@akien-mga akien-mga requested a review from a team as a code owner March 11, 2021 10:04
@akien-mga akien-mga merged commit 13a57d8 into 3.2 Mar 11, 2021
@akien-mga akien-mga deleted the revert-46409-fix-scale-pivot-jitter branch March 11, 2021 10:58
@akien-mga akien-mga mentioned this pull request Mar 11, 2021
@calumk
Copy link

calumk commented Mar 12, 2021

Just a quick note to say disabling "gui/common/snap_controls_to_pixels" does not seem to fix this for me, but on further inspection, it may not be as simple as i thought.

The error is not everywhere in the game, just in specific places, and there is a difference between 3.2.4rc3 and 3.2.4rc4, but it seems to only affect the editor.

The attached screenshot shows rc3 (left) and rc4 (right)

As you can see, in rc4, the editor text is appearing blurry, in rc3, its sharp and clear.

When actually running the game, both export the same, with a mixture of blurry and non-blurry text.
If I start a fresh project, I cannot seem to duplicate this behaviour, so it could be some setting i have toggled somewhere, but for the life of me i cant work out what.
I would put this down to my user error, except for the fact that the rc4 editor is now blurry for me, when in rc3 it was not, so something must have changed to cause this behaviour.

vs

@akien-mga
Copy link
Member Author

@calumk What you describe is what this PR is fixing. This was caused by the changes to scroll_container.cpp reverted in this PR.

Disabling gui/common/snap_controls_to_pixels is not a way to fix the regression in 3.2.4 RC 4, it's a way to fix the original issue that #46409 was trying to address.

@calumk
Copy link

calumk commented Mar 12, 2021

Many thanks @akien-mga,
I look forward to rc5 :)

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

Successfully merging this pull request may close these issues.

2 participants