Patch #28119
closed
Enable lax_spacing for markdown formatting in order to allow markdown blocks not surrounded by empty lines
Description
In the markdown standard format, an empty line is required before and after a block in order to render the content properly.
1. List without an empty line (doesn't work)  |
-> |  |
 |
-> |  |
:lax_spacing for markdown formatting (more details about this option on github) allows also the first format:
 |
-> |  |
We already rolled out this change on our instance because most of the times, we forget to let the empty line and we observe that it doesn't work after we save the changes.
The patch is attached.
Files
Related issues
Updated by Bernhard Rohloff almost 7 years ago
It happened to me just seconds ago.
+1
Updated by Mischa The Evil almost 7 years ago
- Target version set to Unplanned backlogs
This LGTM.
OT: Your patch reminded me btw that we might also want a simple test (like ApplicationHelperTest#test_footnotes) for markdown footnotes (in source:trunk/test/unit/lib/redmine/wiki_formatting/markdown_formatter_test.rb) as enabled per #16072. And while writing this, the questions of 'how many other textile syntax tests are there remaining lurking outside of source:/trunk/test/unit/lib/redmine/wiki_formatting/textile_formatter_test.rb and what do we want to do with them?' comes up... But that is certainly for another issue.
Updated by Go MAEDA almost 7 years ago
- Target version changed from Unplanned backlogs to 4.1.0
+1. It often annoys me.
I think we can commit this change to the trunk. Setting target version to 4.1.0.
Updated by Go MAEDA almost 7 years ago
- Status changed from New to Closed
- Assignee set to Go MAEDA
- Target version changed from 4.1.0 to 4.0.0
Committed. Thank you for the improvement.
Updated by Go MAEDA over 6 years ago
- Has duplicate Defect #25339: Markdown: list renders only with leading newline added
Updated by Go MAEDA over 6 years ago
- Target version changed from 4.0.0 to 3.3.7
Merged to stable branches.