On Dienstag, 2. Mai 2017 11:30:04 CEST Albert Astals Cid wrote: > El dilluns, 1 de maig de 2017, a les 20:31:28 CEST, Martin Koller va escriure: > > Hi, > > > > trying to fix https://bugs.kde.org/show_bug.cgi?id=378930 > > > > I see that the mboxresource has a configuration dialog which uses > > KConfigDialogManager to handle the settings, but it has a QGroupBox widget > > holding QRadioButtons, which represent one setting (e.g. one of 2 possible > > compaction methods). > > > > In the .ui file, the QGroupBox(!) has the name kcfg_CompactFrequency > > but I assume the bug lies in the fact that this stems from Qt4 days, where > > the groupBox did also manage the buttons, whereby now you need a > > QButtonGroup class. > > > > Am I correct and if so, what needs to be changed that KConfigDialogManager > > works again with this radio buttons ? > > Is KConfigDialogManager designed so that it can work with a QButtonGroup and > > a set of radio buttons ? > > Yes, see > https://cgit.kde.org/kconfigwidgets.git/tree/src/kconfigdialogmanager.cpp#n202
Debugging into this, I see that it does not work here since the QGroupBox contains 2 radio buttons and one spinbox, but the kconfigdialogmanager.cpp checks for ... wait https://cgit.kde.org/kconfigwidgets.git/commit/src/kconfigdialogmanager.cpp?id=7db9744511373f74c28c0f40545677c9bd1b3b07 seems to solve the problem already -- Best regards/Schöne Grüße Martin A: Because it breaks the logical sequence of discussion Q: Why is top posting bad? () ascii ribbon campaign - against html e-mail /\ - against proprietary attachments Geschenkideen, Accessoires, Seifen, Kulinarisches: www.lillehus.at
