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

Revert "MenuBar: Tweak string for open user folder option" #11453

Merged
merged 1 commit into from Jan 17, 2023

Conversation

JosJuice
Copy link
Member

This reverts PR #11451.

The user folder can be either global or local. If it is local, we shouldn't call it global.

See also PR #10216.

This reverts commit 6788bda.

The user folder can be either global or local. If it is local,
we shouldn't call it global.

See also da0be24.
@MayImilae
Copy link
Contributor

It would be nicer to have the menu item change depending on whether the user directory was local or global, rather than reverting. But we can still do that later after a revert, so eh.

Copy link
Contributor

@iwubcode iwubcode left a comment

Choose a reason for hiding this comment

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

LGTM

@iwubcode
Copy link
Contributor

iwubcode commented Jan 16, 2023

I missed the initial discussion, so maybe there's an obvious reason but I don't really see a point to distinguishing between "global" and "local". It makes it harder for support (user could say "there's no 'Open Global User Folder') and just in general...why should the user care? It's the folder where they want to make changes for their given instance, does it matter where it is? Anyway, that is just my initial reaction, I'll roll with whatever :)

@delroth delroth merged commit c527aea into dolphin-emu:master Jan 17, 2023
11 checks passed
@JosJuice JosJuice deleted the revert-open-user-dir-qt-hotfix branch January 18, 2023 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants