firka/flutter
firka/flutter
5
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix a bug where ScrollableGrid/ScrollableList would flicker when (#4794)

Browse Source 
navigating away.

Details are in the bug, but when we navigate away, the overscroll
indicator around the Scrollable would change the widget hierarchy. This
would trigger the Scrollable to rebuild, which would cause its
scrollOffset to be clamped to 0 for a frame.

BUG=https://github.com/flutter/flutter/issues/4597
This commit is contained in:
Matt Perry 2016-06-30 15:43:47 -04:00 committed by GitHub
parent f410ca80b7
commit 6298a1aeb1
3 changed files with 13 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
packages/flutter/lib/src/material/overscroll_indicator.dart
View File

@ -226,10 +226,10 @@ class _OverscrollIndicatorState extends State<OverscrollIndicator> {
child: new AnimatedBuilder(
animation: _extentAnimation,
builder: (BuildContext context, Widget child) {
if (_scrollDirection == null) // Haven't seen a scroll yet.
return child;
// We keep the same widget hierarchy here, even when we're not
// painting anything, to avoid rebuilding the children.
return new CustomPaint(
foregroundPainter: new _Painter(
foregroundPainter: _scrollDirection == null ? null : new _Painter(
scrollDirection: _scrollDirection,
extent: _extentAnimation.value,
isLeading: _scrollOffset < _minScrollOffset,

4
packages/flutter/lib/src/rendering/proxy_box.dart
View File

@ -1610,8 +1610,8 @@ class RenderCustomPaint extends RenderProxyBox {
markNeedsPaint();
}
if (attached) {
oldPainter._repaint?.removeListener(markNeedsPaint);
newPainter._repaint?.addListener(markNeedsPaint);
oldPainter?._repaint?.removeListener(markNeedsPaint);
newPainter?._repaint?.addListener(markNeedsPaint);
}
}

12
packages/flutter/lib/src/widgets/clamp_overscrolls.dart
View File

@ -50,14 +50,18 @@ class ClampOverscrolls extends InheritedWidget {
///
/// This utility function is typically used by [Scrollable.builder] callbacks.
static Widget buildViewport(BuildContext context, ScrollableState state, ViewportBuilder builder) {
// TODO(ianh): minScrollOffset and maxScrollOffset are typically determined
// by the container and content size. But we don't know those until we
// layout the viewport, which happens after build phase. We need to rethink
// this.
final bool clampOverscrolls = ClampOverscrolls.of(context);
final double clampedScrollOffset = clampOverscrolls
? state.scrollOffset.clamp(state.scrollBehavior.minScrollOffset, state.scrollBehavior.maxScrollOffset)
: state.scrollOffset;
Widget viewport = builder(context, state, clampedScrollOffset);
if (clampOverscrolls)
viewport = new ClampOverscrolls(value: false, child: viewport);
return viewport;
Widget viewport = builder(context, state, clampedScrollOffset);
if (clampOverscrolls)
viewport = new ClampOverscrolls(value: false, child: viewport);
return viewport;
}
@override