bling / fzf.el

A front-end for fzf
GNU General Public License v3.0
360 stars 50 forks source link

Improve C-g, C-x C-c, enable C-x, fix warnings, reduce debugging messages, etc. #108

Closed JohnC32 closed 1 year ago

JohnC32 commented 1 year ago
  1. C-g in the FZF buffer results in exit code of 130. This is now handled and C-g gracefully removes FZF without error messages showing up.

  2. An exit code of 1 from fzf isn't an error, that's when there's no input and now we don't see an error for this case.

  3. C-x C-c was causing an "active process exist" prompt when the FZF buffer was present. Now if you exit while FZF buffer is present you don't get prompted.

  4. Make C-x the terminal escape character. This means the key bindings behave similar to M-x ansi-term and thus you can use C-x 1 to maximize, etc.

  5. The package used to state it required Emacs 24. However, it would not work in Emacs 24 because of the use of seq-filter. The package requires Emacs 25 and later, so I updated Package-Requires. I tested on Emacs 25, 26, 27 as we use all of these.

  6. Fixed a couple compile warnings. Now this compiles warning free on Emacs 25, 26, 27.

  7. Reduced debugging messages. "FZF blah" messages where showing up in Messages buffer even when there was success. I put these under an fzf-show-debug-messages variable thus reducing noise in Messages.

  8. Added 'Interacting with the FZF buffer' to the readme.