<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, Dec 29, 2013 at 4:12 PM, Michal Humpula <span dir="ltr"><<a href="mailto:michal.humpula@seznam.cz" target="_blank">michal.humpula@seznam.cz</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Sunday 29 of December 2013 14:44:59 David Faure wrote:<br>
> On Thursday 19 December 2013 15:28:27 Dominik Haumann wrote:<br>
> > On Thursday, December 19, 2013 15:12:01 Michal Humpula wrote:<br>
> > > On Thursday 19 of December 2013 14:57:33 Dominik Haumann wrote:<br>
> > > > On Wednesday, December 18, 2013 19:42:43 Michal Humpula wrote:<br>
> > > > > Git commit 4e6c1c3d8cd6dbe4d826ce6720169fd94d8848f6 by Michal<br>
> > > > > Humpula.<br>
> > > > > Committed on 18/12/2013 at 19:42.<br>
> > > > > Pushed by michalhumpula into branch 'frameworks'.<br>
> > > > ><br>
> > > > > port from KDialog to QDialog (KateGlobal::configDialog)<br>
> > > > ><br>
> > > > > M  +1    -4    part/utils/kateglobal.cpp<br>
> > > > > M  +2    -1    part/view/kateview.cpp<br>
> > > > ><br>
> > > > > <a href="http://commits.kde.org/kate/4e6c1c3d8cd6dbe4d826ce6720169fd94d8848f6" target="_blank">http://commits.kde.org/kate/4e6c1c3d8cd6dbe4d826ce6720169fd94d8848f6</a><br>
> > > > ><br>
> > > > > diff --git a/part/utils/kateglobal.cpp b/part/utils/kateglobal.cpp<br>
> > > > > index e576f12..94de177 100644<br>
> > > > > --- a/part/utils/kateglobal.cpp<br>
> > > > > +++ b/part/utils/kateglobal.cpp<br>
> > > > > @@ -310,10 +310,7 @@ void KateGlobal::configDialog(QWidget *parent)<br>
> > > > > -#if 0 //FIXME KF5<br>
> > > > > -  kd->setButtons( KDialog::Ok | KDialog::Cancel | KDialog::Apply |<br>
> > > > > KDialog::Help ); -  kd->setHelp( QString(),<br>
> > > > > KGlobal::mainComponent().componentName() ); -#endif<br>
> > > > > +  kd->setStandardButtons(QDialogButtonBox::Ok |<br>
> > > > > QDialogButtonBox::Cancel | QDialogButtonBox::Apply |<br>
> > > > > QDialogButtonBox::Help );> ><br>
> > > ><br>
> > > > If I'm not mistaken, we can use setStandardButtons() in a lot of other<br>
> > > > places as well instead of adding the respective buttons manually.<br>
> > > ><br>
> > > > Is that correct?<br>
> > > ><br>
> > > > If so, we really should do that, to always keep the correct button<br>
> > > > order<br>
> > > > (e.g. Cancel on the very right or similar).<br>
> > > ><br>
> > > > Greetings,<br>
> > > > Dominik<br>
> > ><br>
> > > The drawback is that currently the buttons are not styled the KDE way:(<br>
> > > I'm guessing this will be fixed in the future in frameworks QPA?<br>
> ><br>
> > Hm, no idea. CCing frameworks-devel.<br>
><br>
> [did you forget to CC Michal too?]<br>
><br>
> What does "styled" mean here exactly?<br>
> The icons on the buttons are missing?<br>
<br>
</div></div>Yes, icons and tooltips. I'm guessing whatever the KStandardGuiItem setups<br>
too. So should the buttons created by setStandardButtons be restyled manually<br>
or should the platform take care of it?<br>
<br>
Attaching screenshot, how it looks for me. Buttons with icons are manually<br>
constructed.<br>
<div class="HOEnZb"><div class="h5"><br>
> > > Sometimes the buttons have different text or icon then standard, so the<br>
> > > need for creating buttons manually is still there, imho.<br>
> ><br>
> > That's fine: You can still get the respective button with:<br>
> >   QPushButton * QDialogButtonBox::button(StandardButton which);<br>
> ><br>
> > And then change the icon or text.<br>
><br>
> Yes.<br>
><br>
> > But this way, the order at least is<br>
> > kind of automatically set. I'm not sure though, whether this is of<br>
> > importance in all cases.<br>
><br>
> Yes, that's the whole point of QDialogButtonBox.<br>
</div></div><br>_______________________________________________<br>
Kde-frameworks-devel mailing list<br>
<a href="mailto:Kde-frameworks-devel@kde.org">Kde-frameworks-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kde-frameworks-devel" target="_blank">https://mail.kde.org/mailman/listinfo/kde-frameworks-devel</a><br>
<br></blockquote></div></div><div class="gmail_extra"><br></div><div class="gmail_extra">We definitely need QDialogButtonBox to look properly in KDE, so if that's not the case at the moment, we'll have to consider it a bug and investigate it.</div>

<div class="gmail_extra"><br></div><div class="gmail_extra">Aleix</div></div>