----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129405/ -----------------------------------------------------------
Review request for KDE Frameworks, Christoph Feck and David Edmundson. Repository: kwidgetsaddons Description ------- Currently it's not possible to hide widgets in a KCollapsibleGroupBox, because the groupbox hides all chidren on collapse event and shows them on expand event. The rationale was explained in this comment: "when collapsed hide contents to save resources and more importantly get it out the focus chain" The focus chain problem can be solved by overriding/restoring the focus policy of children widgets, without changing their visibility. Diffs ----- src/kcollapsiblegroupbox.cpp 273110e995cb25f28a815cb28125c4678ca2ab28 tests/kcollapsiblegroupboxtest.cpp 5a2900e63e42fa81f909c7d1fff0d07033edb025 Diff: https://git.reviewboard.kde.org/r/129405/diff/ Testing ------- Test app has been expanded with an hidden checkbox * Make sure the new checkbox stays hidden after expanding the groupbox. * With both groups collapsed, make sure the tab-focus goes from the first group to the second one (skipping the 7 checkboxes in the first group). Thanks, Elvis Angelaccio