-
Notifications
You must be signed in to change notification settings - Fork 4
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
Multi sélection des énergies dans les filtres #901
Conversation
…arte et la liste des réseaux
34b6993
to
5f9ebb5
Compare
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.
src/components/ui/Popover.tsx
Outdated
'use client'; | ||
|
||
// Based on https://ui.shadcn.com/docs/components/popover | ||
// WIth the added functionnality of the width of the content to be at least the width of the trigger |
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.
On utilise @radix-ui popover mais inspiré de shadcdn ? 🤔
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.
En fait shadcn est une bibilothèque de composant à copier coller dans ton code
Shadcn est basé sur
- tailwind
- radix-ui
J'ai transformé tailwind en styled-components
radix-ui est une librarie headless de composants sans style qui implémentent les comportements les plus courant
@martinratinaud J'ai retesté, mais j'ai toujours le même problème où il faut double cliquer pour ouvrir le popover... 😬 Je suis sous chrome Sur firefox ça fonctionne, même s'il y a un petit artefact graphique... où on voit le menu sans contenu |
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.
C'est bon 👍
Voir https://trello.com/c/ukkxo9tQ/1300-remplacer-le-filtre-energie-majoritaire-par-un-filtre-energies-mobilis%C3%A9es-avec-possibilit%C3%A9-de-s%C3%A9lectionner-plusieurs-%C3%A9nergies-da