-
Notifications
You must be signed in to change notification settings - Fork 46
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
Sbs tache moteur recherche communes #29
base: main
Are you sure you want to change the base?
Sbs tache moteur recherche communes #29
Conversation
webapp/components/CommuneFilter.tsx
Outdated
|
||
export default function CommuneFilter() | ||
{ | ||
const [FilterString,SetFilterString]=useState('') |
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.
par convention, je vois plutôt du "camel case" sur ces variables, donc filteringString
et setFilteringString
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.
Ok, si c'est la convention je vais essayer de m'y tenir (même si je comprends pas l'interêt de ne pas mettre de maj au 1er mot d'une variable).
J'ai push la mise à jour
Hello Stéphane, Merci pour ta PR ! Ok, ça montre que ça marche, c'est un bon proof of concept :) Le rendu UI n'est pas top par contre... C'est chez moi le problème ? Le Combobox est tout blanc, et l'alignement des suggestions à tendance à bouger lors de la frappe... ![]() Tu as essayé de passer par un rewrite ? Cela pourrait éviter de faire une route API spécifique. Je pense qu'on va avoir besoin d'un code formater comme Prettier... L'utilisation de @headlessui/react est qqch qui doit être discuté collectivement et de façon séparée. |
Concernant la mise en page, j'ai absolument rien fait en dehors du fond blanc. Vu que je connais rien en CSS, je ne passe pas de temps là dessus tant qu'il y a pas une spec qui indique comment on veut que ce soit rendu
Non je n'ai pas essayé, car je me dis que pouvoir intercepter la requête et en faire des choses à la volée (genre créer un geojson et le mettre dans une PMTile) ou avoir une base locale de communes pourra se gérer de façon transparente avec une route. Quel est le souci de faire une "route API spécifique" ?
:) j'ai horreur des { en fin de lignes, ainsi que des ; en fin de ligne qui ne servent à rien.
Tu as cité ça, donc j'ai essayé. |
Bon, La PR est en conflit, mais je ne peux pas résoudre tant qu'on n'a pas introduit shadcn dans la main branch, afin que je récupère le setup qui va bien. Il me faudra à minima les composants suivants : |
https://noco.services.dataforgood.fr/dashboard/#/nc/pk1vq1pm8frc5lm/ms9uz8er4jpow7j/Kanban%20dev?rowId=76
Creation d'une api route pour requêter les communes via l'API IGN.
Ajout Tailwind.
Minimal CSS pour la drop down des communes.