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

Added the ability to layer a StateListDrawable over the bitmap. #377

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

eygraber
Copy link

@eygraber eygraber commented Oct 2, 2014

I was getting sick of using FrameLayouts with my ImageViews and layering a View with a StateListDrawable background on top of it. I did basic testing on this using a GridView. Seems to work fine (a little slower than usual if you provide a resource because the StateListDrawable needs to be parsed), but it seemed like I had to scroll an ImageView off the screen and then back on again to get it to display if it was loaded from the cache. Let me know if you want me to do more intensive testing.

@eygraber eygraber closed this Oct 2, 2014
@eygraber eygraber reopened this Oct 2, 2014
@koush
Copy link
Owner

koush commented Oct 3, 2014

Just use placeholder?

@eygraber
Copy link
Author

eygraber commented Oct 3, 2014

I want the state list for the loaded bitmap.
On Oct 2, 2014 11:04 PM, "Koushik Dutta" [email protected] wrote:

Just use placeholder?


Reply to this email directly or view it on GitHub
#377 (comment).

@eygraber
Copy link
Author

eygraber commented Oct 3, 2014

Also, it seems like ion.iml isn't in the right .gitignore. I'm not sure if that needs to be checked into git, so if not, I can update this pull request to not track it anymore (same thing for .idea).

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.

2 participants