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(Accordion): ability to open multiple items #988

Merged
merged 9 commits into from
Dec 16, 2016

Conversation

levithomason
Copy link
Member

@levithomason levithomason commented Dec 5, 2016

Continuation of #976

Copy link
Contributor

@UnbrandedTech UnbrandedTech left a comment

Choose a reason for hiding this comment

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

Awesome! Thank you for the clean up, I tried to step lightly.

@codecov-io
Copy link

codecov-io commented Dec 5, 2016

Current coverage is 100% (diff: 100%)

Merging #988 into master will increase coverage by 0.26%

@@             master   #988   diff @@
======================================
  Files           140    137     -3   
  Lines          2304   2287    -17   
  Methods           0      0          
  Messages          0      0          
  Branches          0      0          
======================================
- Hits           2298   2287    -11   
+ Misses            6      0     -6   
  Partials          0      0          

Powered by Codecov. Last update fa9175a...b44cea8

@levithomason
Copy link
Member Author

No worries, thanks for the work here. Looks like I need to add one more test to get the coverage we need. I've got to switch to other things now so I'll come back here when I get some more time.

…1023)

test(Accordion): add inclusive tests for opening and closing panels
@levithomason
Copy link
Member Author

This may pass now, thanks to @koenvg writing some more tests.

@jeffcarbs
Copy link
Member

@levithomason - the build died on circle so I restarted it and looks like it passed. We g2g here?

@levithomason levithomason merged commit c7f5dc8 into master Dec 16, 2016
@levithomason levithomason deleted the feat/accordion-exclusive branch December 16, 2016 00:03
@levithomason
Copy link
Member Author

Released in [email protected].

DomiR added a commit to DomiR/Semantic-UI-React that referenced this pull request Dec 19, 2016
* pr/2:
  docs(Sidebar): fix filenames fixes Semantic-Org#1048
  docs(changelog): update changelog [ci skip]
  0.62.3
  feat(Checkbox): Indeterminate state (Semantic-Org#1043)
  Feat(typings): Added typings for Radio, Confirm, Select and Textarea (Semantic-Org#1047)
  chore(package): update babel-core to version 6.21.0 (Semantic-Org#1045)
  fix(dropdown): clear value when dropdown is blurred (Semantic-Org#1046)
  fix(typings): fix typings spelling on `Accordion` (Semantic-Org#1044)
  fix(docs): Fix check icon (Semantic-Org#1042)
  docs(changelog): update changelog [ci skip]
  0.62.2
  fix(Header): Remove duplicate class (Semantic-Org#1040)
  test(Examples): assert no console errors (Semantic-Org#645)
  feat(Accordion): ability to open multiple items (Semantic-Org#988)
  feat(docs): Active className on Sidebar item (Semantic-Org#1031)
  chore(package): update node-sass to version 4.0.0 (Semantic-Org#1015)
  chore(package): update debug to version 2.4.1 (Semantic-Org#1028)
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.

5 participants