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

indexInFlexLine is not consistent in onNewFlexItemAdded() method

Open blah1234 opened this issue 1 year ago • 1 comments

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

Issues and steps to reproduce

In FlexboxHelper.calculateFlexLines(), there is the following code block:

if (isWrapRequired(child, mainMode, mainSize, flexLine.mMainSize,
                    getViewMeasuredSizeMain(child, isMainHorizontal)
                            + getFlexItemMarginStartMain(flexItem, isMainHorizontal) +
                            getFlexItemMarginEndMain(flexItem, isMainHorizontal),
                    flexItem, i, indexInFlexLine, flexLines.size())) {
                <snip!>

                flexLine = new FlexLine();
                flexLine.mItemCount = 1;
                flexLine.mMainSize = mainPaddingStart + mainPaddingEnd;
                flexLine.mFirstIndex = i;
 -->               indexInFlexLine = 0;    <---
                largestSizeInCross = Integer.MIN_VALUE;
            } else {
                flexLine.mItemCount++;
-->                indexInFlexLine++;     <---
            }

So for the 1st FlexLine in FlexboxLayout, indexInFlexLine starts at 1, but for all subsequent FlexLines in the FlexboxLayout, indexInFlexLine starts at 0.

Expected behavior

indexInFlexLine should be consistently 0-indexed

Version of the flexbox library

3.0.0

blah1234 avatar Jul 30 '22 02:07 blah1234