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

Disabled Creation of Multiple Loadouts for α Release #1416

Merged
merged 10 commits into from
May 23, 2024

Conversation

Sewer56
Copy link
Member

@Sewer56 Sewer56 commented May 22, 2024

fixes #1389

This disables the creation of more than 1 loadout for the α Release.

image

The pill has been modified so the View button stretches to the size of the remaining space.

Post first α release, we can make the setting visible in Release builds.
(Currently this setting is Debug only and does not ship to users)

image

image

The pill with Add button restored.


Outside of that, unsupported games have been hidden from Release builds by default and the features have very clearly been marked as Experimental since we don't yet have visible Settings categories.

I also renamed the manage-game CLI command, since it was inaccurate with what it did given the recent changes. Somehow that slipped past review.

@Sewer56 Sewer56 added the Design UI/UX This is related to the UI. label May 22, 2024
@Sewer56 Sewer56 requested a review from a team May 22, 2024 16:49
@Sewer56 Sewer56 self-assigned this May 22, 2024
@halgari
Copy link
Collaborator

halgari commented May 22, 2024

Oh gosh, I'm going to be pedantic on this again. Didn't @Pickysaurus mention we wanted "Unsupported" not "Experimental"? Or was I not listening?

@Sewer56
Copy link
Member Author

Sewer56 commented May 22, 2024

Oh gosh, I'm going to be pedantic on this again. Didn't @Pickysaurus mention we wanted "Unsupported" not "Experimental"? Or was I not listening?

The response was to the question 'What to name the setting to enable all games'. Which is called Enable Unsupported Games. (Edit Note: The original suggestion was Enable Experimental Games)

Because we don't have setting categories currently, I prepended text to the dangerous ones.

I could make it so it says (Unsupported) Enable Unsupported Games though, even if it looks a bit weird.
Edit: Did that just now.

Co-authored-by: erri120 <erri120@fossmailer.de>
@Sewer56 Sewer56 merged commit 8b9a358 into main May 23, 2024
3 of 7 checks passed
@erri120 erri120 deleted the disable-multi-loadouts branch May 23, 2024 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Design UI/UX This is related to the UI.
3 participants