We in editor land would like to be able to control the COLS attribute on preformatted text from style. This will be used to control wrapping width in plain text mail compose.

Change platform to all

Will most likely add a "ch" unit (for characters) to 'width' property.

"ch" unit is in place.

Simon: Could you please verify that this is fixed and if so, mark it verified? Thanks

To test this, make a page with the following style: body { whitespace: -moz-pre-wrap; width: 80ch; } the text in this page should then wrap at 80 characters. I can't get this to work in the editor's plain text page now, so we can't mark this verified yet.

Agreed: this doesn't seem to be working. I've tried -moz-pre-wrap and moz-pre-wrap with width set to 40 characters, but text doesn't wrap, either on initial layout or when I type in new text.

QA: this was our problem in the way we were setting style. Peter showed me what we needed to fix, and it's working now. Test it if you want to be sure, otherwise go ahead and mark it verified.

Could you describe the correct usage here?

Here's something that works if specified in EditorInitPagePlain.html. (I have it moved into C++ code in my tree, expect to be ready to check in tomorrow.) BODY { white-space: -moz-pre-wrap; width: 72ch; font-family: -moz-fixed; background-color: rgb(255, 255, 255); margin: 4px 4px 4px 4px; } In addition, the bodyElement->SetAttribute("style", ...) call in nsTextEditRules.cpp is wrong -- it overrides the style in the init page to a non-wrapping preformatted style.

Using revised test case with 8/16 Apprunner, verified bug fixed. Text wraps using "ch" unit for width.