[go: up one dir, main page]

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

Prevent Espanso Sync Tool window from stealing focus. #1982

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

morgan-dgk
Copy link
Collaborator
@morgan-dgk morgan-dgk commented Jun 6, 2024

fixes #1976

Removing call to xdg_activation should prevent the sync tool window from stealing focus for compositors which support the xdg_activation protocol.

Tested under Hyprland/wlroots and appears to resolve issue, but more testing should be performed before merging.

@morgan-dgk morgan-dgk force-pushed the fix/prevent-sync-tool-stealing-focus branch from 9ec1285 to bd25b6b Compare June 6, 2024 01:40
Removing call to `xdg_activation` should prevent the sync tool
window from stealing focus for compositors which support the
xdg_activation protocol.

Tested under Hyprland/wlroots and appears to resolve issue,
but more testing should be performed before merging.
@morgan-dgk
Copy link
Collaborator Author

I'm unsure why clippy is failing on macos/windows build. No changes have been made there.

@AucaCoyan
Copy link
Member
AucaCoyan commented Jun 12, 2024

Prolly because clippy receives updates every few months and we use stable rust (not pinned). So to keep up we need to fix the "new" lints. This happens to fail when the CI is run with latest rust

@AucaCoyan
Copy link
Member

Hi! I fixed clippy and forgot to tell you 😅.
Can you pull latest dev and push it again? Thank you Morgan!

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.

Switching Layouts restarts SyncTool and takes focus
2 participants