Remove the DataMatrix code, just the QR code is enough
ClosedPublic

Authored by vkrause on Apr 7 2019, 9:51 AM.

Details

Summary

QR codes are much more widely used (and therefore recognized) and are more
compact, so the DataMatrix one is just taking up more space and makes this
look ugly.

Diff Detail

Repository
R204 KAddressBook
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
vkrause created this revision.Apr 7 2019, 9:51 AM
Restricted Application added a project: KDE PIM. · View Herald TranscriptApr 7 2019, 9:51 AM
Restricted Application added a subscriber: kde-pim. · View Herald Transcript
vkrause requested review of this revision.Apr 7 2019, 9:51 AM

Shouldn't the code that generates the datamatrix data also removed? Or does that lives elsewhere?

Yep, that's in akonadi-contacts of course...

dvratil accepted this revision.Apr 7 2019, 12:41 PM
This revision is now accepted and ready to land.Apr 7 2019, 12:41 PM
This revision was automatically updated to reflect the committed changes.