| Summary: | Fix "Set status message here" bug | ||
|---|---|---|---|
| Product: | [Frameworks and Libraries] telepathy | Reporter: | George Goldberg <grundleborg> |
| Component: | presence-applet | Assignee: | Telepathy Bugs <kde-telepathy-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | francesco.nwokeka, kde |
| Priority: | NOR | ||
| Version: | unspecified | ||
| Target Milestone: | 0.4.0 | ||
| Platform: | Unlisted Binaries | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | ||
| Bug Depends on: | |||
| Bug Blocks: | 282193 | ||
|
Description
George Goldberg
2011-09-17 11:02:35 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). 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 |