D12686: Fix sidebar view regression of not prompting about unsaved changes

2018-05-14 Thread David Edmundson
This revision was automatically updated to reflect the committed changes. Closed by commit R124:0aa7fd015b0c: Fix sidebar view regression of not prompting about unsaved changes (authored by davidedmundson). REPOSITORY R124 System Settings CHANGES SINCE LAST UPDATE https://phabricator.kde.org

D12686: Fix sidebar view regression of not prompting about unsaved changes

2018-05-04 Thread David Edmundson
davidedmundson updated this revision to Diff 33629. davidedmundson added a comment. Remove debug stupid arc. Spend ages selectively choosing the hunks I want, and then it goes and adds them back anyway to "help". REPOSITORY R124 System Settings CHANGES SINCE LAST UPDATE https://ph

D12686: Fix sidebar view regression of not prompting about unsaved changes

2018-05-04 Thread David Edmundson
davidedmundson updated this revision to Diff 33628. davidedmundson added a comment. Remove debug stupid arc. Spend ages selectively choosing the hunks I want, and then it goes and adds them back anyway to "help". REPOSITORY R124 System Settings CHANGES SINCE LAST UPDATE https://p

D12686: Fix sidebar view regression of not prompting about unsaved changes

2018-05-04 Thread David Edmundson
davidedmundson updated this revision to Diff 33627. davidedmundson added a comment. Also fixes the UI on cancel Checkable property would be updated internally on clicked, breaking our binding and becoming checked on clicked regardless The background color is set by if (checked || hig

D12686: Fix sidebar view regression of not prompting about unsaved changes

2018-05-03 Thread Kai Uwe Broulik
broulik accepted this revision. broulik added a comment. This revision is now accepted and ready to land. The current index or highlighted item in the sidebar still changes if you Cancel in the prompt as that's done from QML REPOSITORY R124 System Settings BRANCH master REVISION DETAIL

D12686: Fix sidebar view regression of not prompting about unsaved changes

2018-05-03 Thread David Edmundson
davidedmundson created this revision. davidedmundson added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. davidedmundson requested review of this revision. REVISION SUMMARY When you switch module with unsaved changes t