feat(v4): UI refresh - #22
Conversation
- Update the UI of Docsearch button, modal and sidepanel to better relfect Typesense brand. - Update Typesense Logo
tharropoulos
left a comment
There was a problem hiding this comment.
two general things
- lint:css goes from 277 to 345 errors with this change (most are autofixable with --fix),
- 3 of the css files lost their trailing newline
|
Thanks for the detailed review @tharropoulos! I've addressed the comments and pushed the updates. Let me know if anything else needs adjustment. |
tharropoulos
left a comment
There was a problem hiding this comment.
thanks for the quick turnaround. one thing to settle before merge is the --fix converting media queries to range syntax (width <= 768px), which needs chrome 104 / safari 16.4 / firefox 102. between that and the prefix removals it's probably worth adding autoprefixer and browserslist to build-css.js so compat is explicit, postcss is already in the pipeline there
- Added `autoprefixer` and `@csstools/postcss-media-minmax`
|
I added Browserslist is explicitly set to Also updated workpace yarn to |
|
|
Change Summary
typesense-docsearch-csspackage while preserving the flexibility for users to customize their look and feel.For reference, here is the before:
And here is the after: https://new-typesense-docsearch-ui-demo-rea.vercel.app/
PR Checklist