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

Découpe la page détail des indicateurs avec des onglets #3521

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

mariheck
Copy link
Contributor

Ticket : https://www.notion.so/accelerateur-transition-ecologique-ademe/Refonte-Onglets-page-indicateur-commune-1606523d57d780539af7d93421cdde37?pvs=4

  • Onglets Données / Sous indicateurs / Actions des référentiels liées / Fiches des plans liées
  • L'onglet données contient le graphe, le tableau, et les différents champs de personnalisation (certains seront ensuite déplacés à la refonte du header)
  • Les sous-indicateurs ne suivent pas encore la maquette, la mise en forme utilisée en prod a été conservée
  • Pas de grosse modification sur l'onglet actions, simple déplacement de la liste des cartes
  • L'onglet fiches permet maintenant de rechercher, de trier, et de faire des actions groupées (réutilisation du composant fait pour la page toutes les fiches actions)

Copy link

@mariheck mariheck force-pushed the TET-4241/onglets-indicateurs branch from c4273ea to ac45a2d Compare January 13, 2025 14:43
@mariheck mariheck force-pushed the TET-4241/onglets-indicateurs branch from 5924d72 to d8032b1 Compare January 20, 2025 14:20
<Tab label={`${enfants.length} Sous indicateurs`}>
<SousIndicateurs definition={definition} enfantsIds={enfants} />
</Tab>
) : undefined}
Copy link
Contributor

Choose a reason for hiding this comment

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

OK étrange (par rapport au fonctionnement d'autres libs de composants UI), à l'occasion on pourra regarder pour changer ce comportement dans le composant Tabs.
Merci pour les explications !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants