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

fix(Header): Remove duplicate class #1040

Merged
merged 1 commit into from
Dec 16, 2016

Conversation

jeffcarbs
Copy link
Member

Using the subheader shorthand would result in the sub header classes being applied twice:

<Header subheader='hey' />

<div class="sub header sub header">hey</div>

This fixes that.

@levithomason
Copy link
Member

Oh, of course! Looks like some related tests need updated.

@jeffcarbs jeffcarbs force-pushed the feature/header-subheader-double-class branch from 82f1e4c to cb4c5b3 Compare December 16, 2016 18:24
@codecov-io
Copy link

codecov-io commented Dec 16, 2016

Current coverage is 96.11% (diff: 100%)

Merging #1040 into master will not change coverage

@@             master      #1040   diff @@
==========================================
  Files           847        847          
  Lines          4710       4710          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           4527       4527          
  Misses          183        183          
  Partials          0          0          

Powered by Codecov. Last update c7fc9d4...cb4c5b3

@jeffcarbs
Copy link
Member Author

Fixed those specs, should be g2g 👍

@levithomason levithomason merged commit 7c25dc9 into master Dec 16, 2016
@levithomason levithomason deleted the feature/header-subheader-double-class branch December 16, 2016 19:10
@levithomason
Copy link
Member

Released in semantic-ui-react@0.62.2

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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants