MarcoIeni / intellimacs

Spacemacs' like key bindings for IntelliJ platform.
MIT License
538 stars 94 forks source link

GotoPreviousError defined twice #24

Closed elken closed 3 years ago

elken commented 3 years ago

https://github.com/MarcoIeni/intellimacs/blob/main/spacemacs/errors.vim#L5 and https://github.com/MarcoIeni/intellimacs/blob/main/spacemacs/errors.vim#L13

Same action bound to 2 bindings; which one would you want to be correct?

MarcoIeni commented 3 years ago

Two key bindings are defined for the same action. It's normal. Do you see a problem there? Does not work for you? For which-key something like this would work: https://github.com/MarcoIeni/intellimacs/blob/56e84916731a9a91c6bb5e8c73e8291af050fa8c/spacemacs/leader.vim#L28-L32

elken commented 3 years ago

It just seems redundant

Get BlueMail for Android

On 10 Mar 2021, 06:41, at 06:41, Marco Ieni notifications@github.com wrote:

Two key bindings are defined for the same action. It's normal. Do you see a problem there? Does not work for you?

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you authored the thread.
Reply to this email directly, <a href="https://github.com/MarcoIeni/intellimacs/issues/24#issuecomment-794970435">view it on GitHub, or <a href="https://github.com/notifications/unsubscribe-auth/AAV5MHRRVFC2SYWTMKIFF3DTC4H73ANCNFSM4Y3BYA2Q">unsubscribe.<img src="https://github.com/notifications/beacon/AAV5MHVPXSZ5APFI3Y4DGYLTC4H73A5CNFSM4Y3BYA22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOF5RESQY.gif" height="1" width="1" alt="" />

MarcoIeni commented 3 years ago

it's the same in spacemacs image Closing this, feel free to add other comments if there is something more :)