Skip to content

Conversation

@undead404
Copy link
Contributor

Recreated #59

@undead404 undead404 added the enhancement New feature or request label Aug 30, 2024
@undead404 undead404 self-assigned this Aug 30, 2024
}
}

@import "./search.scss";
Copy link
Member

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

Copy link
Member

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)
Copy link
Member

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);
Copy link
Member

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. любиш ти всі функції підряд писати без рядків між ними :)

@undead404
Copy link
Contributor Author

Більш вдумливо пройдуся згодом

@undead404 undead404 requested a review from a team September 2, 2024 05:50
@undead404
Copy link
Contributor Author

@alinkedd воно робе

@alinkedd
Copy link
Member

alinkedd commented Sep 3, 2024

@undead404 тоді закрий, будь ласка, коменти із текстом 'proposal is at 86', бо у мене немає прав для цього

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants