gabrielcsapo/docusaurus-plugin-search-local

Generates wrong links with Docusaurus 3.4.0

Opened this issue · 3 comments

It is generating wrong links for example searching for "linter errors" and clicking the first result should go to https://docs.flathub.org/docs/for-app-authors/linter#linter-errors but the generated link goes to https://docs.flathub.org/docs/for-app-authors/linter/docs/for-app-authors/linter#linter-errors

Search index before/after, the h attribute looks wrong and should probably not contain the pathname?

CleanShot 2024-07-18 at 15 27 32@2x

This probably happens because the anchor now includes the pathname on Docusaruus side:

CleanShot 2024-07-18 at 15 31 45@2x

While it wasn't the case before:

CleanShot 2024-07-18 at 15 33 18@2x

Although I wonder why our anchors have changed this way (will investigate) I think the bug is in this plugin that should support anchor links containing a pathname, which remains a perfectly valid anchor.

We'll probably need to introduce these kind of links on purpose on the blog to solve bugs like this: facebook/docusaurus#10287

Going to revert the unexpected Docusaurus change for now, but this only buys you time to fix the problem in this plugin because it remains a bug, and we'll have anchor links with a pathname in the future.

facebook/docusaurus#10311

Search index before/after, the h attribute looks wrong and should probably not contain the pathname?

I tried to fix it with #83 but didn't seem to work and I'm not very familiar with docusaurus too.