Merge branch 'unmount-noscroll' into 'develop'

Don't hijack scrollbar when status is unmounted, fixes #404

Closes #404

See merge request soapbox-pub/soapbox-fe!221
better-thread-display
Alex Gleason 4 years ago
commit 69cd155c21

@ -151,14 +151,16 @@ class Status extends ImmutablePureComponent {
} }
componentWillUnmount() { componentWillUnmount() {
if (this.node && this.props.getScrollPosition) { // FIXME: Run this code only when a status is being deleted.
const position = this.props.getScrollPosition(); //
if (position !== null && this.node.offsetTop < position.top) { // if (this.node && this.props.getScrollPosition) {
requestAnimationFrame(() => { // const position = this.props.getScrollPosition();
this.props.updateScrollBottom(position.height - position.top); // if (position !== null && this.node.offsetTop < position.top) {
}); // requestAnimationFrame(() => {
} // this.props.updateScrollBottom(position.height - position.top);
} // });
// }
// }
} }
handleToggleMediaVisibility = () => { handleToggleMediaVisibility = () => {

Loading…
Cancel
Save