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

Current state of compatibility with FZF's features #628

Open
LoricAndre opened this issue Nov 27, 2024 · 0 comments
Open

Current state of compatibility with FZF's features #628

LoricAndre opened this issue Nov 27, 2024 · 0 comments

Comments

@LoricAndre
Copy link
Contributor

LoricAndre commented Nov 27, 2024

This issue tracks the compatibility of sk with fzf's options.
Please comment with any missing discrepancy that you notice.

Note: even though we aim for roughly the same feature set as fzf, we do allow ourselves to implement the features differently from them.

WIP

  • --tmux option

Planned

  • Case sensitive alt-shift binds , e.g. --bind=alt-g:first,alt-G:last
  • first and last actions (move to first and last line of results respectively)
  • Neovim cursorline like highlighting with --highlight-line (added with fzf v0.53)
  • Missing --info types, can't set --info=inline, have to use --inline-info instead, missing right, inline-right, hidden, etc.
  • Missing event trigger binds such as load|start|change, eg.. --bind=change:reload(...)
  • Missing actions bind|unbind to be able to rebind keys dynamically
  • execute-silent, the new reload has issues with shell escaping inside () (still working on finding the exact bug/repro)
  • --print-query returns nil when used with --interactive (as it's probably returning the hidden query of the non-cmd mode)
  • Shell escaping inconsistencies between how {q} works between fzf and skim
  • no scroll bar option (main fzf window)
  • Missing some hl colors (gutter, scroll bar)
  • Header color overrides ansi escape sequences (can’t have blue header and then highlight a word with a different color)

Done

  • reload action
  • index tiebreak
@LoricAndre LoricAndre pinned this issue Nov 27, 2024
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

No branches or pull requests

1 participant