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

Continuation of support for ES3/WebGL2 #3107

Merged
merged 3 commits into from
Jun 12, 2023
Merged

Conversation

chemaguerra
Copy link
Contributor

@chemaguerra chemaguerra commented Jun 11, 2023

This is in response to:

I continued the work that @raysan5 started for OpenGL ES 3.0 context creation:

1- Fixed #define GRAPHICS_API_OPENGL_33.
2- Adapted rlActiveDrawBuffers() which now can be used to successfully initialize MRTs in WebGL2.
3- Did a (dirty) update to RLGL.ExtSupported.* since some ES 2.0 feature extensions can be taken for granted in ES 3.0.
4- Adapted rlGetGlTextureFormats() to support floating-point textures in ES 3.0.

5- I also edited the Makefile so raylib builds for ES3 (maybe exclude this on PR merge...).

The changes I am PR'ing have been tested in a couple of experimental projects I am conducting at the moment, in a bunch of web browsers across multiple devices (Windows, MacOS, Android, iPad, iPhone).

(3) needs some work to correctly detect the capabilities of the device. i.e., whether the device supports linear filtering for floating-point textures (many devices don't).

src/Makefile Outdated Show resolved Hide resolved
src/Makefile Outdated Show resolved Hide resolved
src/raylib.h Outdated Show resolved Hide resolved
@raysan5
Copy link
Owner

raysan5 commented Jun 11, 2023

@chemaguerra Thank you very much for the addition and the testing! Just added a couple of comments.

@chemaguerra
Copy link
Contributor Author

I made some amends following your comments.

@raysan5 raysan5 merged commit f385d0c into raysan5:master Jun 12, 2023
@raysan5
Copy link
Owner

raysan5 commented Jun 12, 2023

@chemaguerra thanks for the review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants