Skip to content

move PApplet from Activity to Fragment #118

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

Merged
merged 19 commits into from
Jul 23, 2015
Merged

move PApplet from Activity to Fragment #118

merged 19 commits into from
Jul 23, 2015

Conversation

omerjerk
Copy link
Contributor

No description provided.

omerjerk added 16 commits June 13, 2015 21:33
Signed-off-by: Umair Khan <omerjerk@gmail.com>
Signed-off-by: Umair Khan <omerjerk@gmail.com>
Signed-off-by: Umair Khan <omerjerk@gmail.com>
Signed-off-by: Umair Khan <omerjerk@gmail.com>
Signed-off-by: Umair Khan <omerjerk@gmail.com>
Signed-off-by: Umair Khan <omerjerk@gmail.com>
Signed-off-by: Umair Khan <omerjerk@gmail.com>
Signed-off-by: Umair Khan <omerjerk@gmail.com>
Signed-off-by: Umair Khan <omerjerk@gmail.com>
Signed-off-by: Umair Khan <omerjerk@gmail.com>
Signed-off-by: Umair Khan <omerjerk@gmail.com>
Signed-off-by: Umair Khan <omerjerk@gmail.com>
Signed-off-by: Umair Khan <omerjerk@gmail.com>
Conflicts:
	src/processing/mode/android/Manifest.java
@@ -854,9 +861,72 @@ private File mkdirs(final File parent, final String name) throws SketchException
}
return result;
}


private void writeMainActivity(final File srcDirectory) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be read from a file. It's too much code to be writing from the PDE.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I too think so.
I'll make the appropriate changes.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@benfry
What do you think should be the appropriate location for this file (MainActivity.java) ?
I would also like to point out that build.xml is also written by the PDE itself and is around twice the size of MainActivity.java. See - https://github.com/processing/processing-android/blob/master/src/processing/mode/android/AndroidBuild.java#L617

@benfry
Copy link
Contributor

benfry commented Jun 26, 2015

How do fragments relate to Android Wear and active wallpapers? Or are those entirely different things as well, and would need their own type of PApplet?

@omerjerk
Copy link
Contributor Author

Our PApplet, as it is, should theoretically work on Android wear. The only deciding factor will be if the hardware supports OpenGL or not.
Supporting live wallpapers will require some change. Live wallpaper is not an activity. It's a GLSurfaceView.

@benfry
Copy link
Contributor

benfry commented Jun 26, 2015

Ok, thanks. Memory just hazy on all those pieces.

Given the changes in the desktop version and the addition of settings(), we might be able to open up more of that stuff.

omerjerk added 2 commits June 27, 2015 00:34
We're dropping the support for gingerbread devices

Signed-off-by: Umair Khan <omerjerk@gmail.com>
@omerjerk
Copy link
Contributor Author

I've removed the support library.
The sketch will now work on post gingerbread devices.

@codeanticode
Copy link
Contributor

Hi Umair, there are some conflicts between this pull request, and the latest revision in master. Can you take a look and update the modifications?

@omerjerk
Copy link
Contributor Author

Sure. Solving the conflicts.

Conflicts:
	src/processing/mode/android/AndroidBuild.java
codeanticode added a commit that referenced this pull request Jul 23, 2015
move PApplet from Activity to Fragment
@codeanticode codeanticode merged commit 7a7bec0 into processing:master Jul 23, 2015
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