Skip to content

feat(table): enhance table - #340

Open
maximallain wants to merge 4 commits into
codegouvfr:mainfrom
maximallain:ma-enhance-table
Open

feat(table): enhance table#340
maximallain wants to merge 4 commits into
codegouvfr:mainfrom
maximallain:ma-enhance-table

Conversation

@maximallain

@maximallain maximallain commented Nov 7, 2024

Copy link
Copy Markdown
Contributor

Mise à jour du composant Table :

  • retrait de la variable "couleur" (breaking)
  • ajout de la taille (hauteur des lignes)
  • ajout d'une colonne d'en-tête
  • ajout de l'alignement au sein des cellules

Il y a un sujet sur la sélection de la ligne qui implique beaucoup d'autres choses. A voir si on se lance là-dedans ?

@maximallain maximallain changed the title Ma enhance table feat(table): enhance table Nov 13, 2024
@maximallain
maximallain marked this pull request as ready for review November 13, 2024 11:05
@Falinor

Falinor commented May 7, 2025

Copy link
Copy Markdown

Any update on this ?

@kevbarns

Copy link
Copy Markdown
Collaborator

@maximallain Cette PR n'a reçu aucune revue de mainteneur depuis son ouverture en novembre 2024, et votre question de départ (« Il y a un sujet sur la sélection de la ligne […] A voir si on se lance là-dedans ? ») n'a jamais eu de réponse. @Falinor relançait déjà en mai 2025.

Entre-temps le contexte a changé et il faut le poser avant d'aller plus loin :

  • feat(Table): Implement sortable columns #510, ouverte le 27/08/2026 par @Mintoo200, réécrit le même src/Table.tsx pour ajouter le tri de colonnes, et embarque un bump @gouvfr/dsfr 1.14.2 → 1.15.2. Les deux PR entrent en conflit direct.
  • Update Table component #268 suit la refonte complète du composant Table, et @iNeoO y proposait en juillet 2025 un portage déjà réalisé.
  • Votre PR contient un breaking change assumé (retrait de la variante « couleur ») qui n'a jamais été arbitré.

@garronej @revolunet @enguerranws il y a maintenant trois pistes concurrentes sur le même composant. Pouvez-vous dire laquelle sert de base ? Sans cet arbitrage, ces PR vont continuer à se périmer les unes contre les autres.

@maximallain si la piste retenue est la vôtre, dites-le et je regarde le code en détail dans la foulée. Si c'est une autre, autant fermer celle-ci proprement plutôt que de la laisser ouverte deux ans de plus.

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.

3 participants