mapping problem with commandline and search /

[molleweide]

Hi again,

I try this binding

	    {'n', ';', ':'},

and it does work but the command line is not being shown upon pressing ; now.

1.    press `;`     ->    shows nothing
   

2.    press `h`     ->    shows `:h`
   

so the command works but it just doesn't show on the first key press / or initial ;.

Does this only happen to me?

And it is the same with / search commands also.
If I press / nothing shows but I know that the command works under the hood.

[NTBBloodbath]

Hey, that's because Doom sets the silent option for the mappings, maybe we should make it optional :)

[molleweide]

Aaha that makes sense then

[molleweide]

Can i pass a fourth arg table with silent=false or does this have to be modified inside of doom core first?

[NTBBloodbath]

Hi, you will need to do it yourself at the moment, but I'll implement it in a very near future!

You will need to modify this lua/doom/core/functions/init.lua block a bit, basically removing the opts variable and doing a conditional assignment. So it should look like this:

elseif scope == 'mappings' then
	for _, map in ipairs(settings_tbl) do
		-- scope, lhs, rhs, options
		utils.map(map[1], map[2], map[3], map[4] and map[4] or {})
	end

and you will be able to pass your custom options for your mappings (see :h map-arguments and :h nvim_set_keymap() to know the valid options).

Side note: the map() function automatically sets the noremap argument and that's gonna be changed in the future too if needed.

[molleweide]

Hey, that's because Doom sets the silent option for the mappings, maybe we should make it optional :)

yes I think that would be a great idea. another example that doesn't work also is for using the 'map' keyword
instead of 'n' or 'v'.

Hi, you will need to do it yourself at the moment, but I'll implement it in a very near future!

You will need to modify this lua/doom/core/functions/init.lua block a bit, basically removing the opts variable and doing a conditional assignment. So it should look like this:

elseif scope == 'mappings' then
	for _, map in ipairs(settings_tbl) do
		-- scope, lhs, rhs, options
		utils.map(map[1], map[2], map[3], map[4] and map[4] or {})
	end

and you will be able to pass your custom options for your mappings (see :h map-arguments and :h nvim_set_keymap() to know the valid options).

Side note: the map() function automatically sets the noremap argument and that's gonna be changed in the future too if needed.

Thanks man for the direction! I will try to get it to work. brb!!

[NTBBloodbath]

Hey, that's because Doom sets the silent option for the mappings, maybe we should make it optional :)

yes I think that would be a great idea. another example that doesn't work also is for using the 'map' keyword
instead of 'n' or 'v'.

Do you mean nmap and these stuff? If that's the case then it's not possible, the Lua api for the keybindings only takes the keybind mode and if the noremap option was not passed then the mapping is automatically set to <mode>map. You can take a look at :h nvim_set_keymap().

Hi, you will need to do it yourself at the moment, but I'll implement it in a very near future!
You will need to modify this lua/doom/core/functions/init.lua block a bit, basically removing the opts variable and doing a conditional assignment. So it should look like this:

elseif scope == 'mappings' then
	for _, map in ipairs(settings_tbl) do
		-- scope, lhs, rhs, options
		utils.map(map[1], map[2], map[3], map[4] and map[4] or {})
	end

and you will be able to pass your custom options for your mappings (see :h map-arguments and :h nvim_set_keymap() to know the valid options).
Side note: the map() function automatically sets the noremap argument and that's gonna be changed in the future too if needed.

Thanks man for the direction! I will try to get it to work. brb!!

No problem, glad to help. Hope it works for you! :)

[molleweide]

Allright cool I am going to read the nvim api docs and make sure i understand.

[molleweide]

hey are you thinking about including nvim-mapper into doom for key mappings?

[NTBBloodbath]

@molleweide Hey, yeah and it's already implemented in the development branch :)

[molleweide]

very nice! i have to check out the develp branch.

[NTBBloodbath]

I'll be closing this because the feature was already implemented in the development version so I can have a better track of the next version milestone 🙂