bug: wrong labe positioning on double-width chars

[atusy]

Did you check docs and existing issues?

• I have read all the flash.nvim docs
• I have searched the existing issues of flash.nvim
• I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)

NVIM v0.10.0-dev-585+g2e055e49a

Operating system/version

Manjaro Linux

Describe the bug

When patteren has double-width chars, label is placed on a next character of the matches.

Below is an example to search あ from あxあxあxあxあxあx.
You can see labels are placed on x

Steps To Reproduce

1. Prepare buffer

あxあxあxあxあxあx
あxあxあxあxあxあx

2. Do flash

require("flash").jump({pattern = "あ", search = { wrap = false }})

Expected Behavior

labels should be placed on あ

Repro

-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  { "folke/flash.nvim", opts = {} },
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

[folke]

By default labels are placed after the matches, so this works as it should.

There's multiple options to put the labels somewhere else. Please refer to the config section of the readme

[atusy]

Sorry, my example was incomplete.

I noticed this behavior when trying f-motion.

Could you compare these two with a buffer containing あxあxあxあxあxあx?

-- label on x works
require("flash").jump({pattern = "x", search = { wrap = false }, label = { after = {0, 0}, before = false }})

-- trying to label あ
require("flash").jump({pattern = "あ", search = { wrap = false }, label = { after = {0, 0}, before = false }})

[atusy]

I think label = { after = false, before = {0, 0} } is the safer option at here:

flash.nvim/lua/flash/plugins/char.lua

Line 19 in 13022c0

F = { search = { forward = false }, label = { after = { 0, 0 }, before = false } },

[folke]

Should be fixed now!

[atusy]

Thanks, perfect!!