Bug 282210 - Fix "Set status message here" bug
Summary: Fix "Set status message here" bug
Status: RESOLVED FIXED
Alias: None
Product: telepathy
Classification: Unmaintained
Component: presence-applet (show other bugs)
Version: unspecified
Platform: Unlisted Binaries Linux
: NOR normal
Target Milestone: 0.4.0
Assignee: Telepathy Bugs
URL:
Keywords:
Depends on:
Blocks: 282193
  Show dependency treegraph
 
Reported: 2011-09-17 11:02 UTC by George Goldberg
Modified: 2012-07-06 12:47 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description George Goldberg 2011-09-17 11:02:35 UTC
Fix "Set status message here" bug

Assigned to nwoki for 0.2
Comment 1 David Edmundson 2011-09-23 09:59:53 UTC
How this should work (imho):

PlasmaLineEdit has a property called "clickMessage" this should be used instead of setting the text, it's a sort of greyed out message that disappears when you press it.

The problem is that you have a class in the way called "EditableText" which switches between a label and the lineEdit. The label needs to show the text from the lineedit if it exists..or the click message otherwise.

Also make sure to check that "set status message here" is translated. I'm pretty sure yours is not because you load it from a config file (which seems like a weird thing to do anyway).
Comment 2 Francesco Nwokeka 2011-10-03 09:16:35 UTC
Git commit de1d5efe08a4cdfc5387151e2a4017ccd26044c2 by Francesco Nwokeka.
Committed on 03/10/2011 at 11:17.
Pushed by nwokeka into branch 'master'.

Fix "Set personal status here" bug

This bug caused the CM to load an unwanted string.
REVIEW: 102736
BUG: 282210
Reviewed by: George K.

M  +2    -2    package/contents/config/main.xml
M  +60   -10   package/contents/ui/EditableText.qml
M  +1    -1    package/contents/ui/LauncherPanel.qml
M  +2    -0    package/contents/ui/LeftPanel.qml
M  +1    -1    package/contents/ui/main.qml

http://commits.kde.org/telepathy-presence-applet/de1d5efe08a4cdfc5387151e2a4017ccd26044c2