UnifiedPreference icon indicating copy to clipboard operation
UnifiedPreference copied to clipboard

ActionBarSherlock is always required dependency

Open futtetennista opened this issue 12 years ago • 7 comments

Hi Joel,

Thanks very much for this library, it's extremely useful and it works just fine. One issue I had though is that it always requires ActionBarSherlock as a dependency and I don't think one should be forced to use it - even if one could argue that is not such a wise choice. I worked on removing ABS and adding support to it in a separate project, taking the same approach Jake Wharton took with the ABS Map plugin. You can check these two links out and tell me what you think about it, if you think it's fine I'll open a pull request: https://github.com/stefanodacchille/UnifiedPreference/tree/plugin-actionbarsherlock, and https://github.com/stefanodacchille/UnifiedPreference-ActionBarSherlockPlugin .

futtetennista avatar Nov 29 '12 09:11 futtetennista

Nice work. I will cherry-pick from your repo. I do not want to change attr names.

saik0 avatar Nov 29 '12 13:11 saik0

Thanks. Not sure it'll work without changing those attribute name though, I remember trying to do that and failing. But maybe I was just missing something.

futtetennista avatar Nov 29 '12 13:11 futtetennista

Ah, I see what's happening with those duplicate attrs now. Very frustrating.

saik0 avatar Dec 01 '12 06:12 saik0

Thanks. Not sure it'll work without changing those attribute name though, I remember trying to do that and failing. But maybe I was just missing something.

Library projects don't have their own attribute namespace, they get merged into your project. Before we were using the title and icon attrs defined in ABS, note how the xml nodes had no format attribute. Hopefully the "new build system" will have proper libraries, but I suspect it wont be released until after ABS and this project are not needed.

saik0 avatar Dec 01 '12 06:12 saik0

Of course, you're right. Then unfortunately those must change. It sucks that is a breaking change but I think it's still a cleaner approach. On 1 Dec 2012 07:31, "Joel Pedraza" [email protected] wrote:

Thanks. Not sure it'll work without changing those attribute name though, I remember trying to do that and failing. But maybe I was just missing something.

Library projects don't have their own attribute namespace, they get merged into your project. Before we were using the title and icon attrs defined in ABS, note how the xml nodes had no format attribute. Hopefully the "new build system" will have proper libraries, but I suspect it wont be released until after ABS and this project are not needed.

— Reply to this email directly or view it on GitHubhttps://github.com/saik0/UnifiedPreference/issues/5#issuecomment-10914175.

futtetennista avatar Dec 01 '12 08:12 futtetennista

Did you manage to get the demo to build with maven? I get this error:

UnifiedPreference-ActionBarSherlockPlugin/demo/AndroidManifest.xml:28: error: Error: No resource found that matches the given name (at 'theme' with value style/Theme.Sherlock').

saik0 avatar Dec 01 '12 21:12 saik0

Uhm, no I can build the project without errors. I fixed a crash at Runtime when using Maven to build the app though: I forgot to set the correct source directory in plugin and demo POM files.

On 1 December 2012 22:38, Joel Pedraza [email protected] wrote:

Did you manage to get the demo to build with maven? I get this error:

UnifiedPreference-ActionBarSherlockPlugin/demo/AndroidManifest.xml:28: error: Error: No resource found that matches the given name (at 'theme' with value style/Theme.Sherlock').

— Reply to this email directly or view it on GitHubhttps://github.com/saik0/UnifiedPreference/issues/5#issuecomment-10922498.

futtetennista avatar Dec 01 '12 22:12 futtetennista