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

glscreenshot added. #2

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

glscreenshot added. #2

wants to merge 2 commits into from

Conversation

croath
Copy link

@croath croath commented Aug 2, 2013

@croath
Copy link
Author

croath commented Aug 2, 2013

sorry for using my girlfriend's git info 🍭

@mRs-
Copy link
Owner

mRs- commented Aug 4, 2013

it would be better if the normal screenshot function automatically detects if the view contains OpenGL data. Maybe you are using a third party library and you are not aware if there is OpenGL inside the view.

@croath
Copy link
Author

croath commented Aug 4, 2013

If there's a subview rendered with openGL, normal screen capture will return an image with a black area where the OpenGL view should be.

So some image editing may be helpful.

I will think about it.

@mRs-
Copy link
Owner

mRs- commented Aug 5, 2013

Does the subview with the OpenGL inside respond to a OpenGL only responder? We could try it with this.

@mRs-
Copy link
Owner

mRs- commented Aug 26, 2013

Are you still on this issue?

@croath
Copy link
Author

croath commented Aug 26, 2013

i just finished my summer vacation several days ago...

yep i am here, back to work again!

@croath
Copy link
Author

croath commented Sep 4, 2013

no good idea for the view contains multiple OpenGL views, at least, too complex.

but i got an idea for some limited conditions : after iOS 7 beta 4, a public sreenshot API is here.

@mRs-
Copy link
Owner

mRs- commented Sep 4, 2013

yeah i plan to implement the public screenshot api in this library for auto check if you are using iOS7

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.

3 participants