> On Sept. 29, 2011, 1:23 p.m., Martin Klapetek wrote: > > package/contents/ui/EditableText.qml, line 63 > > <http://git.reviewboard.kde.org/r/102736/diff/1/?file=37701#file37701line63> > > > > You shouldn't set the label contents back to the lineedit. The lineedit > > should be the data source and the label should be its visualisation, ie. it > > should only reflect what's in the lineedit and the lineedit should be only > > edited by user.
I need this for startup when reading info from config file. After that this is no more used - Francesco ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102736/#review6911 ----------------------------------------------------------- On Sept. 29, 2011, 12:19 p.m., Francesco Nwokeka wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/102736/ > ----------------------------------------------------------- > > (Updated Sept. 29, 2011, 12:19 p.m.) > > > Review request for Telepathy. > > > Description > ------- > > Fixes the bug stated in the bug report. I've added a "clickMessage" to the > lineedit when text is empty. > > > This addresses bug 282210. > http://bugs.kde.org/show_bug.cgi?id=282210 > > > Diffs > ----- > > package/contents/config/main.xml 6cb185a > package/contents/ui/EditableText.qml 23e86df > package/contents/ui/LauncherPanel.qml 999748a > package/contents/ui/main.qml 6739d27 > > Diff: http://git.reviewboard.kde.org/r/102736/diff/diff > > > Testing > ------- > > > Thanks, > > Francesco Nwokeka > >
_______________________________________________ KDE-Telepathy mailing list KDE-Telepathy@kde.org https://mail.kde.org/mailman/listinfo/kde-telepathy
