-
Notifications
You must be signed in to change notification settings - Fork 2
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
integrate latest changes from Bevy #709
Conversation
a7ec058
to
e51dd20
Compare
Do you know what is the sub-app thingy? |
76d7211
to
7f5455c
Compare
They were introduced as part of bevyengine/bevy#3175 to support their new pipelined renderer I think. |
let inner_size = winit_window.inner_size(); | ||
|
||
// move origin to bottom left | ||
let y_position = f64::from(inner_size.height) - position.y; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this change intentional? I've changed the mouse cursor a couple of weeks ago to use top left as the origin
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably not you can revert back to your change. Mark it with a comment if possible.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this part was not trivial to merge... changes on both sides if I recall.
please reintegrate your changes Kirill, and sorry if I lost something in the process
Integrate latest Bevy changes
Sync to bevyengine/bevy@7356f15