Ticket #34 (closed defect: fixed)

Opened 9 months ago

Last modified 9 months ago

extra whitespace above headers breaks header

Reported by: zak Owned by: why
Priority: minor Milestone:
Component: test Version: SUPER
Keywords: whitespace Cc:

Description

I noticed (in svn/branches/superredcloth r236) if there is more than one blank line above a header, that header is treated as a regular paragraph. Here are some tests to demonstrate:

---
name: header with 1 blank line above
in: |-
  text
  
  h1. Header
out: <p>text</p><h1>Header</h1>
#passes
---
name: header with 2 blank lines above
in: |-
  text
  
  
  h1. Header
out: <p>text</p><h1>Header</h1>
#actual-out: <p>text</p><p>h1. Header</p>
---
name: header with 1 blank line above and with no text
in: |-
  
  h1. Header
out: <h1>Header</h1>
#actual-out: <p>h1. Header</p>

Attachments

whitespace_extra_tests.yml (0.9 kB) - added by zak 9 months ago.
a few tests
superredcloth_scan.rl.diff (0.7 kB) - added by zak 9 months ago.
appears to fix "header with 2 blank lines above" test, but not others

Change History

Changed 9 months ago by zak

a few tests

Changed 9 months ago by zak

appears to fix "header with 2 blank lines above" test, but not others

Changed 9 months ago by jgarber

  • status changed from new to closed
  • resolution set to fixed

Yeah, I messed that up, didn't I? Good catch. Now fixed in [239]

Note: See TracTickets for help on using tickets.