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

dev: When displaying the query window, the settings behind it is also displayed #362

Closed
tisfeng opened this issue Jan 24, 2024 · 1 comment
Assignees
Labels
bug Something isn't working fixed in next release The issue will be closed once next release is available

Comments

@tisfeng
Copy link
Owner

tisfeng commented Jan 24, 2024

It's a known issue that when a query window is displayed, such as the mini window, it also displays the settings page, or the main window if it's turned on, along with it.

Since I'm not familiar with macOS window management, I haven't found a good solution for this, so my previous workaround was: when displaying a query window, if the settings page is displayed, close the settings page first. If the main window is also displayed, orderOut:, then orderBack:.

This solution is ugly, but it can be used temporarily if there is no better solution.

This is old code:

if (preferencesWindowController.isShowing) {

iShot_2024-01-24_21.45.59.mp4
@tisfeng tisfeng added the dev label Jan 24, 2024
@tisfeng tisfeng added fixed in next release The issue will be closed once next release is available and removed dev labels Feb 18, 2024
@phlpsong phlpsong self-assigned this Feb 25, 2024
@tisfeng
Copy link
Owner Author

tisfeng commented Feb 25, 2024

2.6.1 版本已修复该问题。

@tisfeng tisfeng closed this as completed Feb 25, 2024
@tisfeng tisfeng added the bug Something isn't working label Feb 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed in next release The issue will be closed once next release is available
Projects
None yet
Development

No branches or pull requests

2 participants