Can't do this until the full suggestion in#15521 (comment) is implemented.

1. We need a right-click menu that has things like:

   CopyCutPasteDisable mouse

   • The last oneDisable mouse is especially important, it should link to a:help tag that explains
     1. how to:set mouse= in one's init.vim,
     2. mentions that you can just hit: to temporarily opt-out of 'mouse'
     3. mentions this alt+click mapping:https://github.com/justinmk/config/blob/6a0432c4d3a6295c03231b9f0be5030f9294540a/.config/nvim/init.vim#L136-L137
2. we needalt+click to temporarilyset mouse= until the next cursor-move. this is an escape-hatch to allow copying screen elements that otherwise aren't copyable

Can't do this until the full suggestion in#15521 (comment) is implemented.

As menu functionality has been implemented, do I understand right that we lack few default commands, such as:menu PopUp.Paste "+gP, plus new help topic?

As menu functionality has been implemented, do I understand right that we lack few default commands, such as:menu PopUp.Paste "+gP, plus new help topic?

correct

@justinmk All done. The menu is embedded into C code to match with default mappings and autocommands.

I also changed the default forset mousemodel=popup_setpos to enable the menu from startup. But now it looks like a problem with functional tests, in particular,this one.

AFAIU, the test emulates mouse clicks in the status line area, but the popup menu shows up unexpectedly. So what's the plan? To reset mousemodel to "extend", or to set it for this test specifically, or to patch popup menu implementation, so it doesn't show up upon a click outside of buffer area?

But now it looks like a problem with functional tests, in particular,this one.
AFAIU, the test emulates mouse clicks in the status line area, but the popup menu shows up unexpectedly. So what's the plan? To reset mousemodel to "extend", or to set it for this test specifically, or to patch popup menu implementation, so it doesn't show up upon a click outside of buffer area?

@famiu does your PR help here?#19252

It should, yeah

@justinmk Okay, no problem. Should we also merge default autocmds in? Kind ofinit_default_actions() for everything, except the options?

could instead be done in Lua

I see neither l10n nor menu API but that's not important, I guess. Just going to embed a piece of VimScript into Lua for now.

Should we also merge default autocmds in? Kind ofinit_default_actions() for everything, except the options?

no, don't want to decide that right now :)

Menus reallyare just mappings. Soinit_default_mappings does not need to be renamed.

Just going to embed a piece of VimScript into Lua for now.

that's fine--thanks!

Moved default mappings and menus to Lua. Cleaned up C code.

The functional test fails until PR#19252 merged.

Conflicts with PR#19365 asinit_default_mappings() is about to be removed from mapping.c. So need to port the new mapping to Lua when accepted.

• mergedfix: right click in clickable statusline #19252 , try rebasing ?
• let's drop the "Undo" menu item for now; it raises the question "what about Redo?", and also doesn't seem valuable enough, especially as the first item.
• do we want aGo to definition item that maps toctrl-] ?
  • would probably want to use<Cmd>lua vim.lsp.buf.definition()<CR> if#vim.lsp.get_active_clients({bufnr=vim.api.nvim_get_current_buf()}) > 0

Future

1. Enabling mouse opens the door to adding clickable sections to the default 'statusline' and 'tabline'.
   • most important function here would be to expose the message / notification history somehow

1. Rebasing done; test "startup_spec" fixed; "undo" item removed; Visual Paste changed from "p" to "P".

2. The test "statusline_spec" is falling again. But it only complains that it failed to require "vim.shared". And I can't figure why so?!

3. "Go to definition" item currently may look like this:

anoremenu PopUp.Go\to\definition<Cmd>lua\if #vim.lsp.get_active_clients() >0 then\vim.lsp.buf.definition()\else\vim.cmd'normal! <C-]>'\end<CR>vnoremenu PopUp.Go\to\definition<Cmd>lua\if #vim.lsp.get_active_clients() >0 then\vim.lsp.buf.definition()\else\vim.cmd'normal! <C-]>'\end<CR>

It's working but looks ugly, so I didn't add it yet. BTW. "vnoremenu" with the same code needed, because "amenu" would drop from Visual mode, and so it ignored the selected text in favor of the word under cursor.

4. Considering this and also a "save view" capability, I thought that we might need some new helper for that. Perhaps, something like this

---@paramexprLua function or string of"normals"---@paramoptsTable with optional keys:---                               args - array of args for expr---                               cond, fallback - some condition and "fallback" action---                               keepview, remap - some boolean flags---                               more stuff maybe...functionvim.action(expr,opts)...end

Then the menu items could look like this<Cmd>lua vim.action(vim.lsp.buf.definition, { cond = #vim.lsp.get_active_clients() > 0, fallback = '<C-]>' })<CR>. Or maybe<Cmd>lua vim.action(vim.api.nvim_put, { args = {...}, keepview = true } )<CR>, and so on.

But this one could also become redundant if menu API is ever ported to Lua.

5. I was also thinking about new<Lua> token for mappings. Sort of

<Lua>print"Hello world"<CR>

which is a rough equivalent of

<Cmd>lua<< <CR>print"Hello world"<CR>

except the latter is not working, of course. But this probably deserves opening a new issue first. And again, it's more useful for mappings defined in VimScript. Withnvim_set_keymap() and friends floating around it may be considered obsolete from the very beginning.

do we need aaunmenu intest/functional/ui/statusline_spec.lua ?

RUN      T4735 statusline clicks works: -- Tests exited non-zero: Process terminated due to timeout

windows CI failure is unrelated, known flakey test:

  ERROR    test/functional\vimscript\let_spec.lua @ 28: :let :unlet self-referencing node in a List graph #6070  test\functional\helpers.lua:106: EOF was received from Nvim. Likely the Nvim process crashed.

• would probably want to use<Cmd>lua vim.lsp.buf.definition()<CR> if#vim.lsp.get_active_clients({bufnr=vim.api.nvim_get_current_buf()}) > 0

'tagfunc' is set by default in LSP buffers now, so we may not need this special case.

Resolution

• Oceania has always been at war with Eastasia. Nvim has always enabled 'mouse' by default.
• defaults: Revert 'mouse=a' #6022 temporarily disabled 'mouse' with the intention of re-enabling it after we had a full answer for users who were used to the old behavior.
• The full answer is now provided by this PR, specifically:
  • :set mouse=nvi
  • Pressing ":" (to enter cmdline-mode) temporarilydisables mouse. Try it.
  • Right-click shows a popup menu with:Cut,Copy,Paste,Disable mouse
    • TheDisable mouse item shows:help default-mouse which explains how to disable mouse permanently.

Future

1. Do we want aGo to definition item that maps toctrl-] ?
   • Also works for LSP becausevim.lsp sets 'tagfunc' by default.
2. Enabling mouse opens the door to adding clickable sections to the default 'statusline' and 'tabline'.
   • most important function here would be to expose the message / notification history somehow

Locked to keep the summary visible. You can alwayschat or open an issue if you have new information/topics to discuss.