Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Version 2.4.0 breaks theming on GNU+Linux, not fixed in 2.4.2 #3231

Closed
psvenk opened this issue Jun 5, 2019 · 5 comments
Closed

Version 2.4.0 breaks theming on GNU+Linux, not fixed in 2.4.2 #3231

psvenk opened this issue Jun 5, 2019 · 5 comments

Comments

@psvenk
Copy link

psvenk commented Jun 5, 2019

I am running the latest version of KeePassXC on Arch Linux, installed by pacman from the official repositories. After updating to 2.4.0, I saw that KeePassXC stopped using its own icons in some places but instead used the system icon theme, which made the interface inconsistent and ambiguous. I waited until 2.4.2 because #3057 seems to have fixed it for Snap packages, but I continue to face this issue after upgrading to 2.4.2. All of my packages are up to date. This issue is a continuation of #3057 (comment).

Expected Behavior

KeePassXC either uses its own icons everywhere, or detects the system icon theme and uses icons that look native to that icon theme but are not ambiguous or inconsistent.

Current Behavior

I tested with the following icon themes:

  • Bluecurve (my default, and the default on Fedora many years ago)
  • Adwaita (GNOME 3's default)
  • GNOME (GNOME 2's default)
  • Breeze (KDE Plasma's default)
  • Humanity (Ubuntu's default)

With each of these icon themes, with the exception of Adwaita, KeePassXC's icons were inconsistent or ambigous. I will now go in depth about the issues that I noticed.

  • In Bluecurve, GNOME 2, and Humanity, the "lock database" icon is a generic file icon instead of a lock icon.
  • In Bluecurve, Breeze, and Humanity, the "copy username", "copy password", and "copy URL" icons are not consistent.
  • In Humanity, the "search" icon is a computer icon instead of a magnifying glass.
  • In Bluecurve and Humanity, "copy password" and "password generator" use the same system icon.

Bluecurve:
Bluecurve

Adwaita:
Adwaita

GNOME 2:
GNOME 2

Breeze:
Breeze

Humanity:
Humanity

Possible Solution

See Expected Behavior.

Steps to Reproduce

  1. sudo pacman -R keepassxc
  2. sudo pacman -S keepassxc

(or the equivalent commands for your package manager)

Debug Info

KeePassXC - Version 2.4.2
Revision: a775031

Qt 5.12.3
Debugging mode is disabled.

Operating system: Arch Linux
CPU architecture: x86_64
Kernel: linux 5.1.6-arch1-1-ARCH

Enabled extensions:

  • Auto-Type
  • Browser Integration
  • SSH Agent
  • KeeShare (signed and unsigned sharing)
  • YubiKey

Cryptographic libraries:
libgcrypt 1.8.4

@psvenk psvenk added the bug label Jun 5, 2019
@psvenk psvenk changed the title Version 2.4.1 breaks theming on GNU+Linux, not fixed in 2.4.2 Version 2.4.0 breaks theming on GNU+Linux, not fixed in 2.4.2 Jun 5, 2019
@phoerious
Copy link
Member

We have always relied on the system theme, nothing has changed here. If certain icons look different, then its because they are missing in your system's icon theme. We try to use the icons that work best, but sometimes themes have very different icons than others, which we cannot do anything about.

@phoerious phoerious added wontfix and removed bug labels Jun 5, 2019
@droidmonkey
Copy link
Member

droidmonkey commented Jun 6, 2019

I think this warrants some debugging to understand why some themes are giving wildly different icon "types" for the requested icon. We may not be able to fix it, but it's worth understanding.

@droidmonkey droidmonkey reopened this Jun 6, 2019
@psvenk
Copy link
Author

psvenk commented Jun 6, 2019

As to @phoerious's comment saying that KeePassXC has always behaved like this, here is a screenshot of 2.3.4 on my system with the Bluecurve icon theme (from my pacman cache, but originally from the official Arch repositories):

Screenshot at 2019-06-05 21-57-05

The icons that are commonly found in icon themes (new file, open, save) are the system ones, while the ones that are unlikely to be in the icon theme are the ones provided by KeePassXC. I'm not sure what caused the change in 2.4.0.

@bungabunga
Copy link

bungabunga commented Jun 8, 2019

Exactly the same behaviour here on Ubuntu 19.04. That difference between 2.3.4. and 2.4.0.

@droidmonkey
Copy link
Member

droidmonkey commented Jun 8, 2019

The only PR I can think of that impacted the icons for 2.4.0 is this one: #2113

Either way, eventually we will be doing this: #475

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants