Skip to content
This repository was archived by the owner on Jun 10, 2019. It is now read-only.

Commit 960dceb

Browse files
author
Kyle Holmberg
committed
Fix some CodeClimate issues
1 parent ba6ec33 commit 960dceb

3 files changed

Lines changed: 8 additions & 9 deletions

File tree

src/scenes/home/codeSchools/codeSchools.css

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,12 +3,12 @@
33
flex-flow: row wrap;
44
width: 80%;
55
justify-content: space-around;
6-
padding-top: 15px;
6+
padding-top: 10px;
77
}
88

99
/* Space between buttons for vertical, single column */
10-
.filterButtonDiv > * {
11-
margin: 2.5% 0;
10+
.filterButtonDiv > a {
11+
margin: 2% 1%;
1212
}
1313

1414
.leadingParagraph {

src/scenes/home/codeSchools/onlineSchools/onlineSchools.css

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,8 @@
1313
padding: 2.5% 0 2.5% 0;
1414
}
1515

16-
.moocs > h3 {
16+
/* CodeClimate didn't like .moocs > h3 */
17+
.moocs:first-child {
1718
text-align: center;
1819
font-size: 1.8rem;
1920
}

src/shared/components/accordian/accordianItem/accordianItem.js

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,9 @@ class AccordianItem extends Component {
66
render() {
77
return (
88
<dt className={styles.accordianItem}>
9-
<a
10-
className={styles.accordianItemHeading}
11-
aria-expanded="false"
12-
aria-controls="accordion1"
13-
class="accordion-title accordionTitle js-accordionTrigger">First Accordion heading</a>
9+
<a className={styles.accordianItemHeading} aria-expanded="false" aria-controls="accordion1">
10+
First Accordion heading
11+
</a>
1412
</dt>
1513
);
1614
}

0 commit comments

Comments
 (0)