mirrors/element-web
1
0
Fork 0
mirror of https://github.com/element-hq/element-web synced 2024-11-24 18:25:49 +03:00
Code Issues Projects Releases Packages Wiki Activity

Remove fairly redundant condition

Browse source 
Making sure that a node is intersected by the bottom of the wrapper is a bit overkill, given that we iterate from the bottom. This also prevents the scenario of having no nodes that are not precisely intersected, but possibly straddling the bottom of the wrapper.
This commit is contained in:
Luke Barnard 2017-04-05 17:51:07 +01:00
parent 47f29b9454
commit 423babdb17
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/components/structures/ScrollPanel.js
View file

@ -599,8 +599,7 @@ module.exports = React.createClass({
// as the scrollToken.
// If this is false for the entire for-loop, we default to the last node
// (which is why newScrollState is set on every iteration).
if (boundingRect.top < wrapperRect.bottom &&
wrapperRect.bottom < boundingRect.bottom) {
if (boundingRect.top < wrapperRect.bottom) {
// Use this node as the scrollToken
break;
}