Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

fix(input): number input width in webkit #7761

Closed
wants to merge 1 commit into from

Conversation

crisbeto
Copy link
Member

Fixes an issue in Webkit browsers where number inputs would collapse to about 1/3 of their proper size, if they're placed in a flexbox and have min and max attributes.

Fiddle with a simplified example of the issue: https://jsfiddle.net/crisbeto/phm2nuar/3/

Closes #7349.

@crisbeto crisbeto added needs: review This PR is waiting on review from the team pr: merge ready This PR is ready for a caretaker to review and removed needs: review This PR is waiting on review from the team labels Mar 26, 2016
@topherfangio
Copy link
Contributor

LGTM 👍

@ThomasBurleson ThomasBurleson modified the milestones: 1.1.1, Backlog Apr 19, 2016
@ThomasBurleson ThomasBurleson added needs: rebase This PR needs to be rebased on the latest commits from master and conflicts need to be resolved and removed pr: merge ready This PR is ready for a caretaker to review labels Jun 1, 2016
@ThomasBurleson
Copy link
Contributor

@crisbeto - please rebase.

@crisbeto
Copy link
Member Author

crisbeto commented Jun 1, 2016

I don't see any conflicts @ThomasBurleson.

Fixes an issue in Webkit browsers where number inputs would collapse to about 1/3 of their proper size, if they're placed in a flexbox and have `min` and `max` attributes.

Fiddle with a simplified example of the issue: https://jsfiddle.net/crisbeto/phm2nuar/3/

Closes angular#7349.
@crisbeto crisbeto added needs: review This PR is waiting on review from the team and removed needs: rebase This PR needs to be rebased on the latest commits from master and conflicts need to be resolved labels Jun 1, 2016
@Splaktar Splaktar removed this from the - Backlog milestone Jun 28, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs: review This PR is waiting on review from the team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants