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

The Downloads library is wrongly scrolled down when files are sorted #3317

Closed
Softvision-RemusDranca opened this issue May 4, 2020 · 1 comment · Fixed by #3326
Closed
Assignees
Labels
in progress [QA]:Normal issue Label for QA to mark normal issues logged [QA]:Verified fixed Label for QA to mark verified fixed issues
Milestone

Comments

@Softvision-RemusDranca
Copy link

[Affected Versions]:

  • Firefox Reality version 10 RC4 (2999129).

[Affected Platforms]:

  • Pico Neo 2 - Android - 8.1.0
  • Oculus Quest - Android 7.1.1
  • HTC Vive Focus - Android 7.1.2

[Prerequisites]:

  • The headset is turned on and the app is opened.
  • Have an extended list view for the Downloads library (more than 10 downloads).

[Steps to reproduce]:

  1. Click the Downloads button from the tray.
  2. Click the Sort By button.
  3. Select an sort by option and observe the behavior.

[Expected results]:

  • The page is scrolled up to the first item.

[Actual results]:

  • The page is scrolled down.

[Notes]:

  • Attached is a screen recording of the issue: link.
@Softvision-RemusDranca Softvision-RemusDranca added the [QA]:Normal issue Label for QA to mark normal issues logged label May 4, 2020
@emstanley emstanley added this to the #11 polish milestone May 4, 2020
@keianhzo keianhzo self-assigned this May 4, 2020
@keianhzo keianhzo modified the milestones: #11 polish, #10 features May 4, 2020
bluemarvin pushed a commit that referenced this issue May 5, 2020
* Improve downloads sorting


Reset scroll position after changing sorting

* Fix sorting restoration
@Softvision-RemusDranca
Copy link
Author

I have verified this issue and it's no longer reproducible with the Firefox Reality version 10 RC5 (e84d3c9). The list is scrolled up to the first item after the files are sorted.

Tested on HTC Vive Focus (Android 7.1.2), Oculus Quest (Android 7.1.1), and Pico Neo 2 (Android - 8.1.0) devices.

@Softvision-RemusDranca Softvision-RemusDranca added the [QA]:Verified fixed Label for QA to mark verified fixed issues label May 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in progress [QA]:Normal issue Label for QA to mark normal issues logged [QA]:Verified fixed Label for QA to mark verified fixed issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants