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
Add preference to show subscription title #5538
Conversation
87811cc
to
7d050dd
Compare
When the cover is not loaded yet (try clearing the cache), it shows another random color that the View had before. I think it should be set to gray during loading. When the image cannot be loaded (404, etc), the text is white on light gray, which is pretty hard to read. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have some more comments, sorry 🙈
core/src/main/java/de/danoeh/antennapod/core/preferences/UserPreferences.java
Outdated
Show resolved
Hide resolved
app/src/main/java/de/danoeh/antennapod/adapter/SubscriptionsRecyclerAdapter.java
Outdated
Show resolved
Hide resolved
app/src/main/java/de/danoeh/antennapod/adapter/SubscriptionsRecyclerAdapter.java
Outdated
Show resolved
Hide resolved
app/src/main/java/de/danoeh/antennapod/adapter/SubscriptionsRecyclerAdapter.java
Outdated
Show resolved
Hide resolved
app/src/main/java/de/danoeh/antennapod/adapter/CoverLoader.java
Outdated
Show resolved
Hide resolved
app/src/main/java/de/danoeh/antennapod/adapter/CoverLoader.java
Outdated
Show resolved
Hide resolved
Use dark text color when the dominant color is too light
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I didn't realize you already added the Glide loader. GitHub doesn't send an email when force-pushing.
I think it would be fair to also add a comment mentioning where the Glide stuff was taken from
core/src/main/java/de/danoeh/antennapod/core/glide/PaletteBitmapResource.java
Outdated
Show resolved
Hide resolved
core/src/main/java/de/danoeh/antennapod/core/glide/PaletteBitmapResource.java
Outdated
Show resolved
Hide resolved
core/src/main/java/de/danoeh/antennapod/core/glide/PaletteBitmapResource.java
Outdated
Show resolved
Hide resolved
Thanks! Will be in version 2.5.0 :) |
This pull request has been mentioned on AntennaPod Forum. There might be relevant details there: https://forum.antennapod.org/t/antennapod-2-5-release-notes/1636/1 |
Closes #5415
Added a preference to display subscription title and changed the title
TextView
background to the dominant color in the subscription cover image instead of transparent background.