Skip to content

Make drawer demo header not scroll away#13337

Merged
xster merged 2 commits intoflutter:masterfrom
xster:drawer-demo-header
Dec 5, 2017
Merged

Make drawer demo header not scroll away#13337
xster merged 2 commits intoflutter:masterfrom
xster:drawer-demo-header

Conversation

@xster
Copy link
Member

@xster xster commented Dec 4, 2017

Fixes #13335

Also update gallery xcode project with new plugin dependencies

@xster xster requested a review from HansMuller December 4, 2017 21:41
@xster xster requested a review from cbracken December 4, 2017 21:41
};
rootObject = 97C146E61CF9000F007C117D /* Project object */;
} No newline at end of file
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Xcode just auto-adds final newlines

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't mean the newline, I meant all the changes to this file. They seem unrelated to the patch.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh I see. Ya, it's somewhat unrelated. Leftovers from #13195 which added new plugins. We'll just get these diffs if anyone flutter runs on iOS until we check them in.

@Hixie
Copy link
Contributor

Hixie commented Dec 4, 2017

test?
LGTM

@xster
Copy link
Member Author

xster commented Dec 5, 2017

Added test and fixed account header overflow text. Had to refactor a bit.

@xster xster merged commit 9db8966 into flutter:master Dec 5, 2017
@xster xster deleted the drawer-demo-header branch December 5, 2017 19:58
DaveShuckerow pushed a commit to DaveShuckerow/flutter that referenced this pull request May 14, 2018
* Make drawer demo header not scroll

* Add test for gallery drawer demo and fix user accounts drawer header overflow
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UserAccountsDrawerHeader should be a persistent sliver header or not be in a ListView

3 participants