Skip to content

Commit

Permalink
chore: skim v0.14.3 adjustments
Browse files Browse the repository at this point in the history
  • Loading branch information
ibhagwan committed Nov 29, 2024
1 parent 461c9a9 commit c3a3b95
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions lua/fzf-lua/actions.lua
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,14 @@ M.expect = function(actions, opts)
v.prefix and "+" or "",
v.prefix and v.prefix:gsub("accept$", ""):gsub("%+$", "") or ""
))
elseif opts.__SK_VERSION and opts.__SK_VERSION >= 0.14 then
-- sk 0.14 deprecated `--bind`, instead `accept(<key>)` should be used
-- skim does not yet support case sensitive alt-shift binds, they are ignored
-- if k:match("^alt%-%u") then return end
if type(v.prefix) == "string" and not v.prefix:match("%+$") then
v.prefix = v.prefix .. "+"
end
table.insert(binds, string.format("%s:%saccept(%s)", k, v.prefix or "", k))
elseif k ~= "enter" then
-- Skim does not support case sensitive alt-shift binds
-- which are supported with fzf since version 0.25
Expand Down

2 comments on commit c3a3b95

@LoricAndre
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ibhagwan technically, expect is deprecated, not bind, but kudos for the quick reaction !

@ibhagwan
Copy link
Owner Author

@ibhagwan ibhagwan commented on c3a3b95 Nov 29, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know, that’s a comment “typo” lol

175ddbb

Please sign in to comment.