Skip to content

Conversation

xster
Copy link
Member

@xster xster commented Jun 1, 2017

#7508

Not in this PR: a CupertinoBack/CancelButton which needs #10334 solved first

Diffbases: #10337

@xster xster requested review from Hixie and HansMuller June 1, 2017 01:20
@xster xster changed the title CupertinoNavigationBar part 2 - create a CupertinoNavigationBar CupertinoNavigationBar part 2 - create a bare bone CupertinoNavigationBar Jun 1, 2017
@xster xster force-pushed the cupertino-nav-bar branch from e43d53a to 5b0a8ff Compare June 1, 2017 01:39
Copy link
Contributor

@Hixie Hixie Jun 1, 2017

Choose a reason for hiding this comment

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

the color can only be semi-transparent, not translucent. I would just say "If the given [backgroundColor] has an opacity less than 1.0 (which is the case for the default value)..."

Copy link
Member Author

Choose a reason for hiding this comment

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

umm true. English fail. Corrected.

@Hixie
Copy link
Contributor

Hixie commented Jun 1, 2017

LGTM

@xster xster force-pushed the cupertino-nav-bar branch from 5b0a8ff to dbf4ae1 Compare June 1, 2017 23:18
@xster xster merged commit 2aaa7f8 into flutter:master Jun 2, 2017
@xster xster deleted the cupertino-nav-bar branch June 2, 2017 00:17
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 13, 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