LB (Ben Johnston)
LB (Ben Johnston)
OK I have rebased on `main` - just a reminder for future, all PRs must be off `main` not release branches, I may need to raise a new PR. I...
Merged in via https://github.com/wagtail/wagtail/pull/13189 / https://github.com/wagtail/wagtail/commit/aaac0045c688ff65980895b3dc8b30557cb95eaa
Thanks @hunzlahmalik as per the comments on your PR https://github.com/wagtail/wagtail/pull/13397 For now, this needs some further research and discussion, feel free to pursue this discussion on this issue.
@willbarton - any thoughts or insights for this enhancement request? raised from #9267
https://github.com/wagtail/wagtail/pull/12846 has been raised as a rebase of this PR with some initial unit tests, hence closing this PR now.
@RealOrangeOne - I will merge this in. Thank you @vsalvino
Fixed via https://github.com/wagtail/wagtail/pull/12365 also, but this issue was not mentioned. I did a search but must not have searched for the right words.
No problems. I'll leave it to Matt.
@berycz could you possibly update / add exact steps to reproduce? How were you searching these strings? Were you using the util `wagtail.search.utils.parse_query_string` ? This may be another similar case...
Here's a related issue with the way the userbar loads the page for the 'show in explorer' button. Not the same problem but worth reviewing in the context of changes...