----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126590/#review90397 -----------------------------------------------------------
applets/webbrowser/contents/ui/main.qml <https://git.reviewboard.kde.org/r/126590/#comment61836> Try binding this directly: url: plasmoid.configuration.url (Might cause a binding loop, in which case your approach is fine) applets/webbrowser/contents/ui/main.qml (line 57) <https://git.reviewboard.kde.org/r/126590/#comment61835> Note plasmoid.configuration.url is a String whereas url is a QUrl. Working with URLs, especially comparing them, is unintuitive in QML, in doubt use url.toString() and then compare them. I'm wondering if you need this check at all, as I think the underlying QQmlPropertyMap is smart enough to detect non-changes. - Kai Uwe Broulik On Jan. 1, 2016, 4:31 nachm., Mihail Ivchenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126590/ > ----------------------------------------------------------- > > (Updated Jan. 1, 2016, 4:31 nachm.) > > > Review request for Plasma. > > > Bugs: 357387 > http://bugs.kde.org/show_bug.cgi?id=357387 > > > Repository: kdeplasma-addons > > > Description > ------- > > Remember last visited url and restore it on startup. > BUG: 357387 > > > Diffs > ----- > > applets/webbrowser/contents/config/main.xml PRE-CREATION > applets/webbrowser/contents/ui/main.qml 3ebf120 > applets/webbrowser/metadata.desktop 222af48 > > Diff: https://git.reviewboard.kde.org/r/126590/diff/ > > > Testing > ------- > > Successfully restored after logout. > Successfully restored after reboot. > Each plasmoid is correctly restored in case there was more than 1 plasmoid. > > > Thanks, > > Mihail Ivchenko > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
