Skip to content

Allow to use the user/system theme icons for actions. #756

@ghost

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.

Before:
screenshot_20170706_183114

After:
screenshot_20170706_184646

This is the diff file:
icons.txt

KeePassXC - 2.2.0
Operating system: KDE Neon (Ubuntu 16.04)

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions