Skip to content

fix(TaskList): Add class name to rendered HTML - #6323

Merged
mejo- merged 1 commit into
mainfrom
fix/paste_task_list
Sep 10, 2024
Merged

fix(TaskList): Add class name to rendered HTML#6323
mejo- merged 1 commit into
mainfrom
fix/paste_task_list

Conversation

@mejo-

@mejo-mejo- commented Sep 6, 2024

Copy link
Copy Markdown
Member

The upstream Tiptap tasklist implementation uses data-type attributes to identify task lists and task items. We use class names instead as we depend on the markdown-it-task-lists plugin.

This fixes copy & paste of task lists.

Fixes: #5237

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits
  • Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests

@mejo-mejo- added bug Something isn't working 3. to review labels Sep 6, 2024
@mejo-mejo- self-assigned this Sep 6, 2024
@mejo-
mejo-force-pushed the fix/paste_task_list branch from d76282c to d000c8fCompareSeptember 6, 2024 13:28
The upstream Tiptap tasklist implementation uses data-type attributes to
identify task lists and task items. We use class names instead as we
depend on the markdown-it-task-lists plugin.
This fixes copy & paste of task lists.
Fixes: #5237
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
mejo- merged commit fb29f2f into mainSep 10, 2024
@mejo-
mejo- deleted the fix/paste_task_list branch September 10, 2024 06:34
@mejo-

Copy link
Copy Markdown
MemberAuthor

/backport to stable30

@mejo-

Copy link
Copy Markdown
MemberAuthor

/backport to stable29

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewbugSomething isn't working

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Copy & paste of todo/task lists result in bulletpoints

2 participants

@mejo-@juliusknorr