Bug 316626 - Two spaces shows as "  " in notification
Summary: Two spaces shows as "  " in notification
Status: RESOLVED UPSTREAM
Alias: None
Product: telepathy
Classification: Frameworks and Libraries
Component: text-ui (show other bugs)
Version: 0.5.80
Platform: Other Linux
: NOR normal
Target Milestone: 0.6.0
Assignee: Telepathy Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-03-12 23:31 UTC by Martin Klapetek
Modified: 2013-03-15 15:54 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
notification screenshot (12.05 KB, image/png)
2013-03-12 23:31 UTC, Martin Klapetek 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Klapetek 2013-03-12 23:31:05 UTC 
Created attachment 77998 [details]
notification screenshot

When there are two or more spaces in a message, the notification displays only one and then  . See screenshot.
Comment 1 David Edmundson 2013-03-13 00:04:04 UTC 
This is quite different from the \\ bug.

The \\ bug is caused by the notifications including an escaping that only applied to the text-ui.

This appears to be a different bug that KNotification is escaping the & for us (and incorrectly).

QLabel does support   so someone is escaping it somewhere.
Comment 2 Daniele E. Domenichelli 2013-03-13 07:44:21 UTC 
Which notification is this? I cannot reproduce it...
Comment 3 Martin Klapetek 2013-03-13 10:55:31 UTC 
Incoming text-ui message, when the window is not focused.
Comment 4 David Edmundson 2013-03-13 12:07:17 UTC 
This also applied to " and other HTML entities.

Lazy solution is to add a method in KTp::Message() that has the original untouched text.
I'd rather not do that, but we can keep it as an option if we don't fix this before 0.6.0
Comment 5 Daniele E. Domenichelli 2013-03-13 13:25:18 UTC 
I still cannot reproduce it.(In reply to comment #3)
> Incoming text-ui message, when the window is not focused.

I still cannot reproduce it
Comment 6 David Edmundson 2013-03-13 13:28:23 UTC 
Because you are on an older KDE with the old notifications that use QLabel ! 

This is a bug in Plasma.. great.
Comment 7 David Edmundson 2013-03-15 02:34:50 UTC 
Tested in QML viewer, notification just needs

        textFormat: Text.RichText
Comment 8 David Edmundson 2013-03-15 15:54:17 UTC 
Pending fix upstream at: https://git.reviewboard.kde.org/r/109492/

Assuming it gets in, this bug only affects 4.10.0 and 4.10.1 we should be releasing around the time of 4.10.2   so it should be OK to release like this without work arounds.

Comment if you disagree.