zendesk / android-db-commons

Some common utilities for ContentProvider/ContentResolver/Cursor and other db-related android stuff
Apache License 2.0
222 stars 28 forks source link

migration from v4.Loader to Loader #71

Closed grigerlab closed 9 years ago

grigerlab commented 9 years ago

Hello, Do you have plans to migrate to android framework's Loader version (not v4 support library)? Thanks!

chalup commented 9 years ago

Not really. We think that you should use the support-v4 even if you have minSdkVersion > 10, for example to get access to nested Fragments or important fixes in FragmentManager.

grigerlab commented 9 years ago

What fixes are you talking about? v4.Loader is a ported version from main framework sources. minSdkVersion = 14 is the industry standard for now. I don't think I'm going to use ported classes implementation just because of your lib.

chalup commented 9 years ago

Exactly. Important part is, it's a port of the latest code, so you can use the Lollipop implementation of Fragments with all the bug fixes and improvements on devices with older Android OS versions.

Compare ICS and KitKat implementations of Fragment class to see how extensive these changes were.

henieek commented 9 years ago

I always like to mention nested-fragments at this point. You simply can't use getChildFragmentManager() unless your minSdkLevel isn't set to 17 at least. And there're more examples like this.

Unless you are doing something very specific you want to use support's Fragments and Loaders even if your minSdk is high.