Reader: Fix blinking navigation bar on dark mode in Reader Post #22860
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Internal ref: pcdRpT-675-p2
In dark mode, the navigation bar on the Reader Post screen could blink as the user scrolls through the content. This PR fixes that issue by removing the navigation bar tint modification based on scroll position when in dark mode.
In π light mode, the intent was to start with a white tint color for the navigation bar, displayed on top of a featured image having a dark tint. As we scroll past the featured image, the tint color should transition to a dark color (
UIColor.text
to be exact) to avoid low contrast with the view background.In π dark mode, the transition seems a bit buggy. I'm suspecting something unexpected happened to the custom color interpolation logic since we're transitioning from
.white
to.text
(which, in dark mode, results to a white color too!) and sometimes this results in the color being black. To solve this, in dark mode, we actually don't need to transition the color. We can just always keep the.white
color!Here is a preview:
To test
π Testing light mode
In light mode, the navigation bar should behave as before.
π Testing dark mode
Regression Notes
Potential unintended areas of impact
Should be none.
What I did to test those areas of impact (or what existing automated tests I relied on)
Manually tested the changes.
What automated tests I added (or what prevented me from doing so)
N/A.
PR submission checklist:
RELEASE-NOTES.txt
if necessary.Testing checklist: