Commit 02a44839 authored by Matt Perry's avatar Matt Perry Committed by GitHub

Pesto: Fix app bar height when returning from recipe page. (#4579)

The problem was that the Scaffold was getting a new key each time we
navigated to the main page. The key influences where page state is
stored, including the scroll offset. So for a single frame, the Scaffold
incorrectly thinks the scroll offset is 0 and the app bar should be
expanded.

BUG=https://github.com/flutter/flutter/issues/4400
parent f6b8220c
...@@ -51,7 +51,7 @@ class PestoDemo extends StatefulWidget { ...@@ -51,7 +51,7 @@ class PestoDemo extends StatefulWidget {
} }
class _PestoDemoState extends State<PestoDemo> { class _PestoDemoState extends State<PestoDemo> {
final GlobalKey<ScaffoldState> _scaffoldKey = new GlobalKey<ScaffoldState>(); static final GlobalKey<ScaffoldState> _scaffoldKey = new GlobalKey<ScaffoldState>();
final TextStyle favoritesMessageStyle = _textStyle(16.0); final TextStyle favoritesMessageStyle = _textStyle(16.0);
final TextStyle userStyle = _textStyle(12.0, FontWeight.bold); final TextStyle userStyle = _textStyle(12.0, FontWeight.bold);
final TextStyle emailStyle = _textStyle(12.0).copyWith(color: Colors.black54); final TextStyle emailStyle = _textStyle(12.0).copyWith(color: Colors.black54);
...@@ -77,7 +77,7 @@ class _PestoDemoState extends State<PestoDemo> { ...@@ -77,7 +77,7 @@ class _PestoDemoState extends State<PestoDemo> {
Widget _buildAppBar(BuildContext context) { Widget _buildAppBar(BuildContext context) {
final double statusBarHeight = MediaQuery.of(context).padding.top; final double statusBarHeight = MediaQuery.of(context).padding.top;
return new AppBar( return new AppBar(
expandedHeight: 128.0, expandedHeight: _kAppBarHeight,
actions: <Widget>[ actions: <Widget>[
new IconButton( new IconButton(
icon: Icons.search, icon: Icons.search,
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment