[rcore] Add GetWindowPosition()
implementation for PLATFORM_WEB
and fixes #3636 style/format
#3637
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes
Adds implementation for
GetWindowPosition()
that was missing forPLATFORM_WEB
. Does that by usingemscripten
EM_ASM_INT
andDOM
window.screenX
andwindow.screenY
(R621-R625).Fixes [rcore] Add
GetMonitorWidth()
andGetMonitorHeight()
implementations forPLATFORM_WEB
#3636 style/format to match the pattern generally used onraylib
. Does that by changingw
towidth
(R576-R578),h
toheight
(R585-R587) and adding relevant notes (R575, R584). My apologies for missing the code style/convention on the previous PR.Notes
GetWindowPosition()
is relative to the current monitor where the browser window is located.Code example
PLATFORM_WEB
(requiresASYNCIFY
) with:Environment
Linux
(Ubuntu 22.04 64-bit) and tested onFirefox
(115.3.1esr 64-bit) andChromium
(117.0.5938.149 64-bit) for bothminshell.html
andshell.html
files.Edits