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

25.0.0-Alpha-3: Scroll position is not reset when switching between articles & feeds #2548

Closed
uncovery opened this issue Jan 16, 2024 · 0 comments · Fixed by #2800
Closed

25.0.0-Alpha-3: Scroll position is not reset when switching between articles & feeds #2548

uncovery opened this issue Jan 16, 2024 · 0 comments · Fixed by #2800
Labels
bug
Milestone
Vue Rewrite

Comments

@uncovery
Copy link

IMPORTANT

Read and tick the following checkbox after you have created the issue or place an x inside the brackets ;)

  • [x ] I have read the CONTRIBUTING.md and followed the provided tips
  • [x ] I accept that the issue will be closed without comment if I do not check here
  • [x ] I accept that the issue will be closed without comment if I do not fill out all items in the issue template.

Explain the Problem

Scroll position in the left reading pane is maintained across articles read

Steps to Reproduce

in articles:

  1. open an article that has longer contents than the screen height in the right reading pane (e.g. from the New York Times).
  2. scroll to the lower part of the article.
  3. select another article in the list that also has a longer content
  4. the 2nd article will be scrolled to the bottom as well instead of being shown from the top.
    in feeds:
  5. open a feed that has more entries than fit on the screen
  6. scroll down a few pages
  7. open another feed with lots of articles
  8. the 2nd feed is not shown from the top but the same amount of pages down from the top as the previous feed was scrolled to.

System Information

  • News app version: 25.0.0-Alpha-3:
  • Nextcloud version: Nextcloud Hub 7 (28.0.1)
  • Cron type: system cron
  • PHP version: 8.2.14
  • Browser and version: Firefox 121.0.1 (64-bit) & MS Edge Version 120.0.2210.133 (Official build) (64-bit)
  • OS and version: Windows 11
@uncovery uncovery added the bug label Jan 16, 2024
@SMillerDev SMillerDev added this to the Vue Rewrite milestone Jan 16, 2024
@Grotax Grotax mentioned this issue Jan 17, 2024
Closed
19 tasks
@wofferl wofferl mentioned this issue Oct 10, 2024
Merged
@Grotax Grotax mentioned this issue Oct 14, 2024
Merged
Grotax added a commit that referenced this issue Oct 14, 2024
@Grotax
Release 25.0.0-alpha10
bfd8e40 
Changed
- Require NC 29 or 30, dropped support for NC 28
- Require PHP 8.2 or higher

Fixed
- Scroll position is not reset when switching between articles & feeds (#2548)
- Unread counter does not count down when folders or feeds are marked read (#2800)
- Query fetching status didn't work (#2800)
- Keyboard shortcuts are active even when searching (#2738)

Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Grotax added a commit that referenced this issue Oct 14, 2024
@Grotax
Release 25.0.0-alpha10
d21cb5a 
Changed
- Require NC 29 or 30, dropped support for NC 28
- Require PHP 8.2 or higher

Fixed
- Scroll position is not reset when switching between articles & feeds (#2548)
- Unread counter does not count down when folders or feeds are marked read (#2800)
- Query fetching status didn't work (#2800)
- Keyboard shortcuts are active even when searching (#2738)

Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
Vue Rewrite
Development

Successfully merging a pull request may close this issue.

A few small fixes wofferl/news
2 participants
@uncovery @SMillerDev