-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HTML Search: Fix removal of unwanted anchor content from search results #12321
Conversation
14d95ff
to
04d4b4e
Compare
@@ -178,7 +178,7 @@ const Search = { | |||
|
|||
htmlToText: (htmlString, anchor) => { | |||
const htmlElement = new DOMParser().parseFromString(htmlString, 'text/html'); | |||
for (const removalQuery of [".headerlinks", "script", "style"]) { | |||
for (const removalQuery of [".headerlink", "script", "style"]) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was (correctly) spelled .headerlink
before.
@jayaddison @picnixz Can you take a look? Looks like an accidental regression from some previous work. |
04d4b4e
to
8ca1b67
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, looks good to me - thanks @wlach!
Thanks Will! Out of interest, would you be interested in triage permissions, to hopefully make your work on the search features easier? A |
Yep, that'd be great! Thanks. |
Subject: Fix removal of unwanted anchor content from search results
Feature or Bugfix
Purpose
This was accidentally broken in bf0bec3 (#12057), this just fixes it (and updates the existing test so it hopefully doesn't happen again).
Detail
N/A
Relates
Closes #12320