Skip to content

Conversation

@ericsakmar
Copy link
Contributor

Adds the following CSS Grid properties to the list of unitless numbers:

  • gridColumnStart
  • gridColumnEnd
  • gridColumnSpan (ie/ms only)
  • gridRowStart
  • gridRowEnd
  • gridRowSpan (ie/ms only)

Without these values in the list, the code appends a px, which is not valid for these properties.

@gaearon
Copy link
Collaborator

gaearon commented Apr 18, 2017

Is this list exhaustive?

@flarnie flarnie mentioned this pull request Apr 18, 2017
49 tasks
@ericsakmar
Copy link
Contributor Author

ericsakmar commented Apr 18, 2017

It's exhaustive to the CSS grid spec, but not necessarily exhaustive to other newer unitless properties outside of that spec.

@flarnie flarnie self-assigned this Apr 20, 2017
@flarnie
Copy link
Contributor

flarnie commented Apr 20, 2017

Initially my questions were:

  1. Is it too soon to add CSS Grid syntax to React? What if the spec changes?
  2. Does this miss any other CSS Grid properties?

Clearly though if we already support gridColumn and gridRow then we should support the more specific grid properties too. And it looks like this does not miss any other non-px grid properties.

Thanks for submitting this @ericsakmar!

@flarnie flarnie merged commit 6488229 into facebook:master Apr 20, 2017
@flarnie flarnie added this to the 15.6 milestone Apr 20, 2017
flarnie pushed a commit to flarnie/react that referenced this pull request Jun 7, 2017
mgol added a commit to mgol/react that referenced this pull request Sep 4, 2018
gaearon pushed a commit that referenced this pull request Sep 4, 2018
jetoneza pushed a commit to jetoneza/react that referenced this pull request Jan 23, 2019
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.

4 participants