[Clipboard Plasmoid] Port to Prison QML import
ClosedPublic

Authored by broulik on May 6 2020, 10:54 AM.

Details

Summary

Which does the rendering and proper sizing for us, e.g. ensure integer scaling so the barcode remains scannable.
While at it, fix supportsBarcode (wasn't routed through to the delegate component when it was split out) and add Code 128 which is in Prison now.
Also, show a label when the barcode would become too large to display (mostly for the 1D barcodes).
The Prison item does not use a thread for generating the barcode like the dataengine does but it's super fast.

Test Plan



Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.May 6 2020, 10:54 AM
Restricted Application added a project: Plasma. · View Herald TranscriptMay 6 2020, 10:54 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.May 6 2020, 10:54 AM
broulik edited the summary of this revision. (Show Details)
broulik edited the summary of this revision. (Show Details)
broulik edited the test plan for this revision. (Show Details)May 6 2020, 10:58 AM
ngraham added a subscriber: ngraham.May 6 2020, 2:25 PM

+1, very nice.

Technically this looks fine from the Prison POV. Whether the barcode type selection makes sense beyond "because we can" I'm not sure about though (in particular the 1D codes seem to be of questionable use here), but then again that's not something introduced by this patch anyway.

broulik edited the test plan for this revision. (Show Details)May 7 2020, 12:36 PM
vkrause accepted this revision.May 26 2020, 3:32 PM
This revision is now accepted and ready to land.May 26 2020, 3:32 PM
This revision was automatically updated to reflect the committed changes.