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

fix(textarea): update label alignment to baseline #18042

Merged
merged 10 commits into from Apr 16, 2019

Conversation

Projects
None yet
2 participants
@brandyscarney
Copy link
Member

commented Apr 15, 2019

Pull request checklist

Please check if your PR fulfills the following requirements:

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)
  • Build (npm run build) was run locally and any changes were pushed
  • Lint (npm run lint) has passed locally and any fixes were made for failures

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

Issue Number: #16187

What is the new behavior?

  • Adds baseline alignment to labels and textarea in input items
  • Updates the MD label UI to be closer to the spec
  • Adds a spec test used to compare against the MD web implementation

Does this introduce a breaking change?

  • Yes
  • No

Other information

master branch
localhost_3333_src_components_textarea_test_basic(Galaxy S5) (6) localhost_3333_src_components_input_test_spec(Galaxy S5)
localhost_3333_src_components_textarea_test_basic(Galaxy S5) (5) localhost_3333_src_components_textarea_test_basic(Galaxy S5) (4)

@brandyscarney brandyscarney requested a review from liamdebeasi Apr 15, 2019

@liamdebeasi
Copy link
Member

left a comment

Looks good! Just one note:

There used to be a bit of space between the top of the textarea content and the label. It looks like that's no longer there:

master branch
Screen Shot 2019-04-16 at 8 31 35 AM Screen Shot 2019-04-16 at 8 32 04 AM

Note: This only happens on MD mode

@brandyscarney

This comment has been minimized.

Copy link
Member Author

commented Apr 16, 2019

@liamdebeasi Good catch! I'll look into it.

brandyscarney added some commits Apr 16, 2019

@liamdebeasi
Copy link
Member

left a comment

looks good! 🎉

brandyscarney added some commits Apr 16, 2019

@brandyscarney brandyscarney merged commit 38ae362 into master Apr 16, 2019

2 checks passed

build Workflow: build
Details
screenshot Screenshot
Details

@brandyscarney brandyscarney deleted the fix-textarea-alignment branch Apr 16, 2019

Kiku-git added a commit to Kiku-git/ionic that referenced this pull request May 16, 2019

fix(textarea): update label alignment for inputs and textareas (ionic…
…-team#18042)

- aligns label and textarea to baseline
- updates floating and stacked labels in items to align closer to the md spec

fixes ionic-team#16187
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.