Opened 8 years ago

Closed 8 years ago

Last modified 5 years ago

#9883 closed (fixed)

the linebreaks filter strips leading and trailing whitespace from paragraphs

Reported by: Antti Kaihola Owned by: Keith Bussell
Component: Template system Version: master
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: yes Patch needs improvement: yes
Easy pickings: UI/UX:

Description

The documentation for linebreaks says:

Replaces line breaks in plain text with appropriate HTML; a single newline becomes an HTML
line break (<br />) and a new line followed by a blank line becomes a paragraph break (</p>).

For example:
  {{ value|linebreaks }}
If value is Joel\nis a slug, the output will be <p>Joel<br>is a slug</p>.

However, {{value|linebreaks}} does more than that: it strips leading whitespace from the first line and trailing whitespace from the last line of every paragraph:

>>> linebreaks('  one  \n  two  \n  three  \n  four  \n\n  eins  \n  zwei  \n  drei  \n')
<p>one  <br />  two  <br />  three  <br />  four</p>

<p>eins  <br />  zwei  <br />  drei</p>

I admit this rarely is a problem, but I see no reason why it should be doing that (and not mention it in the docs).

My obscure use case is outputting monospace text where some tables are laid out using the 'NO-BREAK SPACE' (U+00A0) Unicode character, which Python's .strip() happily strips away. If a paragraph starts with a table and the first row is indented, the output is misformatted. Using linebreaksbr or converting U+00A0 to &nbsp; are my possible work-arounds, but it would been nice if the Django documentation had warned me :)

Attachments (1)

no_strip.diff (767 bytes) - added by Keith Bussell 8 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 8 years ago by Jacob

milestone: 1.1
Triage Stage: UnreviewedAccepted

Changed 8 years ago by Keith Bussell

Attachment: no_strip.diff added

comment:2 Changed 8 years ago by Keith Bussell

Has patch: set
Resolution: fixed
Status: newclosed
Triage Stage: AcceptedReady for checkin

James found the original commit back from prehistoric Django times and found no good reason to do the stripping. no_strip.diff removes the strip() call in both normal and autoescape code paths.

Tests that utilize this functionality still pass.

comment:3 Changed 8 years ago by Keith Bussell

Resolution: fixed
Status: closedreopened

oops. went too far with the ticket state.

comment:4 Changed 8 years ago by Keith Bussell

Owner: changed from nobody to Keith Bussell
Status: reopenednew

comment:5 Changed 8 years ago by Keith Bussell

Status: newassigned

comment:6 Changed 8 years ago by Gary Wilson

Needs tests: set
Patch needs improvement: set
Triage Stage: Ready for checkinAccepted

Let's have a test that shows the problem.

comment:7 Changed 8 years ago by Jacob

Resolution: fixed
Status: assignedclosed

(In [10225]) Fixed #9883: no longer do strange things with whitespace in the linebreaks filter. Thanks, keithb.

comment:8 Changed 8 years ago by Jacob

(In [10292]) [1.0.X] Fixed #9883: no longer do strange things with whitespace in the linebreaks filter. Thanks, keithb.

Backport of r10225 from trunk.

comment:9 Changed 5 years ago by Jacob

milestone: 1.1

Milestone 1.1 deleted

Note: See TracTickets for help on using tickets.
Back to Top