> On Nov. 22, 2011, 6:27 p.m., Yoann Laissus wrote: > > This event filter is complex enough :D > > I think it's better to group at the same place all the things related to a > > click on a KBookarkAction* > > So, this patch needs to be integrated in this big if (in the > > MouseReleaseEvent) : > > 509 if (watched != toolBar() && mouseEvent && mouseEvent->button() != > > Qt::MidButton) > > > > > > Andrea Diamantini wrote: > Bookmark thing. Yoann, can you please review and eventually merge this?
:D - Andrea ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103206/#review8397 ----------------------------------------------------------- On Nov. 22, 2011, 6:02 p.m., Furkan Üzümcü wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/103206/ > ----------------------------------------------------------- > > (Updated Nov. 22, 2011, 6:02 p.m.) > > > Review request for rekonq. > > > Description > ------- > > This is a Google Code-in 2011 task. Made by Hüseyin Zengin, GCI student. > > > Diffs > ----- > > src/bookmarks/bookmarkstoolbar.cpp 23af65c > > Diff: http://git.reviewboard.kde.org/r/103206/diff/diff > > > Testing > ------- > > Compiles and works. > > > Thanks, > > Furkan Üzümcü > >
_______________________________________________ rekonq mailing list [email protected] https://mail.kde.org/mailman/listinfo/rekonq
