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

feat(Grid): add stretched prop #466

Merged
merged 2 commits into from
Sep 5, 2016
Merged

feat(Grid): add stretched prop #466

merged 2 commits into from
Sep 5, 2016

Conversation

layershifter
Copy link
Member

This PR addes stretched prop to Grid and Grid.Column. This behavior is missing in SUI docs of Grid, but:

@@ -114,6 +115,9 @@ Grid.propTypes = {
/** A grid can have its columns stack on-top of each other after reaching mobile breakpoints. */
stackable: PropTypes.bool,

/** An can stretch its contents to take up the entire column height. */
Copy link
Member

Choose a reason for hiding this comment

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

"A Grid" typo

@codecov-io
Copy link

codecov-io commented Sep 4, 2016

Current coverage is 98.31% (diff: 100%)

Merging #466 into master will not change coverage

@@             master       #466   diff @@
==========================================
  Files            98         98          
  Lines          1421       1421          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           1397       1397          
  Misses           24         24          
  Partials          0          0          

Powered by Codecov. Last update e98b04f...aeb58c8

@levithomason levithomason merged commit 4011053 into master Sep 5, 2016
@levithomason levithomason deleted the fix/grid-prop branch September 5, 2016 17:59
@levithomason levithomason changed the title fix(Grid): Add missed prop feat(Grid): add stretched prop Sep 5, 2016
@levithomason
Copy link
Member

Thanks much! I categorized this as a feature / enhancement since the functionality is new. I consider functionality that behaves improperly as a bug.

@levithomason
Copy link
Member

Released in stardust@0.41.1

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