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.

Sign up for GitHub

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

Jump to bottom

UX/UI : Modification filtres sur la liste des salariés et PASS IAE et sur la liste des fiches salarié ASP #4083

Merged
merged 2 commits into from
Jun 6, 2024
Merged

UX/UI : Modification filtres sur la liste des salariés et PASS IAE et sur la liste des fiches salarié ASP #4083

merged 2 commits into from
Jun 6, 2024

Conversation

hellodeloo
Copy link
Contributor

@hellodeloo hellodeloo commented May 14, 2024 •
edited
Loading

🤔 Pourquoi ?

Évolutions UI

🍰 Comment ?

Maj des templates, ajout de js

💻 Captures d'écran

Avant
capture 2024-05-15 à 10 34 38

capture 2024-05-15 à 15 16 40

Apres
capture 2024-05-15 à 10 34 06

capture 2024-05-15 à 15 16 34

@hellodeloo hellodeloo marked this pull request as draft May 14, 2024 16:02
@notion-workspace Notion Workspace
Copy link

🎨 Refonte UX/UI - Adapter les filtres pour la liste des PASS IAE

@hellodeloo hellodeloo force-pushed the deloo/update-approvals-list-filters-ui branch from 8aa8a8f to cbbb752 Compare May 15, 2024 08:30
@hellodeloo hellodeloo changed the title wip: update approvals list filters ui Refonte UX/UI - Modification des filtres de PASS IAE May 15, 2024
@hellodeloo hellodeloo marked this pull request as ready for review May 15, 2024 08:36
@hellodeloo hellodeloo added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label May 15, 2024
@hellodeloo hellodeloo changed the title Refonte UX/UI - Modification des filtres de PASS IAE UX/UI : Modification des filtres de PASS IAE May 15, 2024
@github-actions GitHub Actions
Copy link

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@hellodeloo hellodeloo force-pushed the deloo/update-approvals-list-filters-ui branch from cbbb752 to fa9d8c2 Compare May 15, 2024 09:01
@hellodeloo hellodeloo changed the title UX/UI : Modification des filtres de PASS IAE UX/UI : Modification des filtres de PASS IAE et Salarié ASP May 15, 2024
@hellodeloo hellodeloo changed the title UX/UI : Modification des filtres de PASS IAE et Salarié ASP UX/UI : Modification filtres sur la listes des salariés et PASS IAE et sur la liste des fiches salarié ASP May 15, 2024
@hellodeloo hellodeloo changed the title UX/UI : Modification filtres sur la listes des salariés et PASS IAE et sur la liste des fiches salarié ASP UX/UI : Modification filtres sur la liste des salariés et PASS IAE et sur la liste des fiches salarié ASP May 15, 2024
@hellodeloo hellodeloo self-assigned this May 15, 2024
@hellodeloo hellodeloo force-pushed the deloo/update-approvals-list-filters-ui branch 2 times, most recently from 641cbb5 to 08d5faf Compare May 16, 2024 13:20
francoisfreitag
francoisfreitag reviewed May 22, 2024
View reviewed changes
Copy link
Contributor

@francoisfreitag francoisfreitag left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Joli !

J’ai réécrit le JS pour le simplifier, virer un console.log() qui trainait et le faire bien fonctionner avec HTMX.

STR 1:

  1. Se connecter en tant que l’EI
  2. http://localhost:8000/employee_record/list
  3. Appuyer sur « Effacer tout »

Le fait d’enlever les filtres fait qu’on ne voit plus aucune fiche salarié. On pourrait rediriger le bouton « Effacer tout » vers la vue nouvelles fiches salariées, qui est la vue par défaut. Ou alors, avoir toutes les FS, ou encore ne pas afficher le bouton « Effacer tout ».

@francoisfreitag
Copy link
Contributor

J’attire ton attention sur le fait que le nouveau JS affiche les filtres actifs tout le temps (pas seulement après un bs.dropdown.hide).
Autrement, au chargement de la page (rechargement avec F5), les filtres actifs ne sont plus mis en valeur.

@hellodeloo
Copy link
Contributor Author

J’attire ton attention sur le fait que le nouveau JS affiche les filtres actifs tout le temps (pas seulement après un bs.dropdown.hide). Autrement, au chargement de la page (rechargement avec F5), les filtres actifs ne sont plus mis en valeur.

Oui, merci pour l'amélioration 👍

@hellodeloo hellodeloo force-pushed the deloo/update-approvals-list-filters-ui branch from 8f250be to 3cedf76 Compare May 24, 2024 13:43
@hellodeloo
Copy link
Contributor Author

hellodeloo commented May 24, 2024 •
edited
Loading

... ou encore ne pas afficher le bouton « Effacer tout ».

@francoisfreitag J'ai supprimé le bouton "Effacer tout" et mis du propre dans les commits.
Par contre, il reste 3 tests qui plantent, c'est apparemment lié au changement de la valeur du radio expiry, de 0 a ""

@hellodeloo hellodeloo force-pushed the deloo/update-approvals-list-filters-ui branch from 3cedf76 to f119e19 Compare May 24, 2024 14:20
@francoisfreitag francoisfreitag force-pushed the deloo/update-approvals-list-filters-ui branch from f119e19 to d8a8a22 Compare May 28, 2024 09:53
@hellodeloo hellodeloo force-pushed the deloo/update-approvals-list-filters-ui branch from d8a8a22 to 25d2485 Compare May 28, 2024 13:20
@francoisfreitag
Copy link
Contributor

Effectivement, c’est réglé ! :)

@hellodeloo
Copy link
Contributor Author

Effectivement, c’est réglé ! :)

Encore un petit "aprrove" et on est bon

francoisfreitag
francoisfreitag reviewed Jun 3, 2024
View reviewed changes
Copy link
Contributor

@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.

STR :

  1. Connecté en tant que EI
  2. http://localhost:8000/employee_record/list?status=SENT&order=-hiring_start_at
  3. Changer le tri pour « la plus ancienne d’abord »
    La page se recharge, le tri est appliqué, mais l’UI affiche toujours « la plus récente d’abord ».

itou/templates/approvals/includes/approvals_filters/users.html Outdated Show resolved Hide resolved
itou/templates/approvals/includes/approvals_filters/reset.html Outdated Show resolved Hide resolved
@hellodeloo hellodeloo force-pushed the deloo/update-approvals-list-filters-ui branch from 25d2485 to 824da9d Compare June 4, 2024 08:58
@hellodeloo
Copy link
Contributor Author 

   La page se recharge, le tri est appliqué, mais l’UI affiche toujours « la plus récente d’abord ».

Ah oui effectivement, je n'avais pas capté.
Par contre, je vais te laisser gérer et te laisser la main sur la fin de la PR car je ne vais pas m'en sortir.

@hellodeloo hellodeloo force-pushed the deloo/update-approvals-list-filters-ui branch from 824da9d to 4f2b107 Compare June 4, 2024 09:05
@francoisfreitag francoisfreitag force-pushed the deloo/update-approvals-list-filters-ui branch from 4f2b107 to 754289f Compare June 5, 2024 10:35
@francoisfreitag francoisfreitag force-pushed the deloo/update-approvals-list-filters-ui branch from 754289f to 96a45c9 Compare June 5, 2024 10:39
@hellodeloo
feat: update approvals list filters ui
d8bf9cd 
The approval expiry field value for ALL changed from "0" to "". That
allows the JS for `has-selected-item` to identify the “empty” option. It
also allowed simplifying the form.
@hellodeloo
feat: update employee record list filters ui
ca86c6e
@hellodeloo hellodeloo force-pushed the deloo/update-approvals-list-filters-ui branch from 96a45c9 to ca86c6e Compare June 6, 2024 08:08
@hellodeloo hellodeloo added this pull request to the merge queue Jun 6, 2024
Merged via the queue into master with commit 57ac00c Jun 6, 2024
11 checks passed
@hellodeloo hellodeloo deleted the deloo/update-approvals-list-filters-ui branch June 6, 2024 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@francoisfreitag francoisfreitag francoisfreitag approved these changes

Assignees

@francoisfreitag francoisfreitag

@hellodeloo hellodeloo

Labels
1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC modifié
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hellodeloo @francoisfreitag