entrouvert
/
godo.js
16
1
Fork 0
Code Issues Pull Requests 2 Releases Wiki Activity

WIP: Editeur sans textarea (#59608) #14

Closed
csechet wants to merge 3 commits from wip/59608-editeur-sans-textarea into main
pull from: wip/59608-editeur-sans-textarea
merge into: entrouvert:main
Conversation 1 Commits 3 Files Changed 5 +155 -57
csechet commented 2024-01-08 18:23:33 +01:00
Owner
Copy Link

"contenteditable" n'est pas transmis à la vue ProseMirror, qui l'override. Donc setter un "contenteditable" sur le godo editor ne fonctionne pas. J'ai donc utilisé un attribut custom "no-edit" (nom discutable), parce que "read-only" a une sémantique différente et c'est pas bien d'overrider les attributs natifs. Ca permettra de styler l'éditeur sans bordure par exemple quand on est en "no-edit", si on veut éditer la doc en place par exemple, et d'avoir un style différent (grisé) pour le read-only.

La propriété "value" pour synchroniser l'état de la vue et du textarea et simplifier le travail de celleui qui devra setter / récupérer le HTML depuis l'éditeur Godo.

"contenteditable" n'est pas transmis à la vue ProseMirror, qui l'override. Donc setter un "contenteditable" sur le godo editor ne fonctionne pas. J'ai donc utilisé un attribut custom "no-edit" (nom discutable), parce que "read-only" a une sémantique différente et c'est pas bien d'overrider les attributs natifs. Ca permettra de styler l'éditeur sans bordure par exemple quand on est en "no-edit", si on veut éditer la doc en place par exemple, et d'avoir un style différent (grisé) pour le read-only. La propriété "value" pour synchroniser l'état de la vue et du textarea et simplifier le travail de celleui qui devra setter / récupérer le HTML depuis l'éditeur Godo.
csechet added 3 commits 2024-01-08 18:23:34 +01:00
gitea/godo.js/pipeline/head This commit looks good Details
0ea161a98a js: allow to use godo as an HTML element (#84649)
gitea/godo.js/pipeline/head This commit looks good Details
48a55a94bd js: add 'value' property on editor (#59608)
csechet force-pushed wip/59608-editeur-sans-textarea from 48a55a94bd to 48b6888d18 2024-01-10 11:56:22 +01:00 Compare
csechet changed title from wip/59608-editeur-sans-textarea to WIP: wip/59608-editeur-sans-textarea 2024-01-10 13:54:25 +01:00
csechet force-pushed wip/59608-editeur-sans-textarea from 48b6888d18 to 80ac8f806e 2024-01-11 16:00:57 +01:00 Compare
csechet changed title from WIP: wip/59608-editeur-sans-textarea to Editeur sans textarea (#59608) 2024-01-11 16:01:34 +01:00
csechet force-pushed wip/59608-editeur-sans-textarea from 80ac8f806e to cc24f8ddf8 2024-01-11 16:04:06 +01:00 Compare
csechet commented 2024-01-16 11:35:57 +01:00
Author
Owner
Copy Link

Préférer "preview" plutôt que "no-edit" pour le nom d'attribut.

Préférer "preview" plutôt que "no-edit" pour le nom d'attribut.
csechet changed title from Editeur sans textarea (#59608) to WIP: Editeur sans textarea (#59608) 2024-01-29 09:13:28 +01:00
csechet closed this pull request 2024-02-01 19:09:17 +01:00
All checks were successful
gitea/godo.js/pipeline/head This commit looks good
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
1 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/godo.js#14
No description provided.
Delete Branch "wip/59608-editeur-sans-textarea"

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?