Be more explicit about action property types
ClosedPublic

Authored by ahiemstra on Oct 8 2019, 1:56 PM.

Details

Summary

Now that Kirigami.Action is a Controls.Action, we can be more explicit
about using the Action type where we store it as a property. This adds a
bit of type safety and additionally allows binding of the action's
properties in more cases.

Test Plan

Run kirigami2gallery, see that things that use actions still work correctly.

Diff Detail

Repository
R169 Kirigami
Branch
explicit_action
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 17446
Build 17464: arc lint + arc unit
ahiemstra created this revision.Oct 8 2019, 1:56 PM
Restricted Application added a project: Kirigami. · View Herald TranscriptOct 8 2019, 1:56 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ahiemstra requested review of this revision.Oct 8 2019, 1:56 PM
apol accepted this revision.Oct 8 2019, 2:18 PM
This revision is now accepted and ready to land.Oct 8 2019, 2:18 PM
This revision was automatically updated to reflect the committed changes.