Skip to content

Commit a3832bb

Browse files
smoralesddbjorge
authored andcommitted
Revert "feat: update requirement content for Native and Custom Widgets, Instructions requirement (#1116)" (#1120)
This reverts commit 1d46d6d.
1 parent 01ca256 commit a3832bb

File tree

5 files changed

+24
-69
lines changed

5 files changed

+24
-69
lines changed

src/assessments/common/instructions-and-labels-note.tsx

Lines changed: 0 additions & 13 deletions
This file was deleted.

src/assessments/custom-widgets/test-steps/instructions.tsx

Lines changed: 10 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,19 @@
11
// Copyright (c) Microsoft Corporation. All rights reserved.
22
// Licensed under the MIT License.
3+
import * as React from 'react';
4+
35
import { CustomWidgetPropertyBag } from 'common/types/property-bag/icustom-widgets';
46
import { VisualizationType } from 'common/types/visualization-type';
57
import { link } from 'content/link';
8+
import { productName } from 'content/strings/application';
9+
import { TestAutomaticallyPassedNotice } from 'content/test/common/test-automatically-passed-notice';
610
import * as content from 'content/test/custom-widgets/instructions';
711
import { AssessmentVisualizationEnabledToggle } from 'DetailsView/components/assessment-visualization-enabled-toggle';
812
import { ScannerUtils } from 'injected/scanner-utils';
9-
import * as React from 'react';
10-
11-
import { NewTabLink } from '../../../common/components/new-tab-link';
1213
import { AnalyzerConfigurationFactory } from '../../common/analyzer-configuration-factory';
1314
import { AssistedTestRecordYourResults } from '../../common/assisted-test-record-your-results';
14-
import { InstructionsAndLabelsNotes } from '../../common/instructions-and-labels-note';
1515
import { NoValue } from '../../common/property-bag-column-renderer';
16+
import * as Markup from '../../markup';
1617
import { ReportInstanceField } from '../../types/report-instance-field';
1718
import { Requirement } from '../../types/requirement';
1819
import { getFlatDesignPatternStringFromRole } from '../custom-widgets-column-renderer';
@@ -25,19 +26,12 @@ const instructionsDescription: JSX.Element = (
2526

2627
const instructionsHowToTest: JSX.Element = (
2728
<div>
28-
<p>
29-
The visual helper for this requirement highlights custom widgets. (A custom widget is an element with a valid{' '}
30-
<NewTabLink href="https://www.w3.org/TR/wai-aria/#widget_roles">ARIA widget role</NewTabLink>.)
31-
</p>
32-
<InstructionsAndLabelsNotes />
29+
<p>For this requirement, {productName} highlights custom widgets.</p>
30+
<TestAutomaticallyPassedNotice />
3331
<ol>
34-
<li>In the target page, examine each highlighted element to determine whether it has a visible label or instructions.</li>
3532
<li>
36-
Verify that all visible labels and instructions are displayed in the Instances list:
37-
<ol>
38-
<li>Any label should appear in the accessible name.</li>
39-
<li>Any additional instructions should appear in the accessible description.</li>
40-
</ol>
33+
For each widget, verify that any instructions visible in the target page are also visible in the{' '}
34+
<Markup.Term>Instances</Markup.Term> list.
4135
</li>
4236
<AssistedTestRecordYourResults />
4337
</ol>
@@ -50,7 +44,7 @@ export const Instructions: Requirement = {
5044
description: instructionsDescription,
5145
howToTest: instructionsHowToTest,
5246
isManual: false,
53-
guidanceLinks: [link.WCAG_1_3_1, link.WCAG_2_5_3],
47+
guidanceLinks: [link.WCAG_1_3_1],
5448
...content,
5549
columnsConfig: [
5650
{

src/assessments/native-widgets/test-steps/instructions.tsx

Lines changed: 14 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,49 +1,44 @@
11
// Copyright (c) Microsoft Corporation. All rights reserved.
22
// Licensed under the MIT License.
3+
import * as React from 'react';
4+
35
import { DefaultWidgetPropertyBag } from 'common/types/property-bag/default-widget';
46
import { VisualizationType } from 'common/types/visualization-type';
57
import { link } from 'content/link';
8+
import { productName } from 'content/strings/application';
9+
import { TestAutomaticallyPassedNotice } from 'content/test/common/test-automatically-passed-notice';
610
import * as content from 'content/test/native-widgets/instructions';
711
import { AssessmentVisualizationEnabledToggle } from 'DetailsView/components/assessment-visualization-enabled-toggle';
812
import { ScannerUtils } from 'injected/scanner-utils';
9-
import * as React from 'react';
10-
1113
import { AnalyzerConfigurationFactory } from '../../common/analyzer-configuration-factory';
1214
import { AssistedTestRecordYourResults } from '../../common/assisted-test-record-your-results';
13-
import { InstructionsAndLabelsNotes } from '../../common/instructions-and-labels-note';
1415
import { NoValue, PropertyBagColumnRendererConfig } from '../../common/property-bag-column-renderer';
1516
import { PropertyBagColumnRendererFactory } from '../../common/property-bag-column-renderer-factory';
1617
import * as Markup from '../../markup';
1718
import { ReportInstanceField } from '../../types/report-instance-field';
1819
import { Requirement } from '../../types/requirement';
1920
import { NativeWidgetsTestStep } from './test-steps';
2021

21-
const description: JSX.Element = (
22-
<span>If a native widget has a visible label or instructions, they must be programmatically related.</span>
23-
);
22+
const description: JSX.Element = <span>If a native widget has visible instructions, they must be programmatically related to it.</span>;
2423

2524
const howToTest: JSX.Element = (
2625
<div>
2726
<p>
28-
The visual helper for this requirement highlights native widgets. Native widgets include
27+
For this requirement, {productName} highlights native widgets. Native widgets include
2928
<Markup.NonBreakingSpace />
30-
<Markup.Tag tagName="button" isBold={false} />,
29+
<Markup.Tag tagName="button" isBold={true} />,
3130
<Markup.NonBreakingSpace />
32-
<Markup.Tag tagName="input" isBold={false} />,
31+
<Markup.Tag tagName="input" isBold={true} />,
3332
<Markup.NonBreakingSpace />
34-
<Markup.Tag tagName="select" isBold={false} />, and
33+
<Markup.Tag tagName="select" isBold={true} />, and
3534
<Markup.NonBreakingSpace />
36-
<Markup.Tag tagName="textarea" isBold={false} /> elements.
35+
<Markup.Tag tagName="textarea" isBold={true} /> elements.
3736
</p>
38-
<InstructionsAndLabelsNotes />
37+
<TestAutomaticallyPassedNotice />
3938
<ol>
40-
<li>In the target page, examine each highlighted element to determine whether it has a visible label or instructions.</li>
4139
<li>
42-
Verify that all visible labels and instructions are displayed in the Instances list:
43-
<ol>
44-
<li>Any label should appear in the accessible name.</li>
45-
<li>Any additional instructions should appear in the accessible description.</li>
46-
</ol>
40+
For each widget, verify that any instructions visible in the target page are also visible in the
41+
<Markup.Term> Instances</Markup.Term> list.
4742
</li>
4843
<AssistedTestRecordYourResults />
4944
</ol>
@@ -74,7 +69,7 @@ export const Instructions: Requirement = {
7469
description,
7570
howToTest,
7671
isManual: false,
77-
guidanceLinks: [link.WCAG_1_3_1, link.WCAG_2_5_3],
72+
guidanceLinks: [link.WCAG_1_3_1],
7873
...content,
7974
columnsConfig: [
8075
{

src/tests/unit/tests/assessments/common/__snapshots__/instructions-and-labels-note.test.tsx.snap

Lines changed: 0 additions & 7 deletions
This file was deleted.

src/tests/unit/tests/assessments/common/instructions-and-labels-note.test.tsx

Lines changed: 0 additions & 14 deletions
This file was deleted.

0 commit comments

Comments
 (0)