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

Compressed forms #800

Closed
wants to merge 9 commits into from
Closed

Compressed forms #800

wants to merge 9 commits into from

Conversation

cchaos
Copy link
Contributor

@cchaos cchaos commented May 7, 2018

Added compressed props to

  • euiFormControlLayout: for shifting the icons (and loading and clear) and reducing width
  • field_search, field_text, field_number, field_password, select, checkbox, checkbox_group, radio, radio_group, text_area, switch (though it doesn't do anything for now)
  • file_picker needed some extra work to make just look like a normal input field that is compressed. May want to consider also allowing for a non-compressed but but normal input style as well.
  • euiFormRow: for reducing spacing between rows and passes down the variable to the child form controls

screen shot 2018-05-07 at 15 22 24 pm

Also fixes #792: Moved bottom padding from help text to form row

cchaos added 8 commits May 7, 2018 15:24
- `euiFormControlLayout`: for shifting the icons (and loading and clear) and reducing width
- `field_search`, `field_text`, `field_number`, `field_password`, `select`, `checkbox`, `checkbox_group`, `radio`, `radio_group`
- __NOT__ to `text_area`, `switch`
- `file_picker` needed some extra work to make just look like a normal input field that is compressed. May want to consider also allowing for a non-compressed but but normal input style as well.
- Also fixes elastic#792: Moved bottom padding from help text to form row
@cchaos cchaos requested review from chandlerprall and snide May 7, 2018 19:31
@snide snide mentioned this pull request May 7, 2018
44 tasks
@snide
Copy link
Contributor

snide commented May 7, 2018

I'll get this merged over your break. Loading it up locally it looks good though 👍 . I'll do a full-scan / review of the code of the code and tweak as needed.

@cchaos
Copy link
Contributor Author

cchaos commented May 7, 2018

Thanks!

@snide snide mentioned this pull request May 9, 2018
@snide
Copy link
Contributor

snide commented May 9, 2018

Closing in favor of #811, which is the same, just lets me make changes while @cchaos is away.

@snide snide closed this May 9, 2018
@cchaos cchaos deleted the compressed-forms branch July 2, 2018 17:44
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.

Form row help text padding makes spacing inconsistent
2 participants