-
Notifications
You must be signed in to change notification settings - Fork 4
feat: Algolia search (recreated) #85
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
base: master
Are you sure you want to change the base?
Conversation
| } | ||
| } | ||
|
|
||
| @import "./search.scss"; |
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.
якщо ти скомпілюєш, то sass все одно розмістить search зверху, якщо не помиляюсь
p.s. також для звичайного css import має бути зверху (та ігнорується знизу) https://developer.mozilla.org/en-US/docs/Web/CSS/@import
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.
так шо, воно у бандлі розміщує зверху (попереду коду і тоді немає сенсу додавати у кінець) чи все-таки тут у місці імпорту?
|
|
||
| .ais-SearchBox-form::before { | ||
| background: rgba(0, 0, 0, 0) | ||
| url(data:image/svg+xml;utf8,%3Csvg%20xmlns%3D%22http%3A%2F%2Fwww.w3.org%2F2000%2Fsvg%22%20width%3D%2216%22%20height%3D%2216%22%20viewBox%3D%220%200%2024%2024%22%20fill%3D%22none%22%20stroke%3D%22%235a5e9a%22%20stroke-width%3D%222%22%20stroke-linecap%3D%22round%22%20stroke-linejoin%3D%22round%22%3E%3Ccircle%20cx%3D%2211%22%20cy%3D%2211%22%20r%3D%228%22%3E%3C%2Fcircle%3E%3Cline%20x1%3D%2221%22%20y1%3D%2221%22%20x2%3D%2216.65%22%20y2%3D%2216.65%22%3E%3C%2Fline%3E%3C%2Fsvg%3E) |
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.
<svg
xmlns="http://www.w3.org/2000/svg"
width="16"
height="16"
viewBox="0 0 24 24"
fill="none"
stroke="#5a5e9a"
stroke-width="2"
stroke-linecap="round"
stroke-linejoin="round"
>
<circle cx="11" cy="11" r="8"></circle>
<line x1="21" y1="21" x2="16.65" y2="16.65"></line>
</svg>можна задати url(svg) в mask, і тоді через background-color управляти його кольором, як тут https://developer.mozilla.org/en-US/docs/Web/CSS/mask-image#examples
mask вже достатньо підтримується https://caniuse.com/css-masks (хоча можна ще явно -webkit-mask задати додатково)
| }, [router.events, handleLoadingEnded]); | ||
| useEffect( | ||
| () => () => { | ||
| router.events.off('routeChangeComplete', handleLoadingEnded); |
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.
router.events працює на 13-му next? просто ось тут дискусія vercel/next.js#42016 або vercel/next.js#41934, шо ніби не працює
p.s. любиш ти всі функції підряд писати без рядків між ними :)
|
Більш вдумливо пройдуся згодом |
Search PR review refactoring proposal
|
@alinkedd воно робе |
|
@undead404 тоді закрий, будь ласка, коменти із текстом 'proposal is at 86', бо у мене немає прав для цього |
Recreated #59