flexbox-layout icon indicating copy to clipboard operation
flexbox-layout copied to clipboard

Wrong statement in FlexboxLayoutManager.computeScrollOffset()

Open setsailz opened this issue 6 years ago • 0 comments

  • [ ] I have searched existing issues and confirmed this is not a duplicate

Issues and steps to reproduce

I see code in FlexboxLayoutManager.computeScrollOffset(), line 2261 :

if (firstLinePosition == 0 || firstLinePosition == NO_POSITION) { return 0; }

The first statement 'firstLinePosition == 0' will cause first line return 0 scrollOffset.

Expected behavior

Remove this statement.

Version of the flexbox library

e.g. 1.1.0

Link to code

if (firstLinePosition == NO_POSITION) { return 0; }

setsailz avatar May 16 '19 06:05 setsailz