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

Page count incorrect when scrolling to bottom #1897

Closed
katosdev opened this issue Sep 28, 2019 · 12 comments · Fixed by #2369
Closed

Page count incorrect when scrolling to bottom #1897

katosdev opened this issue Sep 28, 2019 · 12 comments · Fixed by #2369

Comments

@katosdev
Copy link

katosdev commented Sep 28, 2019

Bug Report

Current Behavior
When scrolling past the last post to the editor, the page count changes to an incorrect value. This has been reproduced across a range of devices and browsers by myself and @Ralkage

Steps to Reproduce

  1. Go to any thread.
  2. Browse to the last post and note the post count (eg 399/399)
  3. Scroll down to the editor
  4. See error - the post count will change (e.g. 349/399)

Video of bug
https://streamable.com/avmrw <--

@PeopleInside
Copy link

Confirmed.

@PeopleInside
Copy link

PeopleInside commented Oct 21, 2019

Hi, this seems to be now fixed in the Flarum community but I cannot see the fix here.
Hope to see how to fix this on my Flarum 10

@rob006
Copy link
Contributor

rob006 commented Oct 22, 2019

This is not fixed on discuss. You can easily reproduce this by creating a long draft, so only your new post will be visible on screen - navigation will show you "1 of X posts" even if you're at the end of the discussion.

02e8d92b

@katosdev
Copy link
Author

In the interest of maintaining updates on this issue, please see below further discussion:
https://discuss.flarum.org/d/21747-page-count-incorrect-when-scrolling-to-bottom-1897

@stale

This comment has been minimized.

@stale stale bot added the stale Issues that have had over 90 days of inactivity label Jan 25, 2020
@rob006

This comment has been minimized.

@stale stale bot removed the stale Issues that have had over 90 days of inactivity label Jan 25, 2020
@stale

This comment has been minimized.

@stale stale bot added the stale Issues that have had over 90 days of inactivity label Apr 24, 2020
@PeopleInside

This comment has been minimized.

@stale stale bot removed the stale Issues that have had over 90 days of inactivity label Apr 24, 2020
@katosdev
Copy link
Author

katosdev commented May 2, 2020

@flarum/core Are we able to push this for B14? :)

@stale

This comment has been minimized.

@stale stale bot added the stale Issues that have had over 90 days of inactivity label Jul 31, 2020
@rob006

This comment has been minimized.

@stale stale bot removed the stale Issues that have had over 90 days of inactivity label Jul 31, 2020
@askvortsov1
Copy link
Sponsor Member

This can be done by modifying the updateScrubber method of PostStreamState by returning count if we're past the end of the end of the stream; that is, no posts are in the viewport, and we are at the end (IE, PostStreamState.viewingEnd() is true)

w-4 added a commit to w-4/core that referenced this issue Oct 8, 2020
askvortsov1 pushed a commit that referenced this issue Oct 15, 2020
Also fixes incorrect page count when scrolling to bottom (#1897)
@askvortsov1 askvortsov1 added this to the 0.1.0-beta.14 milestone Oct 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants