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

Reorganisation des boutons sur l'interface instructeur #9457

Merged
merged 2 commits into from
Sep 11, 2023

Conversation

lisa-durand
Copy link
Contributor

Avec la MAJ du DSFR le switch pour "remonter les notifications" avait un alignement pas très heureux. J'ai donc préféré repartir sur une checkbox plus classique qui prend moins de place. Et j'en ai profité pour créer un ticket pour réfléchir à une refonte plus global du tri pour les instructeurs #9456

J'ai également profité de cette PR pour simplifier l'interface en déplacant le bouton de "personnalisation du tableau" dans le header du tableau - afin que ça soit plus cohérent et moins au mm niveau que les autres actions.

APRES
Capture d’écran 2023-09-05 à 17 36 27

AVANT
Capture d’écran 2023-09-05 à 17 50 14

@lisa-durand lisa-durand force-pushed the create-export-page-ldu branch from ff25b43 to 5fe5320 Compare September 5, 2023 16:07
@colinux
Copy link
Member

colinux commented Sep 7, 2023

J'ai toujours trouvé bizarre d'avoir un toggle ou autre pour cette action (au point où je ne l'avais pas comprise au début, je pensais que ça persistait un peu comme un filtre qu'on pouvait désactiver ensuite). Est-ce que ce serait pas plus clair avec un simple bouton sur l'ensemble du texte ? (mais ça prendrait un peu plus de place)

@lisa-durand
Copy link
Contributor Author

J'ai toujours trouvé bizarre d'avoir un toggle ou autre pour cette action (au point où je ne l'avais pas comprise au début, je pensais que ça persistait un peu comme un filtre qu'on pouvait désactiver ensuite). Est-ce que ce serait pas plus clair avec un simple bouton sur l'ensemble du texte ? (mais ça prendrait un peu plus de place)

Je suis d'accord, j'ai l'impression que pour ce genre de tri sur les colonnes du tableau on utilise le header du tableau mais c'est pas non plus hyoer clair d'ou la création de ce toggle j'ai l'impression. J'ai créé un ticket pour qu'on puisse y reflechir -> #9456

@lisa-durand lisa-durand added this pull request to the merge queue Sep 11, 2023
Merged via the queue into main with commit 2f5310a Sep 11, 2023
@lisa-durand lisa-durand deleted the create-export-page-ldu branch September 11, 2023 08:11
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