-
Notifications
You must be signed in to change notification settings - Fork 36
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
feat(atomic): add atomic-result-html component #2014
Conversation
Thanks for your contribution @olamothe !
|
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.
looks good! although I'm curious what's the use case for this exactly
People that want to display field properties from their result which contains HTML. |
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.
Looks just great! 💎 However, I requested a couple of changes 🙏
...s/atomic/src/components/result-template-components/atomic-result-html/atomic-result-html.tsx
Show resolved
Hide resolved
...s/atomic/src/components/result-template-components/atomic-result-html/atomic-result-html.tsx
Outdated
Show resolved
Hide resolved
Pull Request Report PR Title ✅ Title follows the conventional commit spec. Bundle Size
|
Add two components to output HTML (atomic-html, atomic-result-html), especially useful for results.
Use
DOMPurify
to optionally sanitize the HTML.Also fixed a bug in Headless with highlight function: When there's no highlights (ie: an empty array is passed to the function), it would return the un-escaped value.
https://coveord.atlassian.net/browse/KIT-1122