Feature suggestion: Allow to lock talent loadouts
tflo opened this issue Β· 2 comments
It would be great if we could lock a talent loadout, i.e protect it against modifications or deletion.
Why?
- I often import loadouts from sites like method.gg or ivy-veins and use them as a healthy base from which to build various custom loadouts. Usually I do this by creating a clone and modifying the clone. However, sometimes I forget that a particular loadout is an imported "base" loadout and I accidentally modify or delete it.
- I have some loadouts that I'm absolutely sure are the best for a given environment. However, sometimes I want to test a small tweak and, silly me, instead of tweaking a clone, I tweak the tried and tested original, and of course forget to revert it later.
A simple Lock option would protect me from my stupidness π
Implementation suggestions
The lock/unlock could be implemented via context menu (right-click menu) of a loadout. A little lock icon before (or after) the loadout name in the sidebar would be just the icing on the cake.
The Locked state should make it impossible to save changes to a loadout, to delete it, or to rename it.
Notification
The user should be notified about the Locked state via a pop-up dialog box, telling them that the loadout is locked and β another icing on the cake β offering an Unlock button and a Crete New Loadout button, in addition to the Cancel button.
Now, I see to possibilities when the box should be popped up:
A) Right away when I try to modify a locked loadout, orβ¦
B) only when I try to save, delete, or rename a locked loadout.
ATM, I tend to think that option B would be better, because it still allows to temporarily modify a loadout and load it into the Blizz loadouts ("Apply Changes"), and later to revert back by simply activating the saved loadout in TLM again.
However, option A would be "cleaner" (and probably more transparent), and it would educate me (i.e. force me) to always create a clone loadout for tweaks or experiments, even temporarily.
Thanks in advance for considering my suggestion, and many thanks for this really useful addon!
β Tom
I'll put this on my list π
though I would probably start of by simply disabling the save/rename/delete buttons, and disable the import-into-current-custom-loadout option; rather than a popup
My proposed implementation details is just how I would do it if it was my addon (and if I had the time π).
Of course any implementation would be welcome, and just disabling the buttons and menu entries would be perfectly fine!
BTW, speaking of disabling buttons: I noticed that the Save button is always enabled, even if there is nothing to save (no changes made). This makes me always click the button, even if I only looked at a loadout. Probably an OCD thing of mine, but if this button was dimmed if there is nothing to save, that would be great!