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

Fix extra hideWindow() call after retrieving credentials #8647

Conversation

varjolintu
Copy link
Member

@varjolintu varjolintu commented Oct 25, 2022

At least with macOS this extra call sometimes causes the actual browser window to hide instead of KeePassXC after the extension has retrieved the credentials. If the Access Confirm Dialog has been already shown, hideWindow() is already called earlier.

PR with the incorrect change: #8273

This is easy to reproduce with the following steps:

  1. Have some third application window open, KeePassXC in the background
  2. Go to some login page with your main browser window where the extension retrieves the credentials
  3. The main browser window gets hidden after retrieval, activating the third application's window

Testing strategy

Manually.

Type of change

  • ✅ Bug fix (non-breaking change that fixes an issue)

@varjolintu varjolintu added bug pr: backport pending Pull request yet to be backported to a previous release labels Oct 25, 2022
@varjolintu varjolintu added this to the v2.7.4 milestone Oct 25, 2022
@droidmonkey
Copy link
Member

Which PR introduced this behavior?

@varjolintu
Copy link
Member Author

@droidmonkey #8273

@varjolintu
Copy link
Member Author

No longer needed.

@varjolintu varjolintu closed this Oct 26, 2022
@varjolintu varjolintu deleted the fix/browser_hide_window_after_cred_req branch October 26, 2022 08:16
@droidmonkey droidmonkey removed bug pr: backport pending Pull request yet to be backported to a previous release labels Oct 29, 2022
@droidmonkey droidmonkey removed this from the v2.7.4 milestone Oct 29, 2022
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

Successfully merging this pull request may close these issues.

2 participants