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(Dropdown): Add Header sub-component #471

Merged
merged 2 commits into from
Sep 6, 2016
Merged

Conversation

layershifter
Copy link
Member

This PR:

  • adds Header subcomponent (#382);
  • adds tests to DropdownDivider;
  • adds common.rendersChildren test to DropdownItem.

@codecov-io
Copy link

codecov-io commented Sep 6, 2016

Current coverage is 98.60% (diff: 100%)

Merging #471 into master will increase coverage by 0.28%

@@             master       #471   diff @@
==========================================
  Files            98         99     +1   
  Lines          1426       1433     +7   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           1402       1413    +11   
+ Misses           24         20     -4   
  Partials          0          0          

Powered by Codecov. Last update a5dce40...200af1e

@levithomason
Copy link
Member

Let's extend the Dropdown test to assert this is a sub component.

@@ -64,6 +68,7 @@ describe('Dropdown Component', () => {

common.isConformant(Dropdown)
common.hasUIClassName(Dropdown)
common.hasSubComponents(Dropdown, [DropdownDivider, DropdownHeader, DropdownItem, DropdownMenu])
Copy link
Member Author

Choose a reason for hiding this comment

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

Let's extend the Dropdown test to assert this is a sub component.

Did you mean this? 💭

@levithomason levithomason merged commit 37c54af into master Sep 6, 2016
@levithomason levithomason deleted the feat/dropdown-header branch September 6, 2016 18:46
@levithomason
Copy link
Member

Released in stardust@0.41.3

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