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

js: show remaining characters feedback for text fields with maxlength (#73401) #660

Open
csechet wants to merge 1 commits from wip/73401-longueur-champ-text into main
pull from: wip/73401-longueur-champ-text
merge into: entrouvert:main
Conversation 1 Commits 1 Files Changed 5 +68 -1
csechet commented 2023-09-11 08:39:59 +02:00
Owner
Copy Link
No description provided.
csechet added 1 commit 2023-09-11 08:40:00 +02:00
lguerin reviewed 2023-09-11 09:57:33 +02:00
tests/test_fields.py Outdated
@ -162,0 +170,4 @@
form = Form(use_tokens=False)
fields.TextField(maxlength=110, display_mode='rich').add_to_form(form)
print(form.render())
lguerin commented 2023-09-11 09:57:33 +02:00
Owner
Copy Link

il reste quelques print dans les tests :)

il reste quelques print dans les tests :)
csechet commented 2023-09-11 10:16:59 +02:00
Author
Owner
Copy Link

Woopsie

Woopsie
csechet marked this conversation as resolved
csechet force-pushed wip/73401-longueur-champ-text from e7af632e5a to b2b3cca6bf 2023-09-11 10:18:14 +02:00 Compare
fpeters approved these changes 2023-11-10 10:25:21 +01:00
fpeters left a comment
Owner
Copy Link

Ça marche mais pour godo il n'y a pas de prise en charge du maxlength donc ça peut finir à taper un nombre négatif en nombre de caractères encore autorisés, ce qui est moyen, mais ce corrigerait ailleurs, https://dev.entrouvert.org/issues/80287

(j'aurais envie d'attendre que ce ticket-là soit traité avant de valider ici, mais ça fait déjà longtemps que ça attend)

Ça marche mais pour godo il n'y a pas de prise en charge du maxlength donc ça peut finir à taper un nombre négatif en nombre de caractères encore autorisés, ce qui est moyen, mais ce corrigerait ailleurs, https://dev.entrouvert.org/issues/80287 (j'aurais envie d'attendre que ce ticket-là soit traité avant de valider ici, mais ça fait déjà longtemps que ça attend)
csechet commented 2023-11-10 15:12:36 +01:00
Author
Owner
Copy Link

On peut attendre, je me suis assigné l'autre ticket (et j'attendrai bien que https://dev.entrouvert.org/issues/69407 passe avant de faire des modifs dans Godo aussi, on pourra en parler la semaine prochaine avec Thomas J.).

On peut attendre, je me suis assigné l'autre ticket (et j'attendrai bien que https://dev.entrouvert.org/issues/69407 passe avant de faire des modifs dans Godo aussi, on pourra en parler la semaine prochaine avec Thomas J.).
All checks were successful
gitea/wcs/pipeline/head This commit looks good
Details
This pull request has changes conflicting with the target branch.
  • wcs/qommon/static/js/qommon.forms.js
  • wcs/qommon/templates/qommon/forms/widgets/mini-rich-text.html
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
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#660
No description provided.
Delete Branch "wip/73401-longueur-champ-text"

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?