-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Allow to use the user/system theme icons for actions. #756
Description
Hi, i installed the .deb for the version 2.2.0 (i used to use the mono-based keepass2) and noticed some icons on the toolbar and menu used my user icon theme (breeze-dark) and others don't, for example the "document-new" icon uses the breeze theme but the "entry-new" don't. Then i downloaded the code just for curiosity and by deleting a "false" argument on the calls to the function that loads the icon i made all the action icons in that file to use my user/system theme. I just changed lines like:
m_ui->actionChangeMasterKey->setIcon(filePath()->icon("actions", "database-change-key", false));
to:
m_ui->actionChangeMasterKey->setIcon(filePath()->icon("actions", "database-change-key"));
Because the default KDE theme seems to cover well most if not all of the icons i changed i think it looks better now. So i mostly opened this bug to ask if there's any problem to allow the program to load the icons from the user theme at least on linux for future releases. Thanks in advance for your attention.
This is the diff file:
icons.txt
KeePassXC - 2.2.0
Operating system: KDE Neon (Ubuntu 16.04)

