Skip to content
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

[Dashboard] Bloc zones réglementaires + Création du composant accordéon #1711

Merged
merged 9 commits into from
Sep 25, 2024

Conversation

claire2212
Copy link
Collaborator

@claire2212 claire2212 commented Sep 19, 2024

  • Création du composant Accordion

  • Gestion de la fermeture des autres accordéons

  • Pour le moment j'ai mis un click sur toute la longueur du titre s'il n'y a pas d'autre bouton dans le titre, s'il y en a il faut cliquer sur le chevron pour ouvrir/ fermer l'accordéon

  • Ajout du bloc "Zones réglementaires"

  • Resolve [Brief et Tableau de Bord] Création du bloc zones réglementaires #1714

@claire2212 claire2212 self-assigned this Sep 19, 2024
@claire2212 claire2212 changed the base branch from main to claire/create-dashboard-page September 19, 2024 14:25
@claire2212 claire2212 force-pushed the claire/create-dashboard-accordion branch from ac379fb to 1d22b1f Compare September 20, 2024 14:08
@claire2212 claire2212 changed the title [Dashboard] Création du composant accordéon [Dashboard] Bloc zones réglementaires + Création du composant accordéon Sep 20, 2024
@claire2212 claire2212 mentioned this pull request Sep 20, 2024
25 tasks
Base automatically changed from claire/create-dashboard-page to main September 23, 2024 09:26

const AccordionContainer = styled.div<{ $withCursor: boolean }>`
box-shadow: 0px 3px 6px #70778540;
cursor: ${({ $withCursor }) => ($withCursor ? 'pointer' : 'default')};
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

question: pourquoi mettre tout le container avec le pointer ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

c'était une demande de Xavier de pouvoir cliquer sur tout le container pour ouvrir l'accordéon. J'ai fait sur tout le container quand il n'y a pas d'autre icône et quand il y a une icône (cf bloc signalement) , je l'ai mis seulement sur le chevron.

Copy link
Collaborator

@maximeperraultdev maximeperraultdev Sep 25, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggestion: comme ce n'est pas un bouton mais une div cliquable. Ce serait bien d'avoir un aria-control qui réfère ce qu'il se ferme ou pas ainsi qu'un aria-expanded

font-weight: 500;
`

const HeaderSeparator = styled.div`
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggestion: si tu met la border-bottom sur la div du container ca fait le meme taf non ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Le header a un padding alors que le trait fait toute la longueur c'est pour ça

@claire2212 claire2212 force-pushed the claire/create-dashboard-accordion branch 2 times, most recently from 108e8a2 to fd48c76 Compare September 24, 2024 14:06
@claire2212 claire2212 force-pushed the claire/create-dashboard-accordion branch from 8d08dbd to 4cc4966 Compare September 24, 2024 14:43
@claire2212 claire2212 merged commit 68c76e3 into main Sep 25, 2024
21 checks passed
@claire2212 claire2212 deleted the claire/create-dashboard-accordion branch September 25, 2024 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Brief et Tableau de Bord] Création du bloc zones réglementaires
2 participants