Skip to content

Improved welcome dialog - #16378

Merged
elad335 merged 4 commits into
RPCS3:masterfrom
digant73:improved_welcome
Dec 7, 2024
Merged

Improved welcome dialog#16378
elad335 merged 4 commits into
RPCS3:masterfrom
digant73:improved_welcome

Conversation

@digant73

@digant73 digant73 commented Dec 7, 2024

Copy link
Copy Markdown
Contributor

Followup of #16373 according to some received feedbacks

IMPROVEMENTS:

  • Contextual Welcome dialog:
    • initial welcome: at application startup, a dialog asking to accept or reject RPCS3's usage conditions is provided. In case the reject button (Exit) is pressed, the application is gracefully terminated. Accept button (Continue) is enabled only if the agreement checkbox (I have read the Quickstart guide) is checked
    • standard welcome: welcome dialog opened from Help menu will show only main info and applicable settings. Dark theme, reject button and agreement checkbox are not provided

BUGFIXES:

  • Fixed crash due to access violation on main_window::SaveWindowState() method in case the gui_application::Init() method failed (false was returned) to initialize some modules leaving other modules uninitialized (NULL pointed).

Initial Welcome dialog:

initial_welcome

Standard Welcome dialog:

standard_welcome

Comment thread rpcs3/rpcs3qt/gui_application.cpp Outdated
Comment thread rpcs3/rpcs3qt/main_window.cpp Outdated
Comment thread rpcs3/rpcs3qt/main_window.cpp
Comment thread rpcs3/rpcs3qt/welcome_dialog.cpp Outdated
Comment thread rpcs3/rpcs3qt/welcome_dialog.ui
@Megamouse Megamouse added the GUI label Dec 7, 2024
Comment thread rpcs3/rpcs3qt/gui_application.cpp Outdated
@elad335
elad335 merged commit 68b7e59 into RPCS3:master Dec 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants