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

iOS support #50

Open
felixfaire opened this issue Mar 1, 2016 · 4 comments
Open

iOS support #50

felixfaire opened this issue Mar 1, 2016 · 4 comments

Comments

@felixfaire
Copy link

Is there any reason why the block does not officially support ios. I have just made a small change to the block.xml in my local version to enable iOS projects but i would be interested to know if there are any reasons why it is not already supported.

Thanks

@vtron
Copy link
Contributor

vtron commented Mar 1, 2016

Hi Felix,

Honestly we did not include it because we did not have the time to test on iOS...in theory it should work since we're relying completely on Cinder for this.

We use this library for our projects and we have not had the need for an OpenGL interface on iOS recently, but we can totally see the value.

If you can confirm that it works I'd be happy to accept a pull request on the cinderblock.xml revisions

@lab101
Copy link

lab101 commented Mar 16, 2016

Just started a project on iOS with poScene and had to fix a thing for the hittesting.
Found out that the viewport was double the size.
Had to change the following function void Node::setTransformation()


        const float scale =  ci::app::getWindow()->getContentScale();
        ci::Area viewport = ci::Area(ci::gl::getViewport().first, ci::gl::getViewport().second);

        viewport.x2 /= scale;
        viewport.y2 /= scale;

        mMatrix.set(ci::gl::getModelMatrix(), ci::gl::getProjectionMatrix(), viewport );

@vtron
Copy link
Contributor

vtron commented Mar 17, 2016

Hey @lab101 thanks for the catch, makes sense!

Do you want to issue a pull request? Otherwise I can update it on our side and push.

@lab101
Copy link

lab101 commented Mar 17, 2016

Hey Steve,

It was a quick fix feel free to change it into something better or faster.
Otherwise feel to copy this.

Greets
Kris

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

No branches or pull requests

3 participants