-
Notifications
You must be signed in to change notification settings - Fork 7
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
Fix Filter Options Highlighting #1871
Conversation
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.
can you add some screenshots/recording of what it looks like? also, can you provide a little more detail in the PR description?
Added more details and a screenshot. |
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.
nit: the PR description styling seems a little off because the editor accepts html. you'll need to put the strong tag in a code style (e.g. <strong>
) or escape it (e.g. <strong>)
besides that, lgtm!
Previously, when we moved from
innerHTML
totextContent
, it broke the filter options highlighting since the<strong>
tag is no longer applied properly. This PR fixes the issue while still avoid insecure document method.J=TECHOPS-11319
TEST=auto, manual
Ran
npm run test
. Serve answers-search-ui locally and checked out hh theme. Made sure the highlighting are working.https://drive.google.com/file/d/1lBVOA759BPwsqmENIv6kFe2FasR25EKO/view?usp=sharing