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.

Sign up for GitHub

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

Jump to bottom

Bug/fix some a11y issues #121

Merged
merged 3 commits into from
Aug 20, 2020
Merged

Bug/fix some a11y issues #121

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
f17b38c
Fixed a11y issue linking of label for position type.
nickytonline Aug 19, 2020
c853ec5
Fixed some color contrast issues.
nickytonline Aug 19, 2020
e47a47d
Fixed some more color contrast issues.
nickytonline Aug 19, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions src/assets/main.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5100,7 +5100,7 @@ video {
}

.text-blue-500{
color: #848a9f;
color: #55556D;
}

.text-blue-600{
Expand Down Expand Up @@ -8229,7 +8229,7 @@ video {
}

.btn-teal {
background-color: #8ddbc1;
background-color: #9EE0CA;
color: #323c5f;
}

Expand Down Expand Up @@ -8271,7 +8271,7 @@ video {
}

.input-error {
color: #E53E3E;
color: #AF1818;
font-size: 0.75rem;
}

Expand Down
7 changes: 4 additions & 3 deletions src/components/form/PostAJobForm.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -55,8 +55,9 @@ const PostAJobForm = ({
companyDescription: Yup.string().required(
'Please give a brief description of the company and culture.'
),
companyLogo: Yup.mixed().required('Please provide a .png format image of your company logo')
.test(file => file && file.type === 'image/png'),
companyLogo: Yup.mixed()
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Prettier in action.

.required('Please provide a .png format image of your company logo')
.test((file) => file && file.type === 'image/png'),
companyHQ: Yup.string().required(
'Please provide a location for your office headquarters.'
),
Expand Down Expand Up @@ -140,7 +141,7 @@ const PostAJobForm = ({

<div className='flex flex-col md:w-1/2 mb-3'>
<label
htmlFor='positionType'
htmlFor='position-type'
className='text-blue-900 font-semibold'
>
Position Type
Expand Down