> On Ян. 27, 2017, 7:01 преди обяд, 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 > > Albert Astals Cid wrote: > 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?
So when i know allMenuBarsNative == true, wasNative == true, why you think that i'm not uderstainding? I'm against to store / restore menubar chacked and visabiliity. - Anthony ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129892/#review102269 ----------------------------------------------------------- On Ян. 27, 2017, 12:19 преди обяд, Albert Astals Cid wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/129892/ > ----------------------------------------------------------- > > (Updated Ян. 27, 2017, 12:19 преди обяд) > > > 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 > >
