----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100538/#review1258 -----------------------------------------------------------
I added a new patch to better explain my idea. Basically, you need ONE esc to hide the findbar and another one to hide the zoombar... uhm... let me change it again :D - Andrea On Feb. 6, 2011, 9:30 a.m., Yoann Laissus wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/100538/ > ----------------------------------------------------------- > > (Updated Feb. 6, 2011, 9:30 a.m.) > > > Review request for rekonq. > > > Summary > ------- > > When the user defines a custom action for the esc key, the findbar isn't > hidden with this key. This patch solves it. > > > This addresses bug 264929. > /show_bug.cgi?id=264929 > > > Diffs > ----- > > src/mainwindow.h 33fd20212b9bb69d9450298e4e43324885f8bdf0 > src/mainwindow.cpp f662d7aa77d8045372c43514c6a04ea497a9afbc > > Diff: http://git.reviewboard.kde.org/r/100538/diff > > > Testing > ------- > > > Thanks, > > Yoann > >
_______________________________________________ rekonq mailing list [email protected] https://mail.kde.org/mailman/listinfo/rekonq
