NGnius / PowerTools

Moved to
https://git.ngni.us/NG-SD-Plugins/PowerTools
GNU General Public License v3.0
414 stars 29 forks source link

Rename "Persistent" to "Use per-game profile" #59

Closed MarkoG21 closed 1 year ago

MarkoG21 commented 1 year ago

Please confirm

Expected Behaviour

Greetings, I have a small suggestion for a change of your app: I was searching quite a bit, when I was confused as why PowerTools is using the same Profile for every game, even though in the App, it's showing that it uses the game specific profile. Then I found a Issue where you suggested a user to use the "persistent" setting and now it works.

This lead me to think, why this happened and here are my reasons why I think that "Persistent" is not the best match to describe what the settings does:

Actual Behaviour

Activating the setting to be persistent makes it not persistent.

Steps To Reproduce

-

Anything else?

No response

Version

1.0.5 (Latest stable)

Version

Steam Deck

NGnius commented 1 year ago

Thanks for the suggestion! I will admit I've heard a lot about that toggle, with many people saying quite contradictory things about it. In one of those conversations, I even changed the description to make it more clear that "Persistent" refers to the profile and not PowerTools' state as a whole. In that spirit, I'm going to leave this open and not make any immediate changes because I'm not convinced on what the correct fix is (if there even is one needed at all). Hopefully some other people will weight in as well.

Here's my 2c:

PS In the future please don't submit suggestions as bugs. Bugs are unintentional, while your suggestion is for an intentional design decision.

MarkoG21 commented 1 year ago

Oh I apologize for picking the wrong label ^^, didn't quite know in what category to put this suggestion, thanks for fixing it! Yeah, after reading the description of the toggle, it definetly made sense to me, After reading the definition of persistence, associated with computer science, I definetly understand where you are coming from.

I'm also interested about what other opinions there are about this topic, so I hope there will be some other suggestions/points of view too👍

BadJanitor commented 1 year ago

I found this issue because I was also confused by the setting. While I agree that "Persistent" may be the technically correct term from a from a scientific standpoint, but it's not the best term to describe the effect of the setting. The description helps, but I still interpreted this as a global setting, meaning if it's disabled all PowerTools options are reset to default every reboot.

What about just using both terms and calling it "Persistent Profile", with a description or something like "Restores settings for this application after an app or OS restart"?

ekirchman commented 1 year ago

I was actually just pulling up the issues page to create a request for a "per game" profile setting. But I stumbled upon this thread saying that it's already a feature? At least I think? The wording of it is confusing, at least to me, even as a native English speaker

NGnius commented 1 year ago

Just noticed that my link to the new Peristent toggle description is broken because I refactored that file. The new description reads Save profile and load it next time.

"Persistent Profile" is a pretty good option. I'm slightly concerned it's redundant since profile is mentioned in the description and it's literally right beside the Profile menu item, but I'll mull it over before the next release drops.

I'm going to keep my new description for at least the next update since it seems to communicate everything that "Restores settings for this plugin after an app or OS restart" does while avoiding being as wordy. I also try to keep the descriptions short enough to fit on one line, though translations aren't required to abide by that (mostly because I know that's a classic problem with French that the Canadian government is all too familiar with).

ekirchman commented 1 year ago

I do like "Persistent Profile" better. Using the term "profile" helps keep it in line with Valve's own terminology.

If you want to make the description line shorter, do you need to keep the "or OS restart" in there? As a developer, I understand why you might put that there; one can't assume that certain configurations will survive a reboot. But as an end user, I expect a "profile" to survive a reboot.

That's just my own opinion, however. Thanks for taking the time to clarify :)

NGnius commented 1 year ago

Y'all have got your wish: ec97003