misc: dynamically update first option to alternative label (#60217) #907

Merged
fpeters merged 1 commits from wip/60217-redo-first-element-empty-label into main 2024-01-12 14:56:27 +01:00
Owner
No description provided.
fpeters added 1 commit 2023-12-09 19:18:11 +01:00
fpeters reviewed 2023-12-11 17:56:56 +01:00
@ -2200,4 +2197,0 @@
if not seen and isinstance(widget, SingleSelectWidget):
widget.full_options = widget.full_options[:]
widget.full_options[0] = list(widget.full_options[0])
widget.full_options[0][1] = self.first_element_empty_label
Author
Owner

Plus de modification via le python, gestion via le js.

Plus de modification via le python, gestion via le js.
@ -83,0 +88,4 @@
if (values.every(v => (v === ""))) { // all empty
$widgets.find('select').first().find('option[value=""]').text($(this).attr('data-first-element-empty-label'));
} else {
$widgets.find('select').first().find('option[value=""]').text('---');
Author
Owner

Le comportement maintenant : si le choix vide est choisi dans tous les select au-delà du premier, on modifie le libellé de l'option vide du premier select (pour par exemple afficher "Tout le monde"), sinon c'est --- pour bien marquer que ça va "vider" l'option. Ça permet à la fois d'avoir l'indication sur le comportement particulier quand tout est vide (ex ici "Tout le monde", mais on l'utilise aussi pour "API ouverte") et d'être clair sur l'effet du choix vide quand les autres select contiennent des valeurs.

Le comportement maintenant : si le choix vide est choisi dans tous les select au-delà du premier, on modifie le libellé de l'option vide du premier select (pour par exemple afficher "Tout le monde"), sinon c'est --- pour bien marquer que ça va "vider" l'option. Ça permet à la fois d'avoir l'indication sur le comportement particulier quand tout est vide (ex ici "Tout le monde", mais on l'utilise aussi pour "API ouverte") et d'être clair sur l'effet du choix vide quand les autres select contiennent des valeurs.
fpeters changed title from WIP: misc: dynamically update first option to alternative label (#60217) to misc: dynamically update first option to alternative label (#60217) 2023-12-11 17:57:02 +01:00
lguerin approved these changes 2024-01-08 12:18:48 +01:00
fpeters merged commit 33cd5ae22d into main 2024-01-12 14:56:27 +01:00
fpeters deleted branch wip/60217-redo-first-element-empty-label 2024-01-12 14:56:27 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 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/wcs#907
No description provided.