[Trunk/7.0] word-wrap in Mail-Module does not work in Firefox

[Sebastian Braun]

1) Steps to reproduce the issue you have observed

I got a plain-text mail via fetchmail. The Text gets wrapped into a pre-tag (without line-breaks)

2) The result you observed

The message is shown with word-warp im chrome, but without word-wrap in firefox

3) The result you expected

Similar behavior in both browsers

4) The platform you are using (and browser version, if relevant)

Openerp Addons r9158
Chromium 25.0.1364.160 Ubuntu 13.04
Mozilla Firefox 21.0

According to stackoverflow.com/questions/4413015/browser-compatible-word-wrap-and-whitespace-pre you can fix this with css. I added the attribute "white-space: pre-wrap;" with the firefox inspector to the css-definitions and the word-wrap works in firefox. A patch is attached.

[Amit Bhavsar (Open ERP) (amb-openerp)]

Hello Sebastian,

I have checked your Issue with following revision.
OE 7.0
(Build 11600)
server (4985)
addons (9158)
web (3933)

But I did not face any problem. That's why I have attached the screen capture for your reference. would you please check It and notify us where you have faced the problem.

Thanks and waiting for your reply!

[Sebastian Braun (elmnt)]

Hi Amit,

I think the problem is connected to the plain/text part of my bug-report.

If I understand def _message_extract_payload() from mail/mail_thread.py right, plain/text parts of emails and emails seems to get put inside a pre-tag, which default behavior is not to word-wrap text (and has usually use a different font).

I also prepared an image.

Thanks,
Sebastian

[Amit Bhavsar (Open ERP) (amb-openerp)]

Hello Sebastian,

Yes, I have faced the same problem. and we should use "pre-wrap" for proper formatting. That's why I am confirming this Issue with low Importance.

Thanks!

[Dharmraj Zala(OpenERP) (dharmraj.zala-openerp)]

Hello Sebastian Braun,

I have fixed this issue in https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-1183589-dharmraj branch with
Revision ID : <email address hidden>
Revision No: 8747

Thanks,
Dharmraj.