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

Issue with status bar transparent #2254

Open
omainegra opened this issue Dec 7, 2018 · 3 comments
Open

Issue with status bar transparent #2254

omainegra opened this issue Dec 7, 2018 · 3 comments

Comments

@omainegra
Copy link

omainegra commented Dec 7, 2018

Description

When the status bar is transparent, Fresco doesn't draw the image underneath it

Reproduction

Just run: https://github.com/omainegra/furry-octo-potato
As an extra, you can toggle the commented code and see how it works with a regular ImageView

Issue

device-2018-12-10-101329

Expected

device-2018-12-10-101359

Update

After playing with it I found that redrawing after ViewCompat.setOnApplyWindowInsetsListener fixes the problem.

ViewCompat.setOnApplyWindowInsetsListener(findViewById<SimpleDraweeView>(R.id.draweeView)) { view, insetsCompat ->
    view.requestLayout()
    insetsCompat
} 

Additional Information

  • Fresco version: 1.11.0
  • Platform version: [{ Nexus 6P, Android 8.1 }, { Pixel 3, Android 9 }]
@lambdapioneer
Copy link
Contributor

Hi @omainegra ! Thank you for the excellently crafted issue :)

I think this is undesired behaviour, but I do not know the best place to fix this. @oprisnik might have an intuitive idea.

I will mark this as a good starting issue to encourage community contributions :) it's probably not making it into our high-priority list for day-to-day fixes.

@iadeelzafar
Copy link
Contributor

@lambdapioneer @oprisnik I'd love to take on this issue. Can you guys give some more context on how this could be resolved?

@oprisnik
Copy link
Contributor

Hey! Awesome, there's a sample in the first post here. So you can patch that and then fix the issue where the status bar is not properly considered, see other comments.

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

No branches or pull requests

4 participants