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 single-frame title-screen flicker when loading a game or starting a campaign. #6648

Merged
merged 1 commit into from Apr 28, 2022

Conversation

mesilliac
Copy link
Contributor

The title screen was being displayed for a single frame in between loading dialogs when either starting a new campaign or loading a game.

This was giving me an aneurysm.

I just turned off drawing of the title screen after these processes are confirmed. The next time the title screen needs to be drawn, it is recreated from scratch anyway, so the existing instance should never actually need to redraw.

It was being displayed for a single frame in between loading dialogs.
@github-actions github-actions bot added the UI User interface issues, including both back-end and front-end issues. label Apr 28, 2022
@Vultraz Vultraz merged commit eb9f463 into wesnoth:master Apr 28, 2022
@Wedge009
Copy link
Member

I think that was a known issue - was it formally recorded (and therefore should be closed)? Or perhaps not...

@mesilliac
Copy link
Contributor Author

I searched but didn't find an open issue for it. Could be it was small and obvious enough that nobody bothered to report it.

@Wedge009
Copy link
Member

Wedge009 commented Apr 29, 2022

No worries, wasn't asking you specifically. Thanks for resolving the issue!

PS @Vultraz Could/should this be back-ported (if it's not already)?

@mesilliac mesilliac deleted the loading_screen_flicker_fix branch April 29, 2022 00:35
@Wedge009
Copy link
Member

Never mind, found it in #2395.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI User interface issues, including both back-end and front-end issues.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants