-
Notifications
You must be signed in to change notification settings - Fork 94
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Preserve bullet list style and change default style to -
#3523
Conversation
/compile |
Test summaryRun details
View run in Cypress Dashboard ➡️ Flakiness
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
d164483
to
5ec082f
Compare
This comment was marked as resolved.
This comment was marked as resolved.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All looks good.
I might as well rebase and fix the remaining test so we can get this in. Hope you don't mind.
/* | ||
* This file is for various markdown tests, mainly testing if input and output stays the same. | ||
* | ||
* Please add test belonging to some Node or Mark to the corresponding file in './nodes/` or `./marks/` | ||
*/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Thanks for the separation. Makes lot's of sense.
1. Preserve the bullet style used for lists (also task lists) 2. Set default style to `-` Signed-off-by: Ferdinand Thiessen <[email protected]>
Signed-off-by: Ferdinand Thiessen <[email protected]>
Signed-off-by: Ferdinand Thiessen <[email protected]>
5ec082f
to
3b0f92c
Compare
/compile |
Signed-off-by: nextcloud-command <[email protected]>
📝 Summary
First this generally make text preserve the bullet style used for lists (and also task lists).
And also sets the default list style to
-
as discussed in the comments of #1297 .Added also test cases (and slightly refactored the test code to remove duplicated code (helpers)).
🏁 Checklist
npm run lint
/npm run stylelint
/composer run cs:check
)