diff --git a/.eslintrc b/.eslintrc
index 1c5b0dd..9fa4a67 100644
--- a/.eslintrc
+++ b/.eslintrc
@@ -14,5 +14,8 @@
"expect": false,
"afterEach": false,
"jest": false
+ },
+ "env": {
+ "browser": true
}
}
diff --git a/assets/styles/base/forms.scss b/assets/styles/base/forms.scss
index 5c75c38..c756c7c 100644
--- a/assets/styles/base/forms.scss
+++ b/assets/styles/base/forms.scss
@@ -5,3 +5,7 @@ button {
font-family: $primary-font;
}
+button {
+ cursor: pointer;
+ outline: none;
+}
diff --git a/components/interviews/interviewsList/index.js b/components/interviews/interviewsList/index.js
index f122b64..04f1ff1 100644
--- a/components/interviews/interviewsList/index.js
+++ b/components/interviews/interviewsList/index.js
@@ -3,14 +3,13 @@ import { PropTypes } from 'prop-types';
import Modal from '../../modal';
import './style.scss';
-const InterviewsList = props => {
-
+const InterviewsList = (props) => {
// Sort interviews alphabetically
- const sortedInterviews = props.data.sort((a, b) => { return a.name.localeCompare(b.name); });
+ const sortedInterviews = props.data.sort((a, b) => a.name.localeCompare(b.name));
const interviews = {};
// Build up interviews object with letter key
- sortedInterviews.forEach(interview => {
+ sortedInterviews.forEach((interview) => {
const firstLetter = interview.name.charAt(0);
if (typeof interviews[firstLetter] === 'undefined') {
interviews[firstLetter] = [];
@@ -52,34 +51,31 @@ const InterviewsList = props => {
{ firstLetter }
- {interviews[firstLetter].map(interview => (
- // eslint-disable-next-line
+ { interviews[firstLetter].map(interview => (
-
- {interview.name}
+
- )
- )}
+ ))
+ }
- )
- )
+ ))
}
{
Object.keys(interviews).map(firstLetter => (
+ // eslint-disable-next-line
{ document.querySelector(`#${firstLetter}`).scrollIntoView({ behavior: 'smooth' }); }}
>
{ firstLetter }
- )
- )
+ ))
}
@@ -92,7 +88,7 @@ const InterviewsList = props => {
InterviewsList.propTypes = {
isInterviewsListModalOpen: PropTypes.bool.isRequired,
data: PropTypes.arrayOf(PropTypes.shape({})).isRequired,
- toggleSingleInterview: PropTypes.func.isRequired,
+ toggleSingleInterview: PropTypes.func.isRequired, /* eslint-disable-line */
toggleInterviewsListModal: PropTypes.func.isRequired,
};
diff --git a/components/interviews/interviewsList/style.scss b/components/interviews/interviewsList/style.scss
index 60be0f0..cb8482c 100644
--- a/components/interviews/interviewsList/style.scss
+++ b/components/interviews/interviewsList/style.scss
@@ -41,9 +41,11 @@
padding-bottom: calculateRem(32);
}
- li {
- cursor: pointer;
- outline: none;
+ button {
+ border: 0;
+ padding: 0;
+ text-align: left;
+ font-size: calculateRem(14);
&:hover {
text-decoration: underline;
diff --git a/components/interviews/relatedInterviewsList/index.js b/components/interviews/relatedInterviewsList/index.js
index 3372dee..3df5d17 100644
--- a/components/interviews/relatedInterviewsList/index.js
+++ b/components/interviews/relatedInterviewsList/index.js
@@ -7,13 +7,11 @@ const RelatedInterviewsList = props => (
Related
Interviews
{ props.data.map(interview => (
- // eslint-disable-next-line
-
- { interview.name }
+
))
}
diff --git a/components/interviews/relatedInterviewsList/style.scss b/components/interviews/relatedInterviewsList/style.scss
index 331805f..7e4007f 100644
--- a/components/interviews/relatedInterviewsList/style.scss
+++ b/components/interviews/relatedInterviewsList/style.scss
@@ -3,10 +3,11 @@
.related-interviews-list {
margin-bottom: calculateRem(24);
- li {
+ button {
+ border: 0;
+ padding: 0;
+ text-align: left;
font-size: calculateRem(14);
- cursor: pointer;
- outline: none;
&:hover {
text-decoration: underline;
diff --git a/components/searchResults/index.js b/components/searchResults/index.js
index b1f193f..f52b045 100644
--- a/components/searchResults/index.js
+++ b/components/searchResults/index.js
@@ -8,7 +8,7 @@ const SearchResults = (props) => {
}
// sort array alphabetically
- const sortedInterviews = props.data.sort((a, b) => { return a.name.localeCompare(b.name); });
+ const sortedInterviews = props.data.sort((a, b) => a.name.localeCompare(b.name));
return (