Do not update a scaffold's AppBar scroll offset if the drawer is open #5343
Conversation
|
LGTM. |
jason-simmons
force-pushed
the
bug_5131
branch
from
7727209 to
cb56c24
Compare
There was a problem hiding this comment.
if (_drawerKey.currentState == null || _drawerKey.currentState.isOpen)
There was a problem hiding this comment.
_drawerKey.currentState?.isOpen ?? true should also work.
|
I'm not sure that this the right fix. The source of the problem is the lack of a scrollableKey for the Scaffold. Without that key, any scrollable that ends up on top of the appbar will generate scroll notifications that propel the appbar's animation. We could avoid this problem and some similar ones by putting the scroll notification listener around the scaffold's body, instead of being effectively around the entire scaffold. |
jason-simmons
force-pushed
the
bug_5131
branch
from
cb56c24 to
83bb936
Compare
|
Moved the scroll notification listener to the body widget. PTAL |
There was a problem hiding this comment.
This line could be in the if above.
|
You should probably reword the commit message. LGTM. |
jason-simmons
force-pushed
the
bug_5131
branch
from
83bb936 to
0e013ea
Compare
There was a problem hiding this comment.
Did you mean to add a blank line here?
|
LGTM with a test. |
jason-simmons
force-pushed
the
bug_5131
branch
2 times, most recently
from
0b7be21 to
87297f7
Compare
|
Added a test |
jason-simmons
force-pushed
the
bug_5131
branch
from
87297f7 to
a975f51
Compare
|
It would be useful to put a pointer to the issue in the test, since it's a regression test. LGTM |
3 participants
Fixes #5131