https://bugs.kde.org/show_bug.cgi?id=524138

            Bug ID: 524138
           Summary: Changing of global shortcuts session startup behavior
                    breaks compatibility
    Classification: Plasma
           Product: xdg-desktop-portal-kde
      Version First 6.7.4
       Reported In:
          Platform: Fedora RPMs
                OS: Linux
            Status: REPORTED
          Severity: normal
          Priority: NOR
         Component: general
          Assignee: [email protected]
          Reporter: [email protected]
                CC: [email protected]
  Target Milestone: ---

DESCRIPTION

The change here is problematic for backwards compatibility. We just got a bug
report that the shortcuts aren't bound anymore in our pre-release build that
has a lot of quirks for different portals and versions.

I used to use ListShortcuts to detect if I need to call BindShortcuts on Plasma
so that the shortcuts are bound on app startup. If I implicitly called
BindShortcuts on Plasma it would always show the shortcut binding dialog on any
older Plasma release which are widely in use.

Either ListShortcuts needs to return an empty list before BindShortcuts to make
our behavior work on any Plasma release (and we don't have a bug in KeePassXC)
or I'm using the portal the wrong way which causes BindShortcuts to always open
the shortcuts dialog and there's a fix that we can apply before we ship our
final release that works for any Plasma release and other portal
implementations too.

Any advice as I'm currently looking at the fix for this issue as causing a
regression?

STEPS TO REPRODUCE
1. Start KeePassXC 2.8.0 snapshot AppImage on previous Plasma releases and
latest Plasma release
2. Configure shortcut
3. Restart KeePassXC

OBSERVED RESULT
Shortcut isn't bound because we use ListShortcuts to detect if we need to call
BindShortcuts.

EXPECTED RESULT
Shortcut is bound.

ADDITIONAL INFORMATION

This issue was created from the comment I wrote yesterday in the issue below as
I'm unsure if resolved issues are followed by developers and fixing this is
kind of important as the change was shipped to a point release that is already
supplied by distributions and we're going to get more bug reports about this
when it comes more widespread.

https://bugs.kde.org/show_bug.cgi?id=523063

Downstream bug: https://github.com/keepassxreboot/keepassxc/issues/13578

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to