entrouvert
/
publik-base-theme
15
1
Fork 1
Code Issues Pull Requests 10 Releases Wiki Activity

Permettre de supprimer la première étoile sur les listes affichées sous forme d'étoiles (vote, enquête de satisfaction) (#88084) #533

Open
nroche wants to merge 2 commits from wip/88084-unselect-evaluation-stars into main
pull from: wip/88084-unselect-evaluation-stars
merge into: entrouvert:main
Conversation 2 Commits 2 Files Changed 2 +18 -4
nroche commented 2024-03-12 17:52:42 +01:00
Owner
Copy Link

Je fournis juste ce patch à titre indicatif.
Parce que ça me semble beaucoup compliquer le code pour le résultat attendu.

Remarque : le patch ne traite que les champs affichés sous forme de liste.
J'imagine que pour les listes affichées avec des radio boutons ce serait entendable de ne pas pouvoir tout désélectionner.

Je fournis juste ce patch à titre indicatif. Parce que ça me semble beaucoup compliquer le code pour le résultat attendu. Remarque : le patch ne traite que les champs affichés sous forme de liste. J'imagine que pour les listes affichées avec des radio boutons ce serait entendable de ne pas pouvoir tout désélectionner.
nroche added 1 commit 2024-03-12 17:52:43 +01:00
gitea/publik-base-theme/pipeline/head This commit looks good Details
c664b256c6 templates: allow to unselect evaluation stars (#88084)
tjund commented 2024-03-19 12:11:14 +01:00
Owner
Copy Link

Je regarde.
Mais c'est pas vraiment recommandé d'ajouter une exception sur la première étoile pour supprimer l'évaluation :

  • pas clair pour l'usager = fonctionnalité cachée
  • Pas accessible du tout

Je pense qu'une des piste serait d'ajouter un bouton "effacer l'évaluation" qui s'affiche uniquement qi une évaluation existe et que le champ est optionnel.

Je regarde. Mais c'est pas vraiment recommandé d'ajouter une exception sur la première étoile pour supprimer l'évaluation : * pas clair pour l'usager = fonctionnalité cachée * Pas accessible du tout Je pense qu'une des piste serait d'ajouter un bouton "effacer l'évaluation" qui s'affiche uniquement qi une évaluation existe et que le champ est optionnel.
tjund commented 2024-03-19 15:01:44 +01:00
Owner
Copy Link

https://dev.entrouvert.org/issues/87668 est fermé.
Je pousse une proposition pour archivage au cas où.

https://dev.entrouvert.org/issues/87668 est fermé. Je pousse une proposition pour archivage au cas où.
tjund added 1 commit 2024-03-19 15:09:31 +01:00
gitea/publik-base-theme/pipeline/head This commit looks good Details
037b100b03 use "remove" link to clear evaluation
csechet approved these changes 2024-03-21 17:40:18 +01:00
All checks were successful
gitea/publik-base-theme/pipeline/head This commit looks good
Details
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b wip/88084-unselect-evaluation-stars main
git pull origin wip/88084-unselect-evaluation-stars

Step 2:

Merge the changes and update on Gitea.
git checkout main
git merge --no-ff wip/88084-unselect-evaluation-stars
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
csechet
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/publik-base-theme#533
No description provided.
Delete Branch "wip/88084-unselect-evaluation-stars"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?