-
Notifications
You must be signed in to change notification settings - Fork 30.2k
Closed
Labels
P2Important issues not at the top of the work listImportant issues not at the top of the work listd: api docsIssues with https://api.flutter.dev/Issues with https://api.flutter.dev/f: scrollingViewports, list views, slivers, etc.Viewports, list views, slivers, etc.found in release: 1.22Found to occur in 1.22Found to occur in 1.22frameworkflutter/packages/flutter repository. See also f: labels.flutter/packages/flutter repository. See also f: labels.good first issueRelatively approachable for first-time contributorsRelatively approachable for first-time contributorshas reproducible stepsThe issue has been confirmed reproducible and is ready to work onThe issue has been confirmed reproducible and is ready to work on
Description
Current documentation:
getMaxChildIndexForScrollOffset(double scrollOffset) → int
The maximum child index that is visible at (or before) this scroll offset.
getMinChildIndexForScrollOffset(double scrollOffset) → int
The minimum child index that is visible at (or after) this scroll offset.
How these are actually used seems to indicate this should be the documentation:
getMaxChildIndexForScrollOffset(double scrollOffset) → int
The maximum child index that intersects with (or is before) this scroll offset.
getMinChildIndexForScrollOffset(double scrollOffset) → int
The minimum child index that intersects with (or is after) this scroll offset.
Basically, all the children from getMin...(scrollOffset) to getMax...(scrollOffset+extent) are being laid out but the documentation suggests that getMin...(scrollOffset) to getMax...(scrollOffset) should be all the children being laid out.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
P2Important issues not at the top of the work listImportant issues not at the top of the work listd: api docsIssues with https://api.flutter.dev/Issues with https://api.flutter.dev/f: scrollingViewports, list views, slivers, etc.Viewports, list views, slivers, etc.found in release: 1.22Found to occur in 1.22Found to occur in 1.22frameworkflutter/packages/flutter repository. See also f: labels.flutter/packages/flutter repository. See also f: labels.good first issueRelatively approachable for first-time contributorsRelatively approachable for first-time contributorshas reproducible stepsThe issue has been confirmed reproducible and is ready to work onThe issue has been confirmed reproducible and is ready to work on