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

Fix mouse coordinates on OS X retina displays #24

Merged
merged 1 commit into from
Feb 13, 2014

Conversation

jeremycw
Copy link
Contributor

NSEvent reports mouse coordinates scaled by the pixel density but Godot is currently not expecting this. This causes mouse clicks to happen in unexpected positions. The solution is to report the mouse position scaled up by the - [NSWindow backingScaleFactor] so that Godot gets coordinates in exact pixels like it's expecting.

This is only a stop gap solution until proper HDPI support is added.

godotengine pushed a commit that referenced this pull request Feb 13, 2014
Fix mouse coordinates on OS X retina displays
@godotengine godotengine merged commit 337d98a into godotengine:master Feb 13, 2014
@godotengine
Copy link
Collaborator

merged, thanks!

@draxdeveloper draxdeveloper mentioned this pull request Aug 31, 2015
hungrymonkey referenced this pull request in hungrymonkey/godot Sep 30, 2017
Replaced 'char *' with 'const char *' for consistency with C API
AndreaCatania referenced this pull request in AndreaCatania/godot Jul 15, 2020
Fixed bug when the mesh has duplicate bone names, fixed issue when ex…
AndreaCatania added a commit to GodotECS/godot that referenced this pull request Aug 10, 2021
Add more tests for DataBuffer and handle some corner cases
lawnjelly added a commit to lawnjelly/godot that referenced this pull request Apr 12, 2023
…fy_skele_updates

Canvas item hierarchical culling - notify skeleton changes
YeldhamDev pushed a commit to YeldhamDev/godot that referenced this pull request Dec 5, 2023
Fix some copy-paste mistakes with env vars
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants