Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Work on wider support for detecting display configuration changes #166

Merged
merged 1 commit into from Jan 19, 2013

Conversation

Projects
None yet
3 participants
Contributor

cmsj commented Nov 27, 2012

This is more of a notification than a pull request - I think this Quartz method may provide a more stable and supportable way of updating the layouts when displays are added or removed, but right now it is added on top of your existing detection method.

Assuming the Quartz way is acceptable, I would suggest we remove the existing method entirely and rely fully on Quartz. I would note, however, that I only have Mountain Lion systems to test on, so if this API is unsuitable in earlier versions, some combination will be required!

Extend display reconfiguration detection to work on Mountain Lion. Th…
…is may double-detect on Lion and earlier, I have no such systems to test. I would hope that the Quartz method could become the only method in use.
Owner

jigish commented Nov 27, 2012

thanks for looking at this. i've been busy as of late and have needed to fix this for mountain lion for a while. i'll test this on Lion to see how it works and potentially merge this and remove the other code.

@ghost ghost assigned jigish Nov 27, 2012

arturhoo commented Jan 4, 2013

Any chances of this being accepted? I'm on ML 10.8.2 and monitor changes are not detected.

Owner

jigish commented Jan 5, 2013

yah, I'll accept this and publish a new version sometime this weekend. just came back from a long vacation so I'm a bit spent right now.

jigish added a commit that referenced this pull request Jan 19, 2013

Merge pull request #166 from cmsj/master
Work on wider support for detecting display configuration changes

@jigish jigish merged commit 7e3dc54 into jigish:master Jan 19, 2013

Owner

jigish commented Jan 19, 2013

finally merged. testing now then pushing a new version.

jigish pushed a commit that referenced this pull request Jan 20, 2013

Version 1.0.17
- Merge Pull Request #191 - (Fix Issue #190)
- Merge Pull Request #166 - (Fix Issue #130)
- Merge Pull Request #204
- Merge Pull Request #185

@cmsj cmsj referenced this pull request in Hammerspoon/hammerspoon Sep 8, 2015

Closed

hs.screen.watcher not triggered after hibernate #520

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment