[Containment Actions Config] Cleanup layout and support multiple modifiers
ClosedPublic

Authored by broulik on Aug 31 2017, 9:50 AM.

Details

Summary

Currently we only manage to display one modifier for each action but you can setup multiple (which work fine, it's just the UI).
While at it, port it to a GridLayout to ensure action buttons adjust their width and don't blow up the layout or truncate.

Test Plan

Diff Detail

Repository
R119 Plasma Desktop
Lint
Lint Skipped
Unit
Unit Tests Skipped
broulik created this revision.Aug 31 2017, 9:50 AM
Restricted Application added a project: Plasma. · View Herald TranscriptAug 31 2017, 9:50 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart added a subscriber: mart.Sep 1 2017, 9:43 AM

how does it look when all texts in the buttons at the left are short?

broulik updated this revision to Diff 19039.Sep 1 2017, 10:09 AM

Add Binding for configurationChanged and remove explicit calls now that D7626 landed

mart accepted this revision.Sep 8 2017, 11:58 AM
This revision is now accepted and ready to land.Sep 8 2017, 11:58 AM
This revision was automatically updated to reflect the committed changes.