Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Added MSG_GUI_[OPEN/CLOSE]` for main menu and selector. #2684

Merged
merged 3 commits into from
Feb 14, 2021

Conversation

ohlidalp
Copy link
Member

Fixes issue with "Skip main menu" setting not working.
Remake of #2680

Fixes issue with "Skip main menu" setting not working.
Other GUI windows to be done.
@ohlidalp ohlidalp changed the title ✉️ Added MSG_GUI_[OPEN/CLOSE]` for main menu and selector. Added MSG_GUI_[OPEN/CLOSE]` for main menu and selector. Feb 13, 2021
@tritonas00
Copy link
Collaborator

it opens like

kk

@ohlidalp
Copy link
Member Author

ohlidalp commented Feb 13, 2021

it opens like

I tried all scenarios and this doesn't happen (I'm using OpenGL renderer, Dx9 doesn't build for me).
Must be linux-specific. Try adding this before the message loop:

// Hack - force rendering image to properly init DearIMGUI
App::GetGuiManager()->GetLoadingWindow()->SetProgress(100, "Hack", /*renderFrame=*/true);
App::GetGuiManager()->SetVisible_LoadingWindow(false);

@tritonas00
Copy link
Collaborator

tritonas00 commented Feb 13, 2021

Try adding this before the message loop:

worked fine! 👍

Back to menu results an empty background with this PR

@ohlidalp ohlidalp requested a review from tritonas00 February 13, 2021 19:48
Copy link
Collaborator

@tritonas00 tritonas00 left a comment

Choose a reason for hiding this comment

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

Main Menu and Selector seem to show/hide when they should.

@ohlidalp ohlidalp merged commit dd827ff into RigsOfRods:master Feb 14, 2021
@ohlidalp ohlidalp deleted the gui-msg branch February 14, 2021 13:06
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants