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

Use newer tools/SDK + work w/ API >= 8 (ie, Froyo+) #28

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fat-tire
Copy link

@fat-tire fat-tire commented Sep 1, 2015

Hey guys-- Just found this library a few hours ago and figured I could backport it to work w/Froyo. Gave it a shot! Images below.

Notes:

  • minSDK: 8 / targetSDK: 23
  • No animations before API 11
  • use appcompat/support + minor tweaks to support Froyo+
  • updated gradle plugin, build tools, etc.

The sample app works on Froyo emulator + tested on actual gingerbread device. Could use
verification w/other older devices to make sure nothing broke but seems to work okay.

FROYO (from emulator)
froyo3
froyo2
froyo1

GINGERBREAD (actual device)
gingerbread1
gingerbread2
gingerbread3

* minSDK: 8 / targetSDK: 23
* No animations before API 11
* use appcompat/support + minor tweaks to support Froyo+
* updated gradle plugin, build tools, etc.

The sample app works on Froyo emulator + tested on actual gingerbread device.  Could use
verification w/other older devices to make sure nothing broke but seems to work okay.
@@ -47,7 +47,7 @@ int getSequenceStatus() {

void setSequenceStatus(int status) {
SharedPreferences internal = context.getSharedPreferences(PREFS_NAME, Context.MODE_PRIVATE);
internal.edit().putInt(STATUS + showcaseID, status).apply();
internal.edit().putInt(STATUS + showcaseID, status).commit();
Copy link
Author

Choose a reason for hiding this comment

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

I should mention that these were changed because .apply() didn't show up until API 9.

@ntoskrnl
Copy link

ntoskrnl commented Sep 1, 2015

@deano2390 we are waiting :)

@deano2390
Copy link
Owner

@ntoskrnl Sorry i've been so busy these last few weeks! I will review and merge as soon as I get chance. Thanks for your patience, your contributions to the project are much appreciated :)

@eckovation
Copy link

Hi @deano2390
Any possible update on this?

@fat-tire fat-tire mentioned this pull request Oct 25, 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.

4 participants