Skip to content

Commit

Permalink
Merge pull request #73 from edx/fsheets/EDUCATOR-1951
Browse files Browse the repository at this point in the history
Accessibility Feedback Updates
Including: using h2 instead of h1, adding a final period to error text, and moving a nested ol into an enclosing li correctly
  • Loading branch information
Farhanah Sheets authored Dec 11, 2017
2 parents caaf27c + 898a0bf commit d4c6a16
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 12 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@edx/studio-frontend",
"version": "0.6.1",
"version": "0.6.2",
"description": "The frontend for the Open edX platform",
"repository": "edx/studio-frontend",
"scripts": {
Expand Down
17 changes: 9 additions & 8 deletions src/components/AccessibilityBody/index.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -14,21 +14,22 @@ const AccessibilityBody = (props) => {
/* eslint-disable max-len */
return (
<div>
<h1 className={edxBootstrap['page-header']}>Individualized Accessibility Process for Course Creators</h1>
<h2 className={edxBootstrap['page-header']}>Individualized Accessibility Process for Course Creators</h2>
<p>
At edX, we seek to understand and respect the unique needs and perspectives of the edX global community. We value every course team and are committed to being a leader in expanding access to all, including course team creators and authors with disabilities. To that end, we have adopted a {communityAccessibilityElement} and this process to ensure that course team creators and authors with disabilities are able to develop and post content on our platform via Studio.
</p>
<p>
Course team creators and authors who, because of their disabilities, need assistance to use Studio should take the following steps:
</p>
<ol className={styles['numerical-list']}>
<li>Notify us by email addressed to {emailElement}. In order for edX to fully and promptly assess and respond to your request, please provide the following information:</li>
<ol className={styles['alphabetical-list']}>
<li>Your name and email address;</li>
<li>The edX member institution that you are affiliated with;</li>
<li>A brief description of the challenge or barrier to access that you are experiencing; and</li>
<li>How soon you need access and for how long (e.g., a planned course start date or in connection with a course-related deadline such as a final essay).</li>
</ol>
<li>Notify us by email addressed to {emailElement}. In order for edX to fully and promptly assess and respond to your request, please provide the following information:
<ol className={styles['alphabetical-list']}>
<li>Your name and email address;</li>
<li>The edX member institution that you are affiliated with;</li>
<li>A brief description of the challenge or barrier to access that you are experiencing; and</li>
<li>How soon you need access and for how long (e.g., a planned course start date or in connection with a course-related deadline such as a final essay).</li>
</ol>
</li>
<li>Within 48 business hours of receiving your request, the edX Support Team will respond to confirm receipt and forward your request to the edX Partner Manager for your institution and the edX Website Accessibility Coordinator.</li>
<li>With guidance from the Website Accessibility Coordinator, edX will contact you within 5 business days to discuss your request and gather additional information from you to identify a solution.</li>
<li>EdX will assist you promptly and thoroughly so that you are able to create content on the CMS within your time constraints. Such efforts may include, but are not limited to:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ describe('<AccessibilityPolicyForm />', () => {
const statusAlertType = statusAlert.prop('alertType');
expect(statusAlertType).toEqual('danger');
expect(statusAlert.find('div').first().hasClass('alert-danger')).toEqual(true);
expect(statusAlert.text()).toContain(`We are currently experiencing high volume. Try again later today or send an email message to ${wrapper.props().accessibilityEmail}`);
expect(statusAlert.text()).toContain(`We are currently experiencing high volume. Try again later today or send an email message to ${wrapper.props().accessibilityEmail}.`);
});

it('clears inputs on successful submit', () => {
Expand Down
4 changes: 2 additions & 2 deletions src/components/AccessibilityPolicyForm/index.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ export class AccessibilityPolicyForm extends React.Component {
alertDialog: (
<div>
We are currently experiencing high volume. Try again later today or
send an email message to <a href={`mailto:${accessibilityEmail}`}>{accessibilityEmail}</a>
send an email message to <a href={`mailto:${accessibilityEmail}`}>{accessibilityEmail}</a>.
</div>
),
};
Expand Down Expand Up @@ -225,7 +225,7 @@ export class AccessibilityPolicyForm extends React.Component {
render() {
return (
<div>
<h1 id="a11y-feedback" className={styles['page-header']}>Studio Accessibility Feedback</h1>
<h2 id="a11y-feedback" className={styles['page-header']}>Studio Accessibility Feedback</h2>
{this.renderStatusAlert()}
<section className={styles['form-section']} role="group" aria-labelledby="a11y-feedback">
<p>All fields are required.</p>
Expand Down

0 comments on commit d4c6a16

Please sign in to comment.