Display 'discard unstaged changes' above 'discard all changes'

[strboul]

In the discard changes options popup (invoked by pressing d in the Files pane), when there's a file with MM status, there are currently three options are seen:

discard all changes
discard unstaged changes
cancel

I think it's better to show the 'discard unstaged changes' option first in this list. UX-wise, it's a good practice to go from specific to general, as the impact of discarding some changes is smaller than discarding all changes. For example, when I stage some lines and want to discard the rest, I sometimes too quickly select the first option and end up discarding all changes instead.

[jesseduffield]

My main goal with lazygit is to reduce the number of keypresses required to do things, meaning the most commonly used commands go first (unless there is some more natural ordering that can be done). I rarely find myself discarding unstaged changes. I'm interested to know what the more common use case is, but even if it's more common to want to discard unstaged changes first, I don't think it's worth making a change now because of everybody with the muscle memory for the current options

[ChrisMcD1]

/close

Modern LazyGit has keybindings for each of these options, so you should have no risk of pressing down the incorrect number of times and choosing the wrong option. Changing this setting at this point would just break people's muscle memory