https://bugs.kde.org/show_bug.cgi?id=360205

--- Comment #14 from Jan Kundrát <j...@kde.org> ---
> -    subDate += QStringLiteral("<span style=\"font:bold
> large;\">%1</span>").arg(e.subject.toHtmlEscaped());
> +    subDate += QStringLiteral("<span style=\"font:bold large;
> white-space:pre;\">%1</span>").arg(e.subject.toHtmlEscaped());

While you're changing this, try to consider a situation where the subject 
consists of just one overly long sequence of characters with no spaces in 
the middle -- just a reminder for future.

On topic of preserving whitespace, let's be realistic, please. We already 
wrap the subject in funny ways around the date, and I *like* this. If we 
want to go full RFC-lawyering here, nothing defines how to render these 
subjects, and wrapping them into several lines is an obvious usability 
feature. I do not think that distinguishing between tabs, spaces, emspaces 
and newlines makes a lot of sense in subjects. If anything, my today's vote 
:) is to ensure that newlines do not lead to visible line breaks (think 
possible information hiding) and that spaces are normalized to a reasonable 
amount of whitespace.

> It seems unicode em-spaces get converted to plain spaces by
> QString::toHtmlEscaped

I've checked the source of that method, and it only replaces <, >, & and ".

> Is there an easy way to have a look at the html of a message view in Trojita?

Add a call to qDebug() to an appropriate place. The markup is asked from 
from within src/Gui/SimplePartWidget.cpp.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to