diff options
| author | Elvis Angelaccio <[email protected]> | 2018-06-26 22:13:51 +0200 |
|---|---|---|
| committer | Elvis Angelaccio <[email protected]> | 2018-06-26 22:13:51 +0200 |
| commit | 280c97448141d0685806b46af4c80a354f601f26 (patch) | |
| tree | 7d7000a817687717090f5a984e309373e82493a8 /src/settings/serviceitemdelegate.cpp | |
| parent | 9b7ceaa74da3820f43a8743a4824cf5419466791 (diff) | |
Use collapsible box to configure visible roles
Summary:
A modal dialog that opens another modal dialog is bad UX and should be
avoided whenever possible. These days we have `KCollapsibleGroupBox`
that can be used to show advanced settings without cluttering the
default layout.
This change removes the `AdditionalInfoDialog` and uses a collapsible
groupbox instead. The `Apply` buttons gets enabled whenever a role gets
checked or unchecked, consistently with the existing checkboxes.
Test Plan: Change visible roles in the view properties dialog.
Reviewers: #dolphin, #vdg
Reviewed By: #vdg
Subscribers: ngraham, kfm-devel
Tags: #dolphin
Differential Revision: https://phabricator.kde.org/D13569
Diffstat (limited to 'src/settings/serviceitemdelegate.cpp')
0 files changed, 0 insertions, 0 deletions