Summary: | Add option to disable formatting in plain-text messages | ||
---|---|---|---|
Product: | [Applications] kmail2 | Reporter: | Dan Ramaley <daniel.ramaley> |
Component: | message list | Assignee: | kdepim bugs <kdepim-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | wishlist | CC: | bjoern, hofer-kde, luigi.toscano, ma_wo, montel, philschmidt, wiesner_m.rgbg |
Priority: | NOR | Keywords: | junior-jobs |
Version: | 4.9.0 | ||
Target Milestone: | --- | ||
Platform: | Debian testing | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/kdepimlibs/4a0cc11a2d8c63318cc23d37b3ab3131085838f4 | Version Fixed In: | 4.10 |
Attachments: |
Testcase: mbox file
Testcase rendered in Kmail and Thunderbird |
Description
Dan Ramaley
2010-01-20 20:30:15 UTC
I dont think this is an entry for the wishlist, it's really a bug. First, you cant cut and paste correctly like with graphically showed smileys. And second this "feature" falsify the message, especially if the writer dont intend it. For example if it's a coded text, a symbol, a name... ... type *1234* at your telephon to activate feature XYZ ... ... the product code is *A516111* ... ... the password is *ABCD* ... please print *SuperStore* at the header of the sign ... tested with kmail 1.13.1 still there kmail 1.13.5 Still present in KMail 4.8.0. And I agree, this ist definitely a Bug, not a Feature or Wishlist. While it "may" be common (or at least accepted) to format messages this way on the Internet, "Plain text" should really mean just that. If people want real formatting, they can use HTML. Thank you for your feature request. Kmail1 is currently unmaintained so we are closing all wishes. Please feel free to reopen a feature request for Kmail2 if it has not already been implemented. Thank you for your understanding. Instead of creating a new feature request, please confirm here if the wishlist is still valid for kmail2. See my comment above. Currently using 4.9.0 and it is still there. In the configuration there is an option to enable replacing smiliey with emoticons, the same could be done for _underlined_ *fat* and italic text (don't know how that is marked). Created attachment 75448 [details]
Testcase: mbox file
This is still a wishlist item, rather than a bug, as long as real world testcases, where this otherwise useful (and common) behaviour distorts the message content, can't be fixed appropriately.
I prepared a testcase and comparing Thunderbird and Kmail rendering.
Firstly, Thunderbird renders *bold*, /italics/ and _underscore_ but doesn't remove the markup.
This might be preferable because it ensures no information gets lost, though it is visually less appealing.
So if we stick with our way, we need to be 100% on the secure side.
In more complex cases, both clients to some extent show inconsistentencies, such as:
- the way precendence is handled in multi-formatting, with both browsers showing inconsistencies (why don't we actually render */text/* both italics and bold?)
- Thunderbird render /var/log/ in italics, and *foo*bar* in bold while we don't
- We render numeric-only expressions in bold/italics/whatever, while Thunderbird doesn't
- We render *_foo* bold, but not /*foo* - while Thunderbird does it the other way around.
Firstly, I'd propose to making an informed decision whether we want to keep on removing the markup. If yes, then we need to be more careful about potentially incorrect, potentially complex expressions.
In any case, multi-format rendering should be at least handled in a consistent way, possibly multi-formatted with the exception of /*foo*/ (which is comment markup), or - the other way around - ignoring multiple formatting altogether.
Created attachment 75449 [details]
Testcase rendered in Kmail and Thunderbird
(In reply to comment #7) thank for testcase will look at it soon Git commit 4a0cc11a2d8c63318cc23d37b3ab3131085838f4 by Montel Laurent. Committed on 28/12/2012 at 08:13. Pushed by mlaurent into branch 'KDE/4.10'. Fix Bug 223616 - Add option to disable formatting in plain-text messages FIXED-IN: 4.10 Don't add new options. Just don't remove markup as in thunderbird. so we can copy/paste with loosing infos, and we are sure that we show exact string. M +4 -4 kpimutils/linklocator.cpp http://commits.kde.org/kdepimlibs/4a0cc11a2d8c63318cc23d37b3ab3131085838f4 I decided to not remove "*" "/" etc but change formatting. I didn't want to add more options. For multi-format rendering I will look at what I can do. |