Skip to content

Review notes from #13194#13310

Closed
xster wants to merge 1 commit intoflutter:masterfrom
xster:padding-review
Closed

Review notes from #13194#13310
xster wants to merge 1 commit intoflutter:masterfrom
xster:padding-review

Conversation

@xster
Copy link
Member

@xster xster commented Dec 1, 2017

No description provided.

/// Content can slide under the [navigationBar] when they're translucent with
/// a [MediaQuery] padding hinting the top obstructed area.
/// a [MediaQuery] padding signaling the top obstructed area via
/// [MediaQueryData.padding].
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure I really understand what this paragraph is trying to say.

Copy link
Member Author

Choose a reason for hiding this comment

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

reworded again in #13440

final MediaQueryData existingMediaQuery = MediaQuery.of(context);

// TODO(https://github.com/flutter/flutter/issues/12912):
// TODO(xster):
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: you can remove the newline after the colon, too. that way grepping for "TODO" will show the message

@Hixie
Copy link
Contributor

Hixie commented Dec 2, 2017

LGTM

@xster
Copy link
Member Author

xster commented Dec 5, 2017

#13194 is reverted. I'm gonna close this and do this in the revert of the revert.

@xster xster closed this Dec 5, 2017
@xster xster deleted the padding-review branch December 14, 2017 22:01
@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.

3 participants