gittromney / Playnite-NowPlaying

This is a Playnite Library Extension that creates & manages game caches for locally installed PC Windows and/or supported emulated platform games.
MIT License
12 stars 0 forks source link

[Suggestion] Make it clearer/more user friendly on how to disable caching #15

Closed Rings4 closed 1 week ago

Rings4 commented 1 month ago

Just a small suggestion I thought I'd make given how much I like and admire this plugin.

Currently, to disable caching for a game, you have to "Uninstall" it. If you don't remember the docs/instructions for this plugin, you wouldn't think to click "Uninstall" in order to disable caching.

  1. I think you should add the plugin's logo in front of the "Uninstall" option, and maybe even rename that option to "Uninstall from cache" or similar, so it's clear that this is the setting you need to click in order to disable caching.

  2. Perhaps even combine uninstalling and disabling the cache for this game into the same step, so instead of an Install button, you'll have the Play button back after uninstalling.

If you make these settings that you can toggle, I suggest at least making suggestion 1 the default setting as it seems most intuitive.

Thank you again for this great plugin! This is literally a game changer (pun intended).

gittromney commented 1 month ago

Hi, good suggestions - I'll look into the possibility of doing both 1 + 2, and I'm glad you are enjoying the plugin.

One thing to keep in mind is that there might be different use cases for game caching. As an example, one approach could be where just before playing a game you ENABLE caching + INSTALL it (create its cache); then after finishing it you UNINSTALL it (delete its cache) + DISABLE game caching for it.

An alternative approach, and my recommendation, would be instead to ENABLE all of your eligible games for caching at once (remember, doing this does not actually create any game caches, it just makes changes to those game(s) so that they appear to Playnite as coming from a Library.) Then whenever you play any of those games, you will just do INSTALL + PLAY + UNINSTALL. The INSTALL step will actually create the game cache(s) on your NVMe/SSD, and UNINSTALL will delete the cache(s) to free up disk space. In this use case I wouldn't recommend ever DISABLING caching for your games (unless you needed to do so for some other reason).

Hopefully that explanation makes sense.


From: Rings4 @.> Sent: Monday, August 12, 2024 1:33 PM To: gittromney/Playnite-NowPlaying @.> Cc: Subscribed @.***> Subject: [gittromney/Playnite-NowPlaying] [Suggestion] Make it clearer/more user friendly on how to disable caching (Issue #15)

Just a small suggestion I thought I'd make given how much I like and admire this plugin.

Currently, to disable caching for a game, you have to "Uninstall" it. If you don't remember the docs/instructions for this plugin, you wouldn't think to click "Uninstall" in order to disable caching.

  1. I think you should add the plugin's logo in front of the "Uninstall" option, and maybe even rename that option to "Uninstall from cache" or similar, so it's clear that this is the setting you need to click in order to disable caching.

  2. Perhaps even combine uninstalling and removing from the cache into the same step.

If you make these settings that you can toggle, I suggest at least making suggestion 1 the default setting as it seems most intuitive.

Thank you again for this great plugin! This is literally a game changer (pun intended).

— Reply to this email directly, view it on GitHubhttps://github.com/gittromney/Playnite-NowPlaying/issues/15, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A6I4CRDKK525IW4BK4QVGG3ZRELZVAVCNFSM6AAAAABMM4MXKWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ3DCOBUGMYTOMI. You are receiving this because you are subscribed to this thread.Message ID: @.***>

Rings4 commented 1 month ago

That explanation makes perfect sense! That's why I think adding my suggestions as settings as opposed to outright changing the behaviour of the plugin would be great, as it would conform to the person's needs.

Currently, I don't see myself wanting to set up caching for my games in bulk to then install, play, uninstall. Instead, I want to be able to switch caching on when I find that I'm playing a game that's stored on an HDD often enough to justify the SSD space. But, I'm sure in the future, I'll have a slightly different setup and use it the way you describe. I might end up installing an additional internal HDD solely for that purpose.

gittromney commented 1 week ago

Please see the latest release (v1.3.5), which at least in part addresses your suggestions.

For item 2. there's now an optional "Uninstall + Disable" command that will show up under the Playnite right-mouse menu when the selected game(s) have their game cache(s) Installed (and when Settings->General->"Show 'Uninstall cache + Disable caching...' is checked). Note that, by default, this setting is unchecked.

For item 1, unfortunately, I don't see any way of adding Icons or changing text of Playnite's built-in game menu items (like Install/Uninstall/Play). But, as a work-around, I added optional NowPlaying status/menu tips information to the Playnite right-mouse game menu. Again, this option will need to be enabled, as it is disabled by default. (See Settings->General->"Show status/menu tips...")

Please try it out and let me know what you think. Hopefully, the new features are pretty self-explanatory.
--g

gittromney commented 1 week ago

Here are some examples, with both optional features enabled. In case it's not clear, the menu items with a light grey/monochrome logo are informational only - they don't have any "action" or "command" associated with them.

Game with cache uninstalled: Screenshot 2024-08-30 163712

Game with cache installed: Screenshot 2024-08-30 163959

Ineligible game selected: Screenshot 2024-08-30 164109

A variety of games selected: Screenshot 2024-08-30 164451

Also, with this release (v.1.3.5), you can now install multiple caches at once (staged in a queue, the same as if done from the NowPlaying sidepanel view): Screenshot 2024-08-30 164834

Same goes for uninstalling caches (or 'Uninstalling + Disabling'): Screenshot 2024-08-30 164945

Rings4 commented 1 week ago

Absolutely wonderful, thank you very much for implementing my suggestions!