-
Notifications
You must be signed in to change notification settings - Fork 64
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
ActionBarSherlock is always required dependency #5
Comments
Nice work. I will cherry-pick from your repo. I do not want to change attr names. |
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. |
Ah, I see what's happening with those duplicate attrs now. Very frustrating. |
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 |
Of course, you're right. Then unfortunately those must change. It sucks
|
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'). |
Uhm, no I can build the project without errors. I fixed a crash at Runtime On 1 December 2012 22:38, Joel Pedraza [email protected] wrote:
|
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 .
The text was updated successfully, but these errors were encountered: