----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125779/#review87425 -----------------------------------------------------------
Looks good to me Do you have commit access? src/filewidgets/kurlnavigatormenu_p.h (line 53) <https://git.reviewboard.kde.org/r/125779/#comment60010> btn -> button - Martin Klapetek On Oct. 25, 2015, 9:23 a.m., Ilia Kats wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125779/ > ----------------------------------------------------------- > > (Updated Oct. 25, 2015, 9:23 a.m.) > > > Review request for KDE Frameworks. > > > Repository: kio > > > Description > ------- > > previously, two clicked() events were emitted upon middle click, one with > Qt::MidButton and one with Qt::LeftButton, resulting in e.g. Dolphin both > opening a new tab as well as changing the location of the current tab. This > fixes that > > > Diffs > ----- > > src/filewidgets/kurlnavigatorbutton.cpp e6aafb7 > src/filewidgets/kurlnavigatorbutton_p.h dbb3e7b > src/filewidgets/kurlnavigatormenu.cpp d492c7c > src/filewidgets/kurlnavigatormenu_p.h 70133d1 > > Diff: https://git.reviewboard.kde.org/r/125779/diff/ > > > Testing > ------- > > Opened Dolphin, tried left/right/middle click on the menu -> behaves as > expected. > > > Thanks, > > Ilia Kats > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel