Update AppLink to enable native mouse navigation #7498
Conversation
Add a `href` attribute to enable native mouse navigation (middle click, ctrl+click, etc.)
|
Looks like this broke manual searching. Links as such no longer send a background request, instead the link is followed. This link is from the schedule page btw. http://192.168.1.21:8081/home/searchEpisode?indexername=tvdb&seriesid=351134&season=2&episode=3 |
| to: matchingVueRoute.fullPath | ||
| to: matchingVueRoute.fullPath, | ||
| // Add a `href` attribute to enable native mouse navigation (middle click, ctrl+click, etc.) | ||
| href: new URL(matchingVueRoute.fullPath, base).href |
There was a problem hiding this comment.
This is likely needed to be moved down to the below block, if it's added here it'll need to have some kind of check since not all AppLink components should be openable in another tab.
There was a problem hiding this comment.
Well currently it only affects converted pages' links to other converted pages (like config-general to config). It's possible there's an edge case I missed.
|
It was already reported before this got merged #7462 so I don't think that this change caused that. Haven't had time to check though. Edit: Try running Edit 2: Fixed with #7512 |
Add a
hrefattribute to enable native mouse navigation (middle click, ctrl+click, etc.)(This is for when AppLink is a RouterLink, it's annoying that you can't open it in a new tab)