Relax constraints for processing QGroupBoxes
ClosedPublic

Authored by vkrause on Apr 8 2017, 8:19 AM.

Details

Reviewers
dfaure
dvratil
Summary

So far this was only accepting QGroupBoxes containing only auto-exclusive buttons. Now we also accept QGroupBoxes that contain non-buttons, as long as the buttons are still all auto-exclusive and direct children of the group box.

This is closer to what KButtonGroup was doing and enables porting of uses of those in kdepim-runtime. Group boxes there contain e.g. additional labels with more explanations.

Test Plan

Tested on the Akonadi mbox resource configuration dialog after a port to QGroupBox.

Diff Detail

Repository
R265 KConfigWidgets
Lint
Lint Skipped
Unit
Unit Tests Skipped
vkrause created this revision.Apr 8 2017, 8:19 AM
Restricted Application added a project: Frameworks. · View Herald TranscriptApr 8 2017, 8:19 AM
Restricted Application added a subscriber: Frameworks. · View Herald Transcript
dvratil accepted this revision.Apr 8 2017, 8:51 AM
This revision is now accepted and ready to land.Apr 8 2017, 8:51 AM
vkrause closed this revision.Apr 8 2017, 9:07 AM