----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129892/#review102263 -----------------------------------------------------------
Ship it! Ship It! - David Faure On Jan. 26, 2017, 10:19 p.m., Albert Astals Cid wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/129892/ > ----------------------------------------------------------- > > (Updated Jan. 26, 2017, 10:19 p.m.) > > > Review request for KDE Frameworks, Anthony Fieroni and David Faure. > > > Repository: kconfigwidgets > > > Description > ------- > > It makes no sense to treat no menubars as all menubars native so just bail > early, the app is probably still in construction if there's no menubars but > it actually has a showmenubaraction. > > > Diffs > ----- > > src/kstandardaction.cpp 6e5bba7 > > Diff: https://git.reviewboard.kde.org/r/129892/diff/ > > > Testing > ------- > > Dolphin behaves better regarding the show menubar action. > > > Thanks, > > Albert Astals Cid > >