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.

Sign up for GitHub

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

Jump to bottom

The timeline scroll view doesn't always start at the bottom. #4524

Closed
pixlwave opened this issue Jul 2, 2021 · 4 comments · Fixed by matrix-org/matrix-ios-kit#852 or #5484
Closed

The timeline scroll view doesn't always start at the bottom. #4524

pixlwave opened this issue Jul 2, 2021 · 4 comments · Fixed by matrix-org/matrix-ios-kit#852 or #5484
Assignees
@Anderas
Labels
A-Timeline O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Z-Papercuts Visible. Impactful. Predictable to action. Z-Ready This issue is ready for development Z-WTF WTF issues: High impact, Low Effort
Milestone
iOS platform pape...

Comments

@pixlwave
Copy link
Member

pixlwave commented Jul 2, 2021 •
edited
Loading

This is reproduced in 2 separate ways

The first on iPhone:

  1. Tap a room.
  2. If you don't see the behaviour below go back out of the room.
  3. Tap the room again and the timeline is shown slightly scrolled up.

The second on iPad:

  1. Tap a room to open it.
  2. Swipe the app away to go back to the home screen.
  3. Wait 5/10 seconds.
  4. Open the app again and the timeline is shown significantly scrolled up.

Bug occurring on iPhone:
https://user-images.githubusercontent.com/6060466/124304411-9009cb80-db5b-11eb-9f28-2c9c2dbaeeaf.mov

Tested on both 6.1" and 4" devices running 1.4.3 and from develop.
@pixlwave pixlwave added T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Z-Papercuts Visible. Impactful. Predictable to action. Z-Visibility-1 Z-Impact-2 labels Jul 2, 2021
@pixlwave
Copy link
Member Author

This appears to have been introduced in 1.4.2, here.

@pixlwave pixlwave self-assigned this Jul 14, 2021
@pixlwave pixlwave mentioned this issue Jul 14, 2021
Merged
@pixlwave pixlwave added this to the iOS platform papercuts milestone Aug 17, 2021
@pixlwave
Copy link
Member Author

pixlwave commented Nov 2, 2021

Re-opening this issue as it's now occurring again as of the 1.6.6 TestFlight build.

@pixlwave pixlwave reopened this Nov 2, 2021
@pixlwave pixlwave added A-Timeline O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist Z-Impact-1 and removed Z-Impact-2 labels Nov 2, 2021
@pixlwave pixlwave mentioned this issue Dec 1, 2021
Closed
@pixlwave pixlwave added S-Major Severely degrades major functionality or product features, with no satisfactory workaround and removed S-Minor Impairs non-critical functionality or suitable workarounds exist labels Jan 11, 2022
@pixlwave pixlwave added Z-WTF WTF issues: High impact, Low Effort and removed Z-Visibility-1 Z-Impact-1 labels Jan 11, 2022
@pixlwave pixlwave removed their assignment Jan 24, 2022
@daniellekirkwood daniellekirkwood added the Z-Ready This issue is ready for development label Jan 26, 2022
Anderas added a commit that referenced this issue Feb 2, 2022
@Anderas
Scroll to the bottom of timeline automatically, fixes #4524
3afe6e4 
Signed-off-by: Andy Uhnak <andyuhnak@gmail.com>
@Anderas Anderas mentioned this issue Feb 2, 2022
Merged
Anderas added a commit that referenced this issue Feb 2, 2022
@Anderas
Scroll to the bottom of timeline automatically, fixes #4524
0c6034e 
Signed-off-by: Andy Uhnak <andyuhnak@gmail.com>
@AverageMarcus
Copy link

AverageMarcus commented Feb 18, 2022 •
edited
Loading

Latest release looks to have fixed this for me when going from the channel list page into the channel but I'm still seeing this issue when going from a notification.

@Anderas
Copy link
Contributor

Anderas commented Feb 21, 2022

Thanks @AverageMarcus for reporting, I created a new ticket to track this use case #5639

@pixlwave pixlwave mentioned this issue Feb 21, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Anderas Anderas

Labels
A-Timeline O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Z-Papercuts Visible. Impactful. Predictable to action. Z-Ready This issue is ready for development Z-WTF WTF issues: High impact, Low Effort
Projects
None yet
Milestone
iOS platform papercuts (moved to http...
4 participants
@AverageMarcus @Anderas @pixlwave @daniellekirkwood