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(input): underline showing at end if text-align is set #5280

Merged
merged 1 commit into from Jun 22, 2017

Conversation

devversion
Copy link
Member

Recently the align input binding has been removed in favor of the CSS property text-align.

If a developer sets the text-align property to end the text will start from the end and also the input underline will show incorrectly start from the end.

Setting the absolute positioned underline to left: 0 ensures that the underline always shows-up correctly (because the mat-input-underline is inside of a relative container)

Note: This still works with RTL because the mat-input-underline container is inside of a relative container and it uses a width of 100%)

Fixes #5272

Recently the `align` input binding has been removed in favor of the CSS property `text-align`.

If a developer sets the `text-align` property to `end` the text will start from the end and also the input underline will show incorrectly start from the end.

Setting the absolute positioned underline to `left: 0` ensures that the underline always shows-up correctly (because the `mat-input-underline` is inside of a relative container)

Fixes angular#5272
@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Jun 21, 2017
@mmalerba mmalerba added pr: lgtm action: merge The PR is ready for merge by the caretaker and removed pr: needs review labels Jun 21, 2017
@jelbourn jelbourn merged commit 5c9391d into angular:master Jun 22, 2017
@devversion devversion deleted the fix/input-text-align-underline branch November 11, 2017 10:23
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker cla: yes PR author has agreed to Google's Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug(input-demo): remove align input binding
4 participants