feature: exclusive operator

[musjj]

Did you check the docs?

• I have read all the flash.nvim docs

Is your feature request related to a problem? Please describe.

When doing directional jumps in an operator pending mode, like this:

require("flash").jump {
  search = { forward = true, wrap = false, multi_window = false },
}

You sometimes want to exclude the target from the operation, similar to operations like dtw

Describe the solution you'd like

An option to make the jump behave like an exclusive operator.

Describe alternatives you've considered

Currently, I don't think there's an option that can simulate this behavior.

Additional context

No response

[folke]

I added a new option opts.jump.inclusive
By default it is nil meaning it will be automatically set based on the mode you're in.

You can set it to true/false to force inclusive/exclusive

[musjj]

Thanks, but it seems that the snippet in my original post no longer works (it doesn't seem to do anything). A plain require("flash").jump() still works, though.

[folke]

Damn, you're right. On it

[folke]

Fixed now. Sorry about that :)

[musjj]

Thanks, it's working! But with backward jumps, it seems that it always jumps to the beginning of the match.

For example, using the following config:

require("flash").jump(
    search = { forward = false, wrap = false, multi_window = false },
    jump = { inclusive = false }
)

With the following text, the cursor at ^

The quick brown fox jumped over the lazy dog.
                               ^

While using the d operation, and the candidate ed, you will get:

- The quick brown fox jumped over the lazy dog.
+ The quick brown fox jump the lazy dog.

What I expect:

- The quick brown fox jumped over the lazy dog.
+ The quick brown fox jumped the lazy dog.

[folke]

You should add jump.pos = "end"

[musjj]

Thank you! With this, flash.nvim has reached feature-parity (plus more) with leap.nvim :)

[musjj]

Sorry for being a party pooper, but it turns out that it's still not working quite right.
The following performed on operator-pending mode:

require("flash").jump { 
  search = { forward = false, wrap = false, multi_window = false },
  jump = { inclusive = false, pos = "end" },
}

Doesn't exclude the last character of the match.

[folke]

I added jump.offset, but when using pos="end" and the jump position is before the current position, it will default to 1.

You also probably still want to use inclusive mode.

So you should change it to:

require("flash").jump { 
  search = { forward = false, wrap = false, multi_window = false },
  jump = { pos = "end" },
}

This should now work as expected.

Let me know if it does! :)

[musjj]

Thanks, it's working perfectly!