Skip to content

Conversation

@Hixie
Copy link
Contributor

@Hixie Hixie commented Nov 1, 2016

...until people understand them.

...until people understand them.
/// * [Dialog]
/// * [showDialog]
/// * [ListItem], to display icons and text in a card.
/// * [Dialog] and [showDialog], to display a modal card.
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd just link to showDialog and not to Dialog itself. showDialog has good docs for people who want to show a dialog.

@abarth
Copy link
Contributor

abarth commented Nov 1, 2016

LGTM

@Hixie
Copy link
Contributor Author

Hixie commented Nov 1, 2016

Gonna land this and apply the suggested improvement in a subsequent patch.

@Hixie Hixie merged commit 9d64dcc into flutter:master Nov 1, 2016
@Hixie Hixie deleted the docs branch November 1, 2016 05:27
@InMatrix InMatrix added the from: study Reported in a UX study label Nov 11, 2016
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

from: study Reported in a UX study

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants