Skip to content

fix: use bottom-left format for return value of GetMousePosition#198

Merged
ripel2 merged 3 commits intomainfrom
162-getmouseposition-return-a-pos-in-a-bad-ndc-glfw-rather-than-opengl
Apr 23, 2025
Merged

fix: use bottom-left format for return value of GetMousePosition#198
ripel2 merged 3 commits intomainfrom
162-getmouseposition-return-a-pos-in-a-bad-ndc-glfw-rather-than-opengl

Conversation

@Miou-zora
Copy link
Contributor

Related to:

I moved this function into Window class because we want to get the glfw window and it would be not convenient to send core or Window class into the method

@Miou-zora Miou-zora added the bug Something isn't working label Apr 23, 2025
@Miou-zora Miou-zora requested a review from a team April 23, 2025 03:26
@Miou-zora Miou-zora self-assigned this Apr 23, 2025
@Miou-zora Miou-zora linked an issue Apr 23, 2025 that may be closed by this pull request
@ripel2 ripel2 enabled auto-merge (squash) April 23, 2025 14:46
@sonarqubecloud
Copy link

@ripel2 ripel2 merged commit f4bc2a8 into main Apr 23, 2025
11 checks passed
@ripel2 ripel2 deleted the 162-getmouseposition-return-a-pos-in-a-bad-ndc-glfw-rather-than-opengl branch April 23, 2025 14:55
MasterLaplace pushed a commit that referenced this pull request Apr 29, 2025
Related to:
- #162 

I moved this function into Window class because we want to get the glfw
window and it would be not convenient to send core or Window class into
the method

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GetMousePosition return a pos in a bad NDC (glfw rather than opengl)

2 participants