search

glacambre / firenvim

Embed Neovim in Chrome, Firefox & others.
GNU General Public License v3.0
4.76k stars 146 forks source link

Cannot `set mouse=` #1505

Closed ysl2 closed 1 year ago

ysl2 commented 1 year ago

I found that the vim.cmd('set mouse=') cannot work for firenvim.

glacambre commented 1 year ago

Huh, indeed, it looks like after complaining about https://github.com/neovim/neovim/issues/11372 I never actually took advantage of the fix.

Thank you for reporting this issue, I'll try to fix it soon.

glacambre commented 1 year ago

For now, as a workaround, you can :map <LeftMouse> <nop> and :map <RightMouse> <nop> in order to disable the mouse :)

ysl2 commented 1 year ago

Thanks! :)

ysl2 commented 1 year ago

In fact I want to disable <MiddleMouse>, but I found it didn't work after mapping it to <Nop>

glacambre commented 1 year ago

This is solved in master, the fix will be available to you when I publish a new release on the add-on stores :)

martin-braun commented 1 year ago

 @glacambre Just a heads up, I tried vim.opt.mouse = "c" and vim.opt.mouse = "" and it doesn't work.

2023-07-01 13 57 43

MacOS

I also can confirm that remapping <LeftMouse> and <RightMouse> works, but remapping <MiddleMouse> does not.

glacambre commented 1 year ago

@martin-braun Thanks for letting me know. It looks like I haven't published a new release since I merged this, I'll try to get it done soon.

martin-braun commented 1 year ago

@glacambre No pressure, I'm good with the work-around. I just reported it for the case that there might be still a problem, but failed to see that the last release isn't so young like I initially expected.

Please take your time, stability has more priority. :)