Skip to content

UX/UI: Uniformisation des tableaux#4861

Merged
hellodeloo merged 7 commits intomasterfrom
deloo/standardization-of-table-formatting
Oct 10, 2024
Merged

UX/UI: Uniformisation des tableaux#4861
hellodeloo merged 7 commits intomasterfrom
deloo/standardization-of-table-formatting

Conversation

@hellodeloo
Copy link
Copy Markdown
Contributor

@hellodeloo hellodeloo commented Oct 1, 2024

🤔 Pourquoi ?

Évolutions de l'ui et harmonisation

  • Toujours responsive avec <div.table-responsive>
  • Toujours posséder un <caption> pour l'a11y
  • Toujours un thead>th.col=scope
  • Pas de titre "Action ou autre" dans la “colonne d’action”
  • Icone d'action principale ou icone menu liste des actions en .btn-link
  • Sans bordure autour mais avec bordure d’interligne
  • Avec un .table-hover sur les lignes
  • Lien dans les cellules en .btn-link

@notion-workspace
Copy link
Copy Markdown

@hellodeloo hellodeloo marked this pull request as draft October 1, 2024 15:34
@hellodeloo hellodeloo self-assigned this Oct 1, 2024
@hellodeloo hellodeloo changed the title Deloo/standardization of table formatting UX/UI: Uniformisation des tableaux Oct 1, 2024
@hellodeloo hellodeloo force-pushed the deloo/standardization-of-table-formatting branch 2 times, most recently from 32b06a9 to 639d6e0 Compare October 1, 2024 15:51
@hellodeloo hellodeloo added the modifié Modifié dans le changelog. label Oct 1, 2024
@hellodeloo hellodeloo force-pushed the deloo/standardization-of-table-formatting branch from 639d6e0 to 1c78d01 Compare October 3, 2024 15:22
@hellodeloo hellodeloo marked this pull request as ready for review October 3, 2024 15:33
@hellodeloo
Copy link
Copy Markdown
Contributor Author

Pour infos, j'ai laissé les commits séparés pour la relecture mais rebaserais avant de merger

Copy link
Copy Markdown
Member

@francoisfreitag francoisfreitag left a comment

Choose a reason for hiding this comment

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

J’ai vérifié la plupart des pages dans le navigateur, ça semble bien fonctionner 👍

Les changements sont de bric et de broc, il est difficile de vérifier tout ce qui a été fait. J’ai jeté un œil rapide, mais revue de moins bonne qualité.

<a class="btn-link" {% matomo_event "candidature" "exports" "export-prescripteur" %} aria-label="Télécharger cet export prescripteur" href="{% url 'apply:list_prescriptions_exports_download' %}">
<i class="ri-download-line me-1" aria-hidden="true"></i>
Télécharger
<a class="btn btn-sm btn-link btn-ico-only"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ça devient compliqué, on a un <a> qui doit s’afficher comme un btn-link 🤨 ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Oui, le nom de la classe n'est pas trés bien trouvé

@hellodeloo hellodeloo added this pull request to the merge queue Oct 10, 2024
Merged via the queue into master with commit 000b81b Oct 10, 2024
@hellodeloo hellodeloo deleted the deloo/standardization-of-table-formatting branch October 10, 2024 08:28
leo-naeka added a commit that referenced this pull request Oct 25, 2024
leo-naeka added a commit that referenced this pull request Oct 25, 2024
github-merge-queue bot pushed a commit that referenced this pull request Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants