Skip to content
This repository has been archived by the owner on Dec 13, 2018. It is now read-only.

fix(TypeScript): add grid and gridTemplate #403

Merged
merged 3 commits into from
Apr 4, 2018

Conversation

whtsky
Copy link
Contributor

@whtsky whtsky commented Mar 29, 2018

What:
fix #402

Why:
N/A

How:
https://developer.mozilla.org/en-US/docs/Web/CSS/grid
https://developer.mozilla.org/en-US/docs/Web/CSS/grid-template

Checklist:

  • Documentation
  • Tests
  • Ready to be merged
  • Added myself to contributors table

@codecov-io
Copy link

codecov-io commented Mar 29, 2018

Codecov Report

Merging #403 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #403   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          10     10           
  Lines         177    177           
  Branches       50     50           
=====================================
  Hits          177    177

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7be549c...ca96808. Read the comment docs.

Copy link
Contributor

@Ailrun Ailrun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for PR! Could you update relevant tests too?
I mean, tests in this file.

/**
* Shorthand property that sets the grid-template-rows, grid-template-columns, and grid-template-areas.
*/
gridTemplate?: CSSWideKeyword | string | 'none'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

string | 'none' is just a string type. Please remove 'none'.

@kentcdodds kentcdodds merged commit edcc9b2 into paypal:master Apr 4, 2018
@kentcdodds
Copy link
Contributor

Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Typings grid property missed
4 participants