JagWaugh
(Andrew Waugh)
4월 24, 2017, 4:08오후
1
A user asked about this today.
If you’ve used search to get a list of posts, scroll down then click on one of them, if you then use the back function in the browser it always returns you to the top of the search results.
If you view a category, scroll down, then open a topic, the back button returns you to your previous scrolling position.
8개의 좋아요
sam
(Sam Saffron)
4월 26, 2017, 7:09오후
2
I believe this was reported before but can not find the topic or discussion fragment, its a bug we should fix.
2개의 좋아요
Sure if this is not too hard or risky maybe @eviltrout could take it.
nbianca
(Bianca)
8월 10, 2017, 8:54오후
5
Hey,
I have been trying to fix this issue. I implemented it like this so far: https://github.com/discourse/discourse/compare/master...nbianca:scroll_tracking?expand=1
The issue is that even though scrollTo has the right value in didInsertElement, the scrollTop method does not seem to work (probably rendering has not finished?).
Can anyone take a look, please?
4개의 좋아요
j.jaffeux
(Joffrey Jaffeux)
8월 11, 2017, 8:24오전
6
Hi Bianca,
I would implement it this way:
import Scrolling from "discourse/mixins/scrolling";
export default Ember.Component.extend(Scrolling, {
didInsertElement() {
this._super();
this.bindScrolling({ name: this.get("name") });
},
didRender() {
this._super();
const scrollToY = this.session.get(this.get("trackerName"));
if (scrollToY && scrollToY >= 0) {
Ember.run.next(() => $(window).scrollTop(scrollToY + 1));
}
},
didReceiveAttrs() {
this._super();
this.set("trackerName", `scroll-tracker-${this.get("name")}`);
},
willDestroyElement() {
this._super();
this.unbindScrolling(this.get("name"));
},
scrolled() {
this._super();
this.session.set(this.get("trackerName"), $(window).scrollTop());
},
});
It should work, feel free to modify the coding style. Thanks for your contribution.
4개의 좋아요
nbianca
(Bianca)
8월 12, 2017, 8:07오후
7
Thank you a lot! That fixed my issue and made my code look a lot better.
I submitted this pull request: Scroll is tracked on search page. by nbianca · Pull Request #5040 · discourse/discourse · GitHub
5개의 좋아요