fix(refresher): show when content is fullscreen #29608
Merged
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.
Issue number: resolves #18714
What is the current behavior?
When the
ion-content
has the fullscreen attribute, theion-refresher
will be hidden while refreshing. This can be seen by dragging far enough to trigger it to snap back and refresh.The refresher ends up being hidden behind the background content element.
Screen.Recording.2024-06-10.at.1.36.24.PM.mov
What is the new behavior?
--offset-top
to0px
for the background content element. This reflects the same behavior of when the content is not fullscreen. By setting this to0px
, the refresher is visible while refreshing.ion-content
isfullscreen
or not.Originally, I was going to update the
pullMin
andpullMax
as agreed on from the investigation ticket. However, it ended up adding too much space between the refresher and the content. This is the reason why I decided to modify the background background instead. Otherwise, it wouldn't mimic the behavior when content doesn't have thefullscreen
attribute.Example of what the spacing looked like:
Screen.Recording.2024-06-10.at.1.36.55.PM.mov
Does this introduce a breaking change?
Other information
Preview
How to test: