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

New grid layout spacing can go off-screen #1686

Closed
1 task done
rviljoen opened this issue May 2, 2024 · 0 comments · Fixed by #1687
Closed
1 task done

New grid layout spacing can go off-screen #1686

rviljoen opened this issue May 2, 2024 · 0 comments · Fixed by #1687
Assignees
Labels
Bug Something isn't working
Milestone

Comments

@rviljoen
Copy link
Contributor

rviljoen commented May 2, 2024

Description

When using pull to refresh of the podcast grid layout, if you pull again before the refresh is complete, the spacing of the grid changes and pushes some tiles off the edge of the screen. Only way to restore is to change the grid layout to another option and then back.

@SergioEstevao I suspect this bug may be related to the changes in #1628

Step-by-step reproduction instructions

  1. Go to Podcasts tab
  2. Select grid layout
  3. Pull to refresh
  4. Quickly pull down again, while refresh is still in progress
  5. Spacing between tiles will increase, pushing some tiles off the edge.

Screenshots or screen recording

RPReplay_Final1714637836.mov

Did you search for existing bug reports?

  • I have searched for existing bug reports.

Device, Operating system, and Pocket Casts app version

iPhone 14 Pro, iOS 17.4.1, Pocket Casts 7.63.0.1

@SergioEstevao SergioEstevao self-assigned this May 2, 2024
@SergioEstevao SergioEstevao added the Bug Something isn't working label May 2, 2024
@SergioEstevao SergioEstevao added this to the 7.63 ❄️ milestone May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants