Skip to content

Add details about undoing require() image sizing#5291

Merged
skevy merged 1 commit intofacebook:masterfrom
olegbl:patch-1
Jan 13, 2016
Merged

Add details about undoing require() image sizing#5291
skevy merged 1 commit intofacebook:masterfrom
olegbl:patch-1

Conversation

@olegbl
Copy link
Copy Markdown
Contributor

@olegbl olegbl commented Jan 13, 2016

It can be overridden with undefined to scale dynamically

It can be overridden with undefined to scale dynamically
@facebook-github-bot
Copy link
Copy Markdown
Contributor

By analyzing the blame information on this pull request, we identified @frantic, @brentvatne and @vjeux to be potential reviewers.

@facebook-github-bot facebook-github-bot added GH Review: review-needed CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. labels Jan 13, 2016
skevy added a commit that referenced this pull request Jan 13, 2016
Add details about undoing require() image sizing
@skevy skevy merged commit f25b1a9 into facebook:master Jan 13, 2016
@olegbl olegbl deleted the patch-1 branch January 13, 2016 20:54
@frantic
Copy link
Copy Markdown
Contributor

frantic commented Jan 13, 2016

Sweet, thanks! Long term we could detect use of flex and width/hight on an element and warn the user

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants