Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Simplify basic demos #609

Merged
merged 2 commits into from
Nov 1, 2018
Merged

Simplify basic demos #609

merged 2 commits into from
Nov 1, 2018

Conversation

samiheikki
Copy link
Contributor

@samiheikki samiheikki commented Sep 30, 2018

Use label in all demos and remove simple examples that don't have label


This change is Reviewable

Copy link
Contributor

@limonte limonte left a comment

Choose a reason for hiding this comment

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

The 'Birthday' label for datepickers which allow selecting a date in the future isn't nice. Could 'Birthday' be replaced with something generic like 'Date'? What do you think?

Copy link
Contributor Author

@samiheikki samiheikki left a comment

Choose a reason for hiding this comment

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

Good point. I replaced Birthday label with few demos but left it for "pre-selected value" and "Initial position" demos as the initial date is pointing to few decades ago which is a good use case for Birthday

Reviewable status: 0 of 1 files reviewed, all discussions resolved

Copy link
Contributor

@yuriy-fix yuriy-fix left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 1 files at r2.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @samiheikki)

@limonte limonte merged commit df05139 into master Nov 1, 2018
@limonte limonte deleted the fix/simplify-demos branch November 1, 2018 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants