It wasn't working ideally and introduced some annoying false positiviespull/4866/head
parent
2ff7146b6d
commit
bdc8b4fd91
|
@ -74,7 +74,7 @@ export default class ScrollableList extends PureComponent {
|
||||||
|
|
||||||
// Reset the scroll position when a new child comes in in order not to
|
// Reset the scroll position when a new child comes in in order not to
|
||||||
// jerk the scrollbar around if you're already scrolled down the page.
|
// jerk the scrollbar around if you're already scrolled down the page.
|
||||||
if (someItemInserted && this._oldScrollPosition && (this.node.scrollTop > 0 || this._recentlyMoved())) {
|
if (someItemInserted && this._oldScrollPosition && this.node.scrollTop > 0) {
|
||||||
const newScrollTop = this.node.scrollHeight - this._oldScrollPosition;
|
const newScrollTop = this.node.scrollHeight - this._oldScrollPosition;
|
||||||
|
|
||||||
if (this.node.scrollTop !== newScrollTop) {
|
if (this.node.scrollTop !== newScrollTop) {
|
||||||
|
|
Loading…
Reference in New Issue