----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102152/#review5289 -----------------------------------------------------------
just a quick suggestion: to properly address the use case I believe this is supposed to tackle, I think we'd need a mechanism to have different default bookmark locations based on how it was invoked (e.g. in that case, the user probably wants to dump those links in a "to read later" bookmark folder). - Pierre On July 30, 2011, 5:34 p.m., Johannes Tröscher wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/102152/ > ----------------------------------------------------------- > > (Updated July 30, 2011, 5:34 p.m.) > > > Review request for rekonq. > > > Summary > ------- > > adds an "bookmark this link" action to the link's context menu > > > Diffs > ----- > > src/webview.h b3db5ce > src/webview.cpp a938ad7 > > Diff: http://git.reviewboard.kde.org/r/102152/diff > > > Testing > ------- > > compiles and works > > > Screenshots > ----------- > > > http://git.reviewboard.kde.org/r/102152/s/214/ > > > Thanks, > > Johannes > >
_______________________________________________ rekonq mailing list [email protected] https://mail.kde.org/mailman/listinfo/rekonq
