On 2024-05-22 23:23, Rubén Justo wrote:
On Wed, May 22, 2024 at 10:09:25AM +0200, Dragan Simic wrote:> @@ -1387,7 +1388,7 @@ N_("j - leave this hunk undecided, see next > undecided hunk\n" > "/ - search for a hunk matching the given regex\n" > "s - split the current hunk into smaller hunks\n" > "e - manually edit the current hunk\n" > - "p - print the current hunk\n" > + "p - print the current hunk, 'P' to use the pager\n" I think it would be better to move the description of "P" into a separate line after the "p" line, perhaps like this: "P - use the pager to print the current hunk\n" I know, we'd sacrifice one line of the valuable vertical space this way, but I find it more consistent and much harder to miss the new "P" option.Making 'P' a /version/ of 'p' allows us to skip adding 'P' to the list of available options: (1/1) Stage this hunk [y,n,q,a,d,j,J,k,K,g,/,s,e,p,P,?] This is what I though and this long list is what worries me.
Oh, I wouldn't be worried too much about the length of the list of the options. It's feature-rich, so the list has to be a bit long. :)
But I see your point. I am not opposed to adding a new line.
Thanks.
