helix icon indicating copy to clipboard operation
helix copied to clipboard

Fix `x` behavior on empty lines

Open dpc opened this issue 3 years ago • 1 comments

This is to address my personal issue with #356.

I'm not sure if this is an ideal implementation, or will it get merged, but was easy enough and it should be easy for me to use a personal fork, just like I used to for kakoune: https://github.com/dpc/kakoune-dpc/

dpc avatar Nov 22 '22 04:11 dpc

For people finding this later -- this PR is old of course but it looks like dpc is actively maintaining his patch on https://github.com/dpc/helix

You can apply the current one on NixOS by adding to your systempackages

    (helix.overrideAttrs (old: {
      patches = (old.patches or []) ++ [
        (fetchpatch {
          name = "dpc-fix-x.patch";
          url = "https://github.com/helix-editor/helix/commit/054c33f530168e9fc61940a2a9e441debf6a73cc.patch";
          hash = "sha256-wvjzIHdi2ZH8lKhqAVxLP1xcjtjr92DGXMU8nZY9CkM=";
        })
      ];
    }))

in place of helix.

apoelstra avatar May 16 '25 15:05 apoelstra

Surprisingly, I don't think this patch required much "maintaining". It keeps applying cleanly for more than 2 years now, IIRC.

dpc avatar May 16 '25 16:05 dpc

Could this maybe be merged as a separate command if the default intuition-tripping behavior must be preserved?

eugenesvk avatar Oct 13 '25 06:10 eugenesvk