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

Fix issue with compiling using API 21 #9

Merged
merged 7 commits into from Feb 17, 2015
Merged

Conversation

dandc87
Copy link
Contributor

@dandc87 dandc87 commented Feb 14, 2015

Copied a lot of work from the Calligraphy project to fix issue #7.

It also appears that wrapping the Context multiple times no longer works, so to use Calligraphy in conjunction a work around was created.

Example project now uses the AppCompat library.

*/
private void setUpLayoutFactories() {
// If we are HC+ we get and set Factory2 otherwise we just wrap Factory1
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.HONEYCOMB) {
Copy link
Contributor

Choose a reason for hiding this comment

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

we are using minSdkVersion 14. should this be lower?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This line was taken from Calligraphy, which supports sdk 7. Right now we need sdk 14 because of the animations, but we can keep this to fix issue #1.

mluedke2 added a commit that referenced this pull request Feb 17, 2015
@mluedke2 mluedke2 merged commit 0c031bf into master Feb 17, 2015
@mluedke2 mluedke2 deleted the bugfix/api-21-error branch February 17, 2015 22:54
@mluedke2 mluedke2 mentioned this pull request Feb 17, 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.

None yet

2 participants