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

Commit 3bb02b7

Browse files
committed
Various fixes to appease CodeClimate check
1 parent c831640 commit 3bb02b7

16 files changed

Lines changed: 47 additions & 35 deletions

File tree

src/index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@ import React from 'react';
22
import ReactDOM from 'react-dom';
33
import 'font-awesome/css/font-awesome.css';
44
import App from './App';
5+
import './normalize.css';
56
import './index.css';
67
import './index.global.css';
7-
import './normalize.css';
88

99
ReactDOM.render(
1010
<App />,

src/normalize.css

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -271,14 +271,15 @@ select { /* 1 */
271271
/**
272272
* 1. Prevent a WebKit bug where (2) destroys native `audio` and `video`
273273
* controls in Android 4.
274-
* 2. Correct the inability to style clickable types in iOS and Safari.
274+
* 2. Correct the inability to style clickable types in iOS, Firefox, and Safari.
275275
*/
276276

277277
button,
278278
html [type="button"], /* 1 */
279279
[type="reset"],
280280
[type="submit"] {
281281
-webkit-appearance: button; /* 2 */
282+
-moz-appearance: button;
282283
}
283284

284285
/**
@@ -367,31 +368,34 @@ textarea {
367368
}
368369

369370
/**
370-
* 1. Correct the odd appearance in Chrome and Safari.
371+
* 1. Correct the odd appearance in Chrome, Firefox, and Safari.
371372
* 2. Correct the outline style in Safari.
372373
*/
373374

374375
[type="search"] {
375376
-webkit-appearance: textfield; /* 1 */
377+
-moz-appearance: textfield;
376378
outline-offset: -2px; /* 2 */
377379
}
378380

379381
/**
380-
* Remove the inner padding and cancel buttons in Chrome and Safari on macOS.
382+
* Remove the inner padding and cancel buttons in Chrome, Firefox, and Safari on macOS.
381383
*/
382384

383385
[type="search"]::-webkit-search-cancel-button,
384386
[type="search"]::-webkit-search-decoration {
385387
-webkit-appearance: none;
388+
-moz-appearance: none;
386389
}
387390

388391
/**
389-
* 1. Correct the inability to style clickable types in iOS and Safari.
392+
* 1. Correct the inability to style clickable types in iOS, Firefox, and Safari.
390393
* 2. Change font properties to `inherit` in Safari.
391394
*/
392395

393396
::-webkit-file-upload-button {
394397
-webkit-appearance: button; /* 1 */
398+
-moz-appearance: button;
395399
font: inherit; /* 2 */
396400
}
397401

src/scenes/home/codeSchools/approvedSchools/approvedSchools.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ class ApprovedSchools extends Component {
5252

5353
return (
5454
<Section
55+
id="approvedSchools"
5556
title="VA-Approved Schools"
5657
headingLines={false}
5758
margin

src/scenes/home/codeSchools/codeSchools.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -50,24 +50,24 @@ class CodeSchools extends Component {
5050

5151
<div className={styles.filterButtonDiv}>
5252
<LinkButton
53+
link="#approvedSchools"
5354
text="Code Schools with VA Approval"
5455
theme="blue"
55-
link="#schoolsByApproval"
5656
/>
5757
<LinkButton
58+
link="#partnerSchools"
5859
text="Partner Schools with Scholarships"
5960
theme="blue"
60-
link="#schoolsByScholarship"
6161
/>
6262
<LinkButton
63+
link="#onlineSchools"
6364
text="Online Schools"
6465
theme="blue"
65-
link="#onlineSchools"
6666
/>
6767
<LinkButton
68+
link="#schoolsByState"
6869
text="Search by State"
6970
theme="blue"
70-
link="#schoolsByState"
7171
/>
7272
</div>
7373
</Section>

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

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

16-
.moocs h3 {
16+
.moocs > h3 {
1717
text-align: center;
1818
font-size: 1.8rem;
1919
}
@@ -30,6 +30,6 @@
3030
}
3131

3232
.boxShadow {
33-
box-shadow: 0px 3px 10px 1px rgba(0, 0, 0, 0.1);
33+
box-shadow: 0 3px 10px 1px rgba(0, 0, 0, 0.1);
3434
transition: all 0.3s cubic-bezier(.25,.8,.25,1);
3535
}

src/scenes/home/codeSchools/partnerSchools/partnerSchools.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ class PartnerSchools extends Component {
1515
render() {
1616
return (
1717
<Section
18-
id="schoolsByScholarship"
18+
id="partnerSchools"
1919
title="Partnered Schools"
2020
theme="white"
2121
headingLines={false}
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
.accordian {
2-
2+
border: #555 solid;
33
}
Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
.accordianItem {
2-
2+
border: #888 black;
33
}
44

55
.accordianItemHeading {
6-
font-size:26px;
7-
padding:20px;
8-
text-align:center;
6+
font-size: 26px;
7+
padding: 20px;
8+
text-align: center;
99
}
Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
11
.formInput {
22
composes: formControl from '../form.css';
3-
}
3+
}
4+
5+
.formInput > input::placeholder {
6+
color: #999;
7+
}

src/shared/components/form/formInput/formInput.js

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -35,14 +35,15 @@ class FormInput extends Component {
3535
return (
3636
<div className={styles.formInput}>
3737
{this.props.label && <Label htmlFor={this.props.id}>{this.props.label}</Label>}
38+
3839
<input
3940
id={this.props.id}
4041
type={this.props.inputType}
4142
value={this.state.text}
4243
placeholder={this.props.placeholder}
4344
onChange={this.handleChange}
4445
/>
45-
{ !this.state.isValid && <span>{this.props.validationErrorMessage}</span>}
46+
{!this.state.isValid && <span>{this.props.validationErrorMessage}</span>}
4647
</div>
4748
);
4849
}
@@ -66,10 +67,7 @@ FormInput.defaultProps = {
6667
validationErrorMessage: null,
6768
validateFunc: null,
6869
onChange: null,
69-
intputType: "text"
70+
inputType: "text"
7071
};
7172

72-
// TODO: Add prop for icon placement on left-side of input bar
73-
// TODO: Decrease opacity of placeholder text
74-
7573
export default FormInput;

0 commit comments

Comments
 (0)