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

FIX: Mobile topic-timeline not closing on scrolling past last post #8730

Merged
merged 1 commit into from Jan 16, 2020

Conversation

martin-brennan
Copy link
Contributor

See https://meta.discourse.org/t/post-excerpts-are-missing-from-the-expanded-mobile-timeline/132768/4.

  • When we dragged the topic-timeline handle past the last post
    in a topic we were not closing the timeline as we would if
    scrolling to a specific post
  • This also fixes the issue where when scrolling past the end of
    the topic with a massive last post, none of the post could be
    seen

* when we dragged the topic-timeline handle past the last post
  in a topic we were not closing the timeline as we would if
  scrolling to a specific post
* this also fixes the issue where when scrolling past the end of
  the topic with a massive last post, none of the post could be
  seen
@discoursebot
Copy link

You've signed the CLA, martin-brennan. Thank you! This pull request is ready for review.

@discoursebot
Copy link

This pull request has been mentioned on Discourse Meta. There might be relevant details there:

https://meta.discourse.org/t/post-excerpts-are-missing-from-the-expanded-mobile-timeline/132768/9

@SamSaffron
Copy link
Member

lets give this a shot!

@SamSaffron SamSaffron merged commit 18bba86 into master Jan 16, 2020
@SamSaffron SamSaffron deleted the issue/post-excerpts-missing-on-mobile-timeline branch January 16, 2020 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants