Page MenuHome GnuPG

qt,qt5: Install and use pinentry icon
ClosedPublic

Authored by TobiasFella on Thu, Mar 28, 2:29 PM.

Details

Diff Detail

Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

TobiasFella created this revision.

I'm wondering whether it wouldn't be better to rename (or copy) the icon to pinentry.png. It's weird if a program installs an icon with a generic name in a generic location.

I suggest: Rename (or copy if we need the icon with the original name) the icon to pinentry.png. The rest looks okay for now.

This revision now requires changes to proceed.Mon, Apr 8, 10:19 AM
TobiasFella retitled this revision from qt,qt5: Install document-encrypt icon for desktop file to qt,qt5: Install and use pinentry icon.
TobiasFella edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Mon, Apr 8, 5:03 PM