Email text for merge proposals gets re-wrapped.

Bug #724864 reported by Henning Eggers
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Triaged
High
Unassigned

Bug Description

This is an annoying bug because it makes merge proposals hard to read.

When I create a merge proposal using "bzr propose-merge" I use my local editor to create the mp description. Naturally, that is configured to wrap at 78 (and am I ;-) and as far as I can tell, I am not the only one. But when the description reaches the mp in the UI, the lines have been re-wrapped (but not re-flowed!) to a shorter line width (72?). The result looks like this:
https://code.launchpad.net/~henninge/launchpad/bug-720673-import-origin/+merge/50157

Sure, I can remember to wrap at 72 lines but why that short line length in the first place?

Tags: easy
j.c.sackett (jcsackett)
Changed in launchpad:
status: New → Triaged
importance: Undecided → High
Revision history for this message
Gavin Panella (allenap) wrote :

72 is a typical email limit: 72 columns + space for three levels of
"> " quoting.

One possible set of rules to follow could be:

- Don't wrap/reflow text before storing in the database.

- For displaying or emailing, only wrap/reflow lines of greater than
  78 columns.

- When required, wrap/reflow to 72 columns.

Revision history for this message
Robert Collins (lifeless) wrote : Re: [Bug 724864] Re: Email text for merge proposals gets re-wrapped.

Do we need to wrap at all other than when sending notifications?
Browsers are not email clients.

Revision history for this message
Robert Collins (lifeless) wrote :

I suspect this is easy. IMBW.

tags: added: easy
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.