----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114479/#review46065 -----------------------------------------------------------
kio/bookmarks/kbookmarkdialog.cc <https://git.reviewboard.kde.org/r/114479/#comment32866> This seems like it could select a different item with the same text, in case of duplicates. Can't group.address() be used instead? (I don't know if this dialog has methods for finding a bookmark by its address like 0/2/1) - David Faure On Dec. 15, 2013, 4:40 p.m., Dawit Alemayehu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/114479/ > ----------------------------------------------------------- > > (Updated Dec. 15, 2013, 4:40 p.m.) > > > Review request for kdelibs and David Faure. > > > Bugs: 152158 > http://bugs.kde.org/show_bug.cgi?id=152158 > > > Repository: kdelibs > > > Description > ------- > > When a user creates a new bookmark folder in the Add Bookmark dialog, make it > the current selected item. > > > Diffs > ----- > > kio/bookmarks/kbookmarkdialog.cc 713ceff > > Diff: https://git.reviewboard.kde.org/r/114479/diff/ > > > Testing > ------- > > > File Attachments > ---------------- > > Add new folder w/o patch > > https://git.reviewboard.kde.org/media/uploaded/files/2013/12/15/1be1b4c9-eddd-43cf-b3fa-18cc0a44b212__before.png > Add new folder w/ patch > > https://git.reviewboard.kde.org/media/uploaded/files/2013/12/15/353b3c50-ccc8-4390-9d76-a9f85a703987__after.png > > > Thanks, > > Dawit Alemayehu > >
