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

Added centering of screen for multiple monitors #8

Merged
merged 1 commit into from Apr 18, 2014

Conversation

thorst
Copy link
Collaborator

@thorst thorst commented Apr 18, 2014

Proper pull for supporting multiple monitors. Note that while this fixes the issues for modern browsers, IE is only half fixed.

Old behavior:
No matter the browser, if centered, the pop up would be in the center of the first monitor

New behavior on standards browser:
If centered the popup will be on the center of the monitor which your browser is located.

New behavior on internet explorer browser:
If centered the popup will be on the center of the monitor which your browser is located. BUT, it will base the center coordinates off of your primary monitor. So if your monitor where your browser is located is at a different resolution than your primary, it may be slightly off center.

@thorst thorst mentioned this pull request Apr 18, 2014
lencioni added a commit that referenced this pull request Apr 18, 2014
Add centering of screen for multiple monitors
@lencioni lencioni merged commit 2127f65 into mkdynamic:master Apr 18, 2014
@thorst thorst deleted the multi-moni branch April 21, 2014 01:56
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.

None yet

2 participants