----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/4414/#review6240 -----------------------------------------------------------
/trunk/KDE/kdebase/workspace/plasma/generic/runners/bookmarks/bookmarksrunner.cpp <http://reviewboard.kde.org/r/4414/#comment5839> it shouldn't continue the loop when the context is no longer valid (and therefore go through all url's in the bookmarks), it should return. - Aaron On 2010-06-22 15:16:45, Jan Gerrit Marker wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/4414/ > ----------------------------------------------------------- > > (Updated 2010-06-22 15:16:45) > > > Review request for Plasma. > > > Summary > ------- > > Corrects the behavior of the bookmarks runner regarding Firefox: > 1. Firefox intern bookmarks (like the "Recently used bookmarks" folder) are > not shown > 2. Only bookmarks with a non empty name and url are shown > 3. The profile written to the config file is updated if it does not exist > anymore > > Additionally I polished the Firefox regarding method. > I did not commit directly because I was not sure whether it is allowed at > this point of the KDE SC development. > > > Diffs > ----- > > > /trunk/KDE/kdebase/workspace/plasma/generic/runners/bookmarks/bookmarksrunner.cpp > 1141326 > > Diff: http://reviewboard.kde.org/r/4414/diff > > > Testing > ------- > > I have tested it with current trunk - everything seems to be ok. > > > Thanks, > > Jan Gerrit > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
