Editing Template talk:Navbar (section)

== Bug in move option ==

Further to the section above, the move option does not work correctly when the template name includes a space - using {{nowrap|1={{tlx|view|Example template|m}}}} produces {{view|Example template|m}} note that the second link displays as "template<span title="Move this template">move</span>" and the link goes to https://en.wikipedia.org/w/index.php?title=Special:Movepage&amp;target=Template:Example which is not what is intended - the link should display as "<span title="Move this template">move</span>" and the link should be https://en.wikipedia.org/w/index.php?title=Special:Movepage&amp;target=Template:Example_template

It makes no difference if the space is replaced by an underscore, nor if the first parameter is named - that is, {{tlx|view|Example template|m}}, {{tlx|view|Example_template|m}}, {{tlx|view|2=template=Example template|3=m}} and {{tlx|view|2=template=Example_template|3=m}} all yield identical results.

However, if the space is replaced by a plus sign, as in {{tlx|view|Example+template|m}} the move link is now correct, but the view link is broken: {{view|Example+template|m}} It seems that there is no way to have both the view and move links correct. --[[User:Redrose64|<span style="color:#a80000; background:#ffeeee; text-decoration:inherit">Red</span>rose64]] &#x1f339; ([[User talk:Redrose64|talk]]) 12:20, 7 January 2023 (UTC)

This bug is presumably in [[Module:Navbar]]. Curiously, [[Module talk:Navbar]] redirects here, but [[Template talk:View]] redirects to [[Template talk:VTE]]. --[[User:Redrose64|<span style="color:#a80000; background:#ffeeee; text-decoration:inherit">Red</span>rose64]] &#x1f339; ([[User talk:Redrose64|talk]]) 12:25, 7 January 2023 (UTC)
:Same with {{tl|v}}:
:{{tlx|v|Example template|m}}}} produces {{v|Example template|m}}. -[[User:DePiep|DePiep]] ([[User talk:DePiep|talk]]) 12:29, 7 January 2023 (UTC)
::Yes, so I've set up some testcases at [[Template:Navbar/testcases#Demonstrating bug in move option]]. --[[User:Redrose64|<span style="color:#a80000; background:#ffeeee; text-decoration:inherit">Red</span>rose64]] &#x1f339; ([[User talk:Redrose64|talk]]) 13:20, 7 January 2023 (UTC)
:::So, for the time being remove from documentation, or add a heavy warning? [[User:DePiep|DePiep]] ([[User talk:DePiep|talk]]) 07:46, 8 January 2023 (UTC)
::::I highly doubt it's even in use, and the failure point is a small subset of potential template names, so I think "fix the issue" is preferable. That being said I, I haven't looked at it in too much detail but I suspect it has to do with how the module is creating the URLs for the links (much like how elink templates need to use {{t|urlencode}} to properly format URLs). [[User:Primefac|Primefac]] ([[User talk:Primefac|talk]]) 08:07, 8 January 2023 (UTC)
:::::I edited [[Module:Navbar/sandbox]] with what I suspect will fix the problem. I would want to think about the change much more, and test it, before updating the main module. One of my edit summaries says [[Module:Navbar/sandbox]] was the same as [[Module:Navbar]] before I started. That is not quite correct—the sandbox has <code>local cfg = mw.loadData('Module:Navbar/configuration/sandbox')</code> whereas the main module omits <code>/sandbox</code>. That should be fixed before updating. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 10:07, 8 January 2023 (UTC)