manager: change JS selector for weekday indexes field (#74539) #37

Merged
vdeniaud merged 1 commits from wip/74539-js-plage-horaire-ne-pas-se-baser into main 2023-02-15 12:27:21 +01:00
Owner
No description provided.
fpeters approved these changes 2023-02-15 12:21:35 +01:00
fpeters left a comment
Owner

Mais avec ou sans ce changement, c'est ok.

Mais avec ou sans ce changement, c'est ok.
@ -40,3 +40,3 @@
return;
if(this.value == 'every-week') {
$('div#weekday_indexes').parent().hide();
$('ul#id_weekday_indexes').parent().parent().hide();
Owner

On peut en profiter pour retirer le nom de balise, avoir uniquement l'id.

On peut en profiter pour retirer le nom de balise, avoir uniquement l'id.
Author
Owner

Dac tu fais bien de me le dire, j'ignore tout des bonnes pratiques JS.

Dac tu fais bien de me le dire, j'ignore tout des bonnes pratiques JS.
vdeniaud force-pushed wip/74539-js-plage-horaire-ne-pas-se-baser from 2cc692f168 to e6d6f520d6 2023-02-15 12:24:56 +01:00 Compare
vdeniaud merged commit e6d6f520d6 into main 2023-02-15 12:27:21 +01:00
vdeniaud deleted branch wip/74539-js-plage-horaire-ne-pas-se-baser 2023-02-15 12:27:21 +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/chrono#37
No description provided.