-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Update RGFW #4176
Update RGFW #4176
Conversation
@ColleagueRiley Thanks for the update! |
CORE.Window.display.width = screenSize.w; | ||
CORE.Window.display.height = screenSize.h; | ||
/* | ||
I think this is needed by Raylib now ? |
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.
Why?
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.
I saw it in the glfw platform, I'm not sure if it's an init function Raylib requires or not.
@@ -643,7 +645,7 @@ Vector2 GetWindowScaleDPI(void) | |||
{ | |||
RGFW_monitor monitor = RGFW_window_getMonitor(platform.window); | |||
|
|||
return (Vector2){((u32)monitor.scaleX)*platform.window->r.w, ((u32) monitor.scaleX)*platform.window->r.h}; | |||
return (Vector2){monitor.scaleX, monitor.scaleX}; |
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.
Not following raylib coding conventions, a space after {
and before }
should be used
@@ -483,14 +483,14 @@ void SetWindowIcons(Image *images, int count) | |||
// Set title for window | |||
void SetWindowTitle(const char *title) | |||
{ | |||
RGFW_window_setName(platform.window, title); | |||
RGFW_window_setName(platform.window, (char*)title); |
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.
Not following raylib coding conventions, raylib uses (char *)
// Set cursor position in the middle | ||
SetMousePosition(CORE.Window.screen.width/2, CORE.Window.screen.height/2); | ||
|
||
RGFW_window_mouseHold(platform.window, RGFW_AREA(CORE.Window.screen.width / 2, CORE.Window.screen.height / 2)); |
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.
Not following raylib coding conventions, raylib does not use spaces between *
and /
operators
Added some minor formatting comments, no worries, I can review them. |
A couple of updates and bug fixes for RGFW and rcore_desktio_RGFW.c