cargo.el icon indicating copy to clipboard operation
cargo.el copied to clipboard

Simplify keybindings

Open ayrat555 opened this issue 5 years ago • 4 comments

ayrat555 avatar Jan 04 '21 07:01 ayrat555

Maybe we can add the simplified keys in parallel with the originals? That way we don't break anyone's workflow and we get the simplified keys. What do you think?

kwrooijen avatar Jan 04 '21 10:01 kwrooijen

it seems a little fishy to me. In my opinion, we should either break everyone now (and document how to unbreak users in readme) or document how to change the default and stick to it.

mookid avatar Jan 04 '21 10:01 mookid

I'm not a fan breaking changes. If people want to change the keybindings they can. But I don't have a view of how many people actually use this package and the original keybindings. If we do decide to change it, maybe copy the spacemacs standard?

https://github.com/syl20bnr/spacemacs/blob/ee9201efe789e1f1dbff3c15aa9e394265c664e1/layers/%2Blang/rust/packages.el#L35-L57

kwrooijen avatar Jan 04 '21 11:01 kwrooijen

I'm ok with any variant.

Another approach can be creating a new library and deprecate this one. IMHO I think breaking workflow for some users if we provide a better workflow is ok.

Also, does elpa provide updates based on major version of the library?

ayrat555 avatar Jan 04 '21 17:01 ayrat555