[desktoppackage/ConfigurationShortcuts] Specify spacing on ColumnLayout and use strict === equality
Specify spacing on ColumnLayout
There's only a 1px difference between default (5px) and smallSpacing (4px), but this is what should always be explicitly specified using Units constants.
Use strict === equality in JavaScript
The property var keySequence
has a value of undefined, empty string,
or a valid QKeySequence gadget type.
Once frameworks/kdeclarative!108 (merged) lands, keySequence will be QKeySequence at all times. Comparing it via === in JavaScript has semantics of checking operands for being literally the same object (as in: memory address). I'm not sure how that semantics apply to gadgets which are supposed to be copied on every occasion, but in the worst case scenario they'll just always be considered non-equal, and configuration would be updated every time, which does no harm.
CC: @mart @davidre @carlschwan