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

Black icons with alpha remain black #442

Closed
6 tasks done
rubengees opened this issue Mar 23, 2019 · 6 comments
Closed
6 tasks done

Black icons with alpha remain black #442

rubengees opened this issue Mar 23, 2019 · 6 comments
Assignees
Labels

Comments

@rubengees
Copy link
Contributor

rubengees commented Mar 23, 2019

About this issue

After upgrading from 3.1.0 to 3.2.2, black icons with alpha remain completly black.
I reproduced this in the sample (On Android 9.0), when changing this line:

to:

holder.image.getIcon().color(Color.parseColor("#77000000"));
3.1.0 3.2.2
3 1 0 3 2 2

I also quickly checked the latest alpha and were able to produce similar results.

Details

  • Used library version: 3.2.2
  • Used support library version: appcompat 1.0.2
  • Used gradle build tools version: 3.3.2
  • Used tooling / Android Studio version: 3.5 Canary 7

Checklist

@zTrap zTrap added the bug label Mar 27, 2019
@zTrap zTrap self-assigned this Mar 27, 2019
@zTrap zTrap mentioned this issue Mar 28, 2019
@zTrap
Copy link
Contributor

zTrap commented Mar 28, 2019

Fixed in #443

@marbat87
Copy link
Contributor

Could you fix this in 4.0.0-b1 version also?

Thanks in advance

@mikepenz
Copy link
Owner

mikepenz commented Apr 1, 2019

Fixed in v3.2.3

@marbat87 we will fix this in an upcoming update to the v4.x version too. Please report on the v4 PR as a reminder

@mikepenz mikepenz closed this as completed Apr 1, 2019
@rubengees
Copy link
Contributor Author

@zTrap @mikepenz I tried the latest version, but am still able to reproduce that in some places in my app.

One place in the sample (current develop branch) is the drawer:

In previous versions (like 3.1.0) the icons were gray by default. Now they are black. Setting withIconColor(Color.parseColor("#77000000")) for example here:

PrimaryDrawerItem pdi = new PrimaryDrawerItem()

has no effect.

Another place is when setting a compound drawable on any TextView:

final Drawable compoundDrawable = new IconicsDrawable(this)
        .icon(FontAwesome.Icon.faw_android)
        .sizeDp(24)
        .color(Color.parseColor("#77000000"));

textView.setCompoundDrawablesWithIntrinsicBounds(compoundDrawable, null, null, null);

I tried that in the sample in the PlaygroundActivity and also got entirely black icons.

mikepenz added a commit that referenced this issue Apr 12, 2019
@mikepenz
Copy link
Owner

@rubengees sorry for the delay. Fixed this bug too

@rubengees
Copy link
Contributor Author

No problem, works great, thanks! 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants