entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 30 Releases Activity

widgets: update rich text on live update (#75274) #165

Merged
csechet merged 1 commits from wip/75274-prefill-dynamique-champ into main 2023-03-29 15:42:10 +02:00
Conversation 0 Commits 1 Files Changed 2 +6 -2
csechet commented 2023-03-15 11:54:17 +01:00
Owner
Copy Link

Je n'ai pas utilisé l’évènement "change", parce que ça amène trop de glissades sur les boucles de mise à jour. Avoir un évènement personnalisé permet aussi d'avoir une sémantique plus fine coté gestionnaires d'évènements (c'est des mots savants pour dire : me demandez pas d'utiliser l'event "change" stp, je passe déjà une sale journée).

Je n'ai pas utilisé l’évènement "change", parce que ça amène trop de glissades sur les boucles de mise à jour. Avoir un évènement personnalisé permet aussi d'avoir une sémantique plus fine coté gestionnaires d'évènements (c'est des mots savants pour dire : me demandez pas d'utiliser l'event "change" stp, je passe déjà une sale journée).
csechet force-pushed wip/75274-prefill-dynamique-champ from bc49513784 to 2b3aa13364 2023-03-15 11:55:07 +01:00 Compare
tjund requested review from tjund 2023-03-15 16:27:09 +01:00
tjund requested changes 2023-03-15 16:27:20 +01:00
wcs/qommon/static/js/qommon.forms.js
@ -612,3 +612,3 @@
if ($(widget).is('.widget-prefilled') || $(widget).is('.widget-readonly') || data.modified_field == 'user') {
// replace text input value
$(widget).find('input[type=text], input[type=tel], input[type=numeric], input[type=email], input[type=date], textarea').val(value.content);
const text_inputs = $(widget).find('input[type=text], input[type=tel], input[type=numeric], input[type=email], input[type=date], textarea');
tjund commented 2023-03-15 16:18:03 +01:00
Owner
Copy Link

(de jQuery tu peux te passer ?)

(de jQuery tu peux te passer ?)
bdauvergne commented 2023-03-15 16:38:35 +01:00
Owner
Copy Link

Il y a déjà du jQuery partout dans qommon.*.js je ne changerai pas de style pour 3 lignes; s'il y a une migration globale, pas forcément globale, mais au moins par grosse maille, vers du vanilla JS je veux bien, mais changer de style au milieu du fichier ça va juste perdre les gens et ne pas nous apporter grand chose.

Il y a déjà du jQuery partout dans qommon.*.js je ne changerai pas de style pour 3 lignes; s'il y a une migration globale, pas forcément globale, mais au moins par grosse maille, vers du vanilla JS je veux bien, mais changer de style au milieu du fichier ça va juste perdre les gens et ne pas nous apporter grand chose.
tjund commented 2023-03-15 16:44:12 +01:00
Owner
Copy Link

Alors au moins prefixer avec $ les variabels qui sont des object jQUery :
$text_inputs

Alors au moins prefixer avec $ les variabels qui sont des object jQUery : `$text_inputs`
csechet commented 2023-03-15 16:55:00 +01:00
Author
Owner
Copy Link

Il y a déjà du jQuery partout dans qommon.*.js je ne changerai pas de style pour 3 lignes;

Effectivement, pour ça, et ne pas sortir du cadre du patch (tu notera que j'ai utilisé dispatchEvent en dessous :) ).

s'il y a une migration globale, pas forcément globale, mais au moins par grosse maille, vers du vanilla JS

Si on bouge des bouts dans Gadjo ou qu'on découpe un peu ces gros fichiers pour ajouter des tests unitaires, ça sera l'occasion de faire ça.

> Il y a déjà du jQuery partout dans qommon.*.js je ne changerai pas de style pour 3 lignes; Effectivement, pour ça, et ne pas sortir du cadre du patch (tu notera que j'ai utilisé dispatchEvent en dessous :) ). > s'il y a une migration globale, pas forcément globale, mais au moins par grosse maille, vers du vanilla JS Si on bouge des bouts dans Gadjo ou qu'on découpe un peu ces gros fichiers pour ajouter des tests unitaires, ça sera l'occasion de faire ça.
csechet force-pushed wip/75274-prefill-dynamique-champ from 2b3aa13364 to 1b509529d1 2023-03-15 16:52:02 +01:00 Compare
tjund requested changes 2023-03-29 11:25:29 +02:00
wcs/qommon/static/js/qommon.forms.js
@ -614,1 +614,3 @@
$(widget).find('input[type=text], input[type=tel], input[type=numeric], input[type=email], input[type=date], textarea').val(value.content);
const text_inputs = $(widget).find('input[type=text], input[type=tel], input[type=numeric], input[type=email], input[type=date], textarea');
text_inputs.val(value.content)
text_inputs.each((_, el) => el.dispatchEvent(new Event('wcs:live-update')));
tjund commented 2023-03-15 16:45:59 +01:00
Owner
Copy Link

Pourquoi un underscore pour le param index ? je ne trouve pas ça très clair.

Pourquoi un underscore pour le param index ? je ne trouve pas ça très clair.
tjund commented 2023-03-29 11:25:22 +02:00
Owner
Copy Link

Pas vu de réponse à cette demande

Pas vu de réponse à cette demande
tjund commented 2023-03-29 11:47:06 +02:00
Owner
Copy Link

réponse via jabber :

C'est pour dire que la paramètre n'est pas utilisé
C'est une convention assez utilisée, en python aussi (et le linter va gueuler d'ailleurs, si on le fait pas)

réponse via jabber : > C'est pour dire que la paramètre n'est pas utilisé > C'est une convention assez utilisée, en python aussi (et le linter va gueuler d'ailleurs, si on le fait pas)
tjund marked this conversation as resolved
tjund approved these changes 2023-03-29 11:47:18 +02:00
csechet force-pushed wip/75274-prefill-dynamique-champ from 1b509529d1 to 96810d58bb 2023-03-29 14:49:52 +02:00 Compare
csechet merged commit 96810d58bb into main 2023-03-29 15:42:10 +02:00
csechet deleted branch wip/75274-prefill-dynamique-champ 2023-03-29 15:42:10 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
tjund
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/wcs#165
No description provided.
Delete Branch "wip/75274-prefill-dynamique-champ"

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?