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

UI: Add games directory and allow switching of games via controller in popup-menu #1627

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

TrentSeed
Copy link

At the moment, switching between games in Xemu typically requires using a mouse/keyboard. When using a game controller, it's possible to open the popup menu, and select "Load Disc...", however that launches a native file browser which does not have controller input support, requiring use of a mouse/keyboard to select the target file. The goal of this PR is to make switching between games a bit more convenient and possible using a controller.

This PR introduces a new setting called "Games directory" within Settings > General > Miscellaneous. By default there is no value specified, and when clicked the user is presented with a file browser to select the desired directory containing .iso game files.

When the popup menu is activated (via right click, or start/back from an Xbox Controller S), there is now a "Games" submenu button that when selected, will present the user with a sorted list of games based on the configured directory. Selecting a game will result in Xemu loading the corresponding disc file. In the event no .iso files are detected in the directory, or if the directory is still undefined, there is an option "No games found" that when selected will open the main settings UI so that the user may adjust the games directory as needed.

All Settings

Root Popup Menu

Games Popup Menu - With .iso Files Found

Games Popup Menu - No Files Found / Directory Not Set

ui/xui/popup-menu.cc Outdated Show resolved Hide resolved
ui/xui/popup-menu.cc Outdated Show resolved Hide resolved
ui/xui/popup-menu.cc Outdated Show resolved Hide resolved
@mborgerson
Copy link
Member

Thanks for addressing the review feedback. I've rebased onto the latest version of the master branch. Can you please fix the build issues, then I'll happily merge.

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

Successfully merging this pull request may close these issues.

3 participants