<br><br><div class="gmail_quote">On Sun, Nov 1, 2009 at 11:32 AM, Fernando Vilas <span dir="ltr"><<a href="mailto:fvilas@iname.com">fvilas@iname.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div><div></div><div class="h5">On Sunday 01 November 2009 08:05:39 you wrote:<br>
> On Sun, Nov 1, 2009 at 4:42 AM, Cristian Oneţ<br>
<<a href="mailto:onet.cristian@gmail.com">onet.cristian@gmail.com</a>>wrote:<br>
> > > <a href="http://websvn.kde.org/?view=rev&revision=1033628" target="_blank">http://websvn.kde.org/?view=rev&revision=1033628</a><br>
> ><br>
> > Was this change really neccessary for the settings UI files?<br>
> > KConfigDialog doesn't seems to work with these widgets. A simple test<br>
> > involves changing the<br>
> > value in a button group and observing that the "Apply" button remains<br>
> > disabled<br>
> > signaling that for the KConfig system nothing has changed.<br>
> ><br>
> > I think this issue is a blocker for the beta release.<br>
><br>
> Also, the initial values are not being filled into the buttons.<br>
> I think we have to revisit this change.<br>
><br>
<br>
</div></div>I'm working on it now. The values need to be read and set. The changes are<br>
not being seen because nothing is connected to the buttonClicked() signal.<br>
<font color="#888888"></font><br></blockquote></div><br>I think in the case of the KConfig dialogs, all that is done by the config code, because I don't remember having to setup anything when creating the forecast dialog. So, I think we should take advantage of that, and in this case revert the change to KButtonGroups.<br>