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

Drop unsupported HorizontalConstraints #433

Merged
merged 30 commits into from
Jan 15, 2019
Merged

Conversation

montezume
Copy link
Contributor

@montezume montezume commented Jan 14, 2019

Remove unsupported horizontalConstraints

Closes #325

FieldLabel

Drop xs

Show screenshots

xs
screen shot 2019-01-14 at 10 24 36 am

Fields

Select fields

This covers AsyncCreatableSelectField,AsyncSelectField, CreatableSelectField and SelectInput

Drop xs

Show screenshots **xs**

screen shot 2019-01-14 at 10 28 05 am

Date inputs and fields

This covers DateField,DateRangeField and DateTimeField.

Drop xs, s

Show screenshots **xs**

screen shot 2019-01-14 at 10 37 56 am

s
screen shot 2019-01-14 at 10 38 01 am

Multiline text fields

This covers LocalizedMultilineField and MultilineTextField

  • (for any "localized" input, we need to drop xs and s because the bottom "show all languages" overlaps passed the S width anyways...

Drop xs and s

Show screenshots

xs

screen shot 2019-01-14 at 10 49 01 am

s

screen shot 2019-01-14 at 10 48 56 am

LocalizedTextField

  • (for any "localized" input, we need to drop xs and s because the bottom "show all languages" overlaps passed the S width anyways...
    Drop xs and s
Show screenshots

xs

screen shot 2019-01-14 at 10 53 05 am

s

screen shot 2019-01-14 at 10 53 09 am

MoneyField

Drop xs and s

Show screenshots

xs

screen shot 2019-01-14 at 10 54 13 am

s

screen shot 2019-01-14 at 10 54 17 am

NumberField

Drop xs

Show screenshots

xs

screen shot 2019-01-14 at 11 04 57 am

TextFields

Covers TextField and PasswordField

Drop xs

Show screenshots

xs

screen shot 2019-01-14 at 11 04 08 am

TimeField

Drop xs

Show screenshots

xs

screen shot 2019-01-14 at 11 05 51 am

Inputs

Select inputs

Covers AsyncCreatableSelectInput, AsyncSelectInput, SelectInput and CreatableSelectInput

Drop xs

Show screenshots

xs

screen shot 2019-01-14 at 11 07 57 am

Date Inputs

Covers DateInput, DateRangeInput and DateTimeInput

Drop xs, s

Show screenshots

xs

screen shot 2019-01-14 at 11 15 43 am

s

screen shot 2019-01-14 at 11 15 48 am

LocalizedMoneyInput

Drop xs, s

Show screenshots

xs

screen shot 2019-01-14 at 11 16 39 am

s

screen shot 2019-01-14 at 11 16 42 am

Multiline text inputs

This covers LocalizedMultilineInput and MultilineTextInput

Drop xs, s

Show screenshots

xs

screen shot 2019-01-14 at 11 20 03 am

s

screen shot 2019-01-14 at 11 20 08 am

Text Inputs

Covers TextInput and PasswordInput

Drop xs

Show screenshots

xs

screen shot 2019-01-14 at 11 22 50 am

TimeInput

Drop xs

Show screenshots

xs

screen shot 2019-01-14 at 11 23 16 am

@montezume montezume self-assigned this Jan 14, 2019
@montezume montezume changed the title Ml drop horz constraints Drop unsupported HorizontalConstraints Jan 14, 2019
Copy link
Contributor

@dferber90 dferber90 left a comment

Choose a reason for hiding this comment

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

🚢

Copy link
Contributor

@tdeekens tdeekens left a comment

Choose a reason for hiding this comment

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

This is nice!

@montezume montezume added the 👨‍🎨 Status: UI/UX Review Requires review from design team label Jan 14, 2019
@montezume montezume force-pushed the ml-drop-horz-constraints branch from 3e222b6 to f214e7c Compare January 15, 2019 09:04
@montezume montezume force-pushed the ml-drop-horz-constraints branch from f214e7c to e09a71d Compare January 15, 2019 09:20
@montezume montezume merged commit 7c53c7d into master Jan 15, 2019
@montezume montezume deleted the ml-drop-horz-constraints branch January 15, 2019 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants