Bug 305739 - Newline in article title causes display disortion on selection
Summary: Newline in article title causes display disortion on selection
Status: RESOLVED FIXED
Alias: None
Product: akregator
Classification: Applications
Component: general (show other bugs)
Version: 4.8.5
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-08-24 18:33 UTC by Rolf Eike Beer
Modified: 2012-09-04 21:00 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 4.9.2


Attachments
Screenshot unselected feed (12.47 KB, image/png)
2012-09-04 17:58 UTC, Rolf Eike Beer
Details
Screenshot with disortion (11.47 KB, image/png)
2012-09-04 17:59 UTC, Rolf Eike Beer
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Rolf Eike Beer 2012-08-24 18:33:31 UTC
I had a newfeed (lwn.net) selected which showed it's articles. Everything looked normal. But when I selected a particular article all lines in the article overview suddenly doubled their size.

Reproducible: Always

Steps to Reproduce:
The culprit was the article beginning with this tags:

    <item rdf:about="http://lwn.net/Articles/512765/rss">
      <title>Latest Sophos Cyber Security Challenge seeks to address critical Linux
security skills gap (SecurityPark.net)</title>

The newline between "Linux" and "security" was part of the feed. It was filtered out when the article was not selected but caused the item view disortion once the article got selected.


Expected Results:  
The newline should always be treated as simple space in the title.
Comment 1 Allen Winter 2012-08-30 20:53:16 UTC
I can't reproduce this with the lwn.net feed you mention. Perhaps they removed the newline in the meantime.

Or maybe I misunderstand.

If you still experience this, could you show me a screenshot?
Comment 2 Rolf Eike Beer 2012-08-30 21:28:00 UTC
Yes, the article in question is now missing in the feed. Just download the current feed and add a newline to the headline of any article. Problem is easily reproducable then.
Comment 3 Rolf Eike Beer 2012-09-04 17:58:40 UTC
Created attachment 73654 [details]
Screenshot unselected feed
Comment 4 Rolf Eike Beer 2012-09-04 17:59:09 UTC
Created attachment 73655 [details]
Screenshot with disortion
Comment 5 Allen Winter 2012-09-04 21:00:18 UTC
Git commit 403100eda01e8704fc69aad61f72318e402fa71e by Allen Winter.
Committed on 04/09/2012 at 22:26.
Pushed by winterz into branch 'KDE/4.9'.

consistently clean the article title using stripHtml() instead of
HtmlToPlainText(). stripHtml() replaces newlines and tabs properly.
FIXED-IN: 4.9.2

M  +2    -2    akregator/src/articlemodel.cpp

http://commits.kde.org/kdepim/403100eda01e8704fc69aad61f72318e402fa71e