> On Jan. 17, 2011, 4:25 p.m., Andrea Diamantini wrote: > > I like this, there is just one small thing to fix before merge. Or did you > > discuss something different in ml? I couldn't follow it.. > > Furkan Üzümcü wrote: > They talked about adding a button to the url bar like the add bookmark > button. But nobody said anything about it in ml. I'm updating the patch. > > Lionel Chauvin wrote: > I thought about reintroduce an "add bookmark" popup that allow to choose > the bookmark folder and that propose to add this bookmark in the favorite too > (with a checkbox).
I like that, Lionel. But I don't think it lets this be "unuseful". I'm for merging if you and ... (who implemented this?).. have no objections. Do you? - Andrea ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100286/#review957 ----------------------------------------------------------- On Jan. 16, 2011, 4:24 p.m., Furkan Üzümcü wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/100286/ > ----------------------------------------------------------- > > (Updated Jan. 16, 2011, 4:24 p.m.) > > > Review request for rekonq. > > > Summary > ------- > > Added support for entering link directly into the Favorites section in the > New Tab . > > > Diffs > ----- > > src/newtabpage.cpp 7dcc958 > src/previewselectorbar.h 9684a01 > src/previewselectorbar.cpp c8ab5b0 > src/websnap.h e5b550f > src/websnap.cpp 54cf105 > > Diff: http://git.reviewboard.kde.org/r/100286/diff > > > Testing > ------- > > * Tested and works! > > > Thanks, > > Furkan > >
_______________________________________________ rekonq mailing list [email protected] https://mail.kde.org/mailman/listinfo/rekonq
