> On Jan. 27, 2017, 5:01 a.m., Anthony Fieroni wrote: > > So, wait a minute i just quote you > > *This patch attempts to fix it by iterating all the main windows when they > > are shown and if all the menubars of all mainwindows are native, then hides > > the show menu bar action (basically erasing it from existence).* from > > https://git.reviewboard.kde.org/r/129261/ > > *This has nothing to do with whether you are using Unity or Plasma* from > > https://git.reviewboard.kde.org/r/129885/ > > I want an explanation, i never use Unity, i will never use it, but i want > > know in Unity - Kate, Dolphin etc. on showEvent allMenuBarsNative == true, > > wasNative == true ? > > If so, i'm -1 for this approach
You don't seem to understand the code, what makes you think you're able to give a -1 when you don't understand the code? You want an explanation? What makes you think you should be given one? You're not the maintainer of the code, you have not contributed at all to kconfigwidgets and on top of that you've been nothing other than agressive both in this and in the other review request. Still i'll try to explain it to you, but don't think your attitude is helping, it's not. > but i want know in Unity - Kate, Dolphin etc. on showEvent allMenuBarsNative > == true, wasNative == true ? This depends if it's the first show or not. On the first show obviously wasNative will be false since it is how it is initialized in the constructor. So on the first show it'll be allMenuBarsNative == true, wasNative == false and on the second show, yes it will be allMenuBarsNative == true, wasNative == true. I sincerely think the code is quite easy to understand, what you don't understand? - Albert ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129892/#review102269 ----------------------------------------------------------- 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 > >
