Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

C-z is really hard to hit and RSI causing. #547

Closed
wants to merge 1 commit into from

Conversation

lewang
Copy link
Member

@lewang lewang commented Jul 1, 2014

  1. Make new binding C-o.
  2. Prefer this new binding in documentation.

I can update the wiki once this PR is merged.

@tuhdo
Copy link
Contributor

tuhdo commented Jul 1, 2014

Yes, the new binding is much better. Please merge it.

@thierryvolpiatto
Copy link
Member

It is not what I want for helm. I would prefer extending right arrow key (what we already have in many places) to whole helm (customizable of course). We have already discussed this IIRC and I was against for persistent-action.

@lewang
Copy link
Member Author

lewang commented Jul 2, 2014

What specifically do you object to?

  • Adding C-o binding.
  • Documenting C-o instead of C-z, although C-z still works.

On Wed, Jul 2, 2014 at 8:31 AM, Thierry Volpiatto notifications@github.com
wrote:

It is not what I want for helm. I would prefer extending right arrow key
(what we already have in many places) to whole helm (customizable of
course). We have already discussed this IIRC and I was against for
persistent-action.


Reply to this email directly or view it on GitHub
#547 (comment).

Le

@thierryvolpiatto
Copy link
Member

Le Wang notifications@github.com writes:

What specifically do you object to?

I don't see why C-o would be better than C-z.
If one want C-o or whatever, it is configurable AFAIK.
As I said previously I am in favor of using the arrow keys like we do in
*grep, *moccur, *imenu etc...

Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997

@lewang
Copy link
Member Author

lewang commented Jul 2, 2014

On Wed, Jul 2, 2014 at 10:50 AM, Thierry Volpiatto <notifications@github.com

wrote:

I don't see why C-o would be better than C-z.

C-z is very hard to hit without mangling one's fingers weirdly.

If one want C-o or whatever, it is configurable AFAIK.

Yes absolutely. This is not about removing C-z binding. I'm just
proposing making C-o the preferred binding in the documentation. This
makes it more friendly to new users. Users who are already accustomed to
C-z do not need to change anything.

As I said previously I am in favor of using the arrow keys like we do in

*grep, *moccur, *imenu etc...

The persistent action is one of the most common keystrokes a user makes in
Helm. This is about making the default more ergonomic for new users while
preserving the old behaviour. Arrow keys work but are too out of the way
for such a common command.

Le

@thierryvolpiatto
Copy link
Member

Le Wang notifications@github.com writes:

Yes absolutely. This is not about removing C-z binding.

I understood, but again it is not what I want.

Thus C-o is already bound to `helm-next-source'.

Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997

@lewang
Copy link
Member Author

lewang commented Jul 2, 2014

On Wed, Jul 2, 2014 at 12:14 PM, Thierry Volpiatto <notifications@github.com

wrote:

Le Wang notifications@github.com writes:

Yes absolutely. This is not about removing C-z binding.

I understood, but again it is not what I want.

Thus C-o is already bound to `helm-next-source'.

I didn't realize that C-o was bound by default. How about M-o or maybe you
can propose a compromise binding. C-z is just too unergonomic to be the
default.

Le

@rranelli
Copy link

rranelli commented Jul 6, 2014

+1 for the unergonomics of C-z.

@thierryvolpiatto
Copy link
Member

I have now removed C-j and C-e commands, so they are now free, I think using C-j for persistent action would be the best now, maybe you can change C-o to C-j and apply your patch on top of this.

Thanks.

@lewang
Copy link
Member Author

lewang commented Jul 7, 2014

will do today.

thierryvolpiatto pushed a commit that referenced this pull request Jul 8, 2014
* helm-files.el: same.
* helm-help.el: same.
@thierryvolpiatto
Copy link
Member

Ok I have replaced all occurences of C-z where needed, feel free to reopen if you find some more.

@thierryvolpiatto thierryvolpiatto deleted the alternative-C-z-binding branch July 15, 2014 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants