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

Added OpenGL3 example for SDL2 #356

Merged
merged 1 commit into from
Nov 16, 2015
Merged

Added OpenGL3 example for SDL2 #356

merged 1 commit into from
Nov 16, 2015

Conversation

zach2good
Copy link
Contributor

Previous SDL implementation didn't work for me once I started using modern SDL code (it worked if I ripped out all my rendering code).
Spliced the GLFW OpenGL examples on top of the SDL example to create an option that works for me, could be useful for others.

screenshot

…pliced the GLFW OpenGL examples on top of the SDL example to create an option that works for me. Could be useful for others.
@ocornut
Copy link
Owner

ocornut commented Sep 30, 2015

Thanks! could you clarify what wouldn't work with the GL2 code?

@zach2good
Copy link
Contributor Author

I'm still new to openGL so this might be anecdotal evidence. When I had nothing else in my project apart from the IMGUI and SDL2 code, it would render fine. As soon as I started adding OGL3.3 rendering code into my program the gui disappeared. If I remember correctly OpenGL 3 onwards depreciated lots of OpenGL 2 rendering stuff, so the old 2-based code wouldn't render in a more modern context.

@ocornut ocornut merged commit e130b09 into ocornut:master Nov 16, 2015
ocornut added a commit that referenced this pull request Nov 16, 2015
ocornut added a commit that referenced this pull request Nov 16, 2015
@ocornut
Copy link
Owner

ocornut commented Nov 16, 2015

Thanks, I have now merged this & done a few fixes to match the existing GL3 example code.

@zach2good
Copy link
Contributor Author

Cool, happy to help!

ocornut added a commit that referenced this pull request Jul 30, 2016
…les (#323)

Missing support Vulkan (#549), Apple (#575, #247), SDL (#58, #356),
Allegro, Marmalade (#368, #375)
ocornut added a commit that referenced this pull request Sep 18, 2017
…les (#323)

Missing support Vulkan (#549), Apple (#575, #247), SDL (#58, #356),
Allegro, Marmalade (#368, #375)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants