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

Update README.md with macos build & install tip #268

Closed
wants to merge 1 commit into from

Conversation

Quezion
Copy link
Contributor

@Quezion Quezion commented Jun 22, 2023

Usually I'd bundle this into another PR but the MacOS implementation of bringToFront is ugly and might take a few weeks to land.

@Quezion
Copy link
Contributor Author

Quezion commented Jan 19, 2024

Little update on this stale PR:
I'll clean up the Mac implementation of *isFront this week & get it PR'd, either reusing this one or opening a new one.

The README update in this PR should also be amended to list bringToFront in the large table of implemented features, that way we can callout the missing X11 implementation.

@Quezion
Copy link
Contributor Author

Quezion commented Jan 20, 2024

For some reason, using a local JWM version with publicly released bringToFront code (see merged #269) is failing to correctly steal focus in Mac. The code runs but simply fails to steal focus -- calls to isFront afterwards always returns false. Perhaps this is a permissions issue that the local build works around?

I'm not sure yet but I'll need to PR bringToFront once I figure out a fix. In the meantime, I'm going to close this PR & bundle it into #286 to reduce PR churn.

@Quezion Quezion closed this Jan 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant