Merge pull request #309 from flacoman91/accessibility-updates

Accessibility updates
cfpb · Jul 10, 2020 · 736d575 · 736d575
2 parents 1bce7d6 + 2d04a5f
commit 736d575
Show file tree

Hide file tree

Showing 16 changed files with 15 additions and 40 deletions.
diff --git a/src/__tests__/__snapshots__/App.spec.jsx.snap b/src/__tests__/__snapshots__/App.spec.jsx.snap
@@ -1215,7 +1215,6 @@ exports[`initial state renders without crashing 1`] = `
               <button
                 className="a-btn"
                 onClick={[Function]}
-                title="Close Filters"
               >
                 Close Filters
               </button>

diff --git a/src/components/Charts/LineChart.less b/src/components/Charts/LineChart.less
@@ -80,7 +80,7 @@
     p {
         font-size: 12px;
         font-weight: 600;
-        color: @gray-80;
+        color: @gray;
 
         &.y-axis-label {
             margin-left: @gutter-normal;

diff --git a/src/components/Charts/StackedAreaChart.less b/src/components/Charts/StackedAreaChart.less
@@ -15,7 +15,7 @@
   p {
     font-size: 12px;
     font-weight: 600;
-    color: @gray-80;
+    color: @gray;
 
     &.y-axis-label {
       margin-left: @gutter-normal;

diff --git a/src/components/DateInput/DateInput.less b/src/components/DateInput/DateInput.less
@@ -4,4 +4,7 @@
   .a-text-input {
     padding-right: @grid_gutter-width;
   }
+  .a-btn {
+    color: @gray;
+  }
 }
diff --git a/src/components/Filters/AggregationBranch.jsx b/src/components/Filters/AggregationBranch.jsx
@@ -95,8 +95,7 @@ export class AggregationBranch extends React.Component {
             <span className="u-visually-hidden">{item.key}</span>
           </label>
           <button className="flex-all a-btn a-btn__link"
-                  onClick={this._toggleChildDisplay}
-                  title={item.key}>
+                  onClick={this._toggleChildDisplay}>
             <span>{item.key}</span>
             {chevronIcon}
           </button>

diff --git a/src/components/Filters/FilterPanelToggle.jsx b/src/components/Filters/FilterPanelToggle.jsx
@@ -11,8 +11,6 @@ export class FilterPanelToggle extends React.Component {
           <p>&nbsp;</p>
           <button
             className={ 'a-btn' }
-            title={ this.props.showFilters ? 'Close Filters' :
-              'Filter results' }
             onClick={ () => this.props.onFilterToggle() }>
             { this.props.showFilters ? 'Close Filters' : 'Filter results' }
           </button>

diff --git a/src/components/Filters/__tests__/__snapshots__/AggregationBranch.spec.jsx.snap b/src/components/Filters/__tests__/__snapshots__/AggregationBranch.spec.jsx.snap
@@ -28,7 +28,6 @@ exports[`component::AggregationBranch snapshots renders with all checked 1`] = `
     <button
       className="flex-all a-btn a-btn__link"
       onClick={[Function]}
-      title="foo"
     >
       <span>
         foo
@@ -161,7 +160,6 @@ exports[`component::AggregationBranch snapshots renders with indeterminate 1`] =
     <button
       className="flex-all a-btn a-btn__link"
       onClick={[Function]}
-      title="foo"
     >
       <span>
         foo
@@ -294,7 +292,6 @@ exports[`component::AggregationBranch snapshots renders with none checked 1`] =
     <button
       className="flex-all a-btn a-btn__link"
       onClick={[Function]}
-      title="foo"
     >
       <span>
         foo

diff --git a/src/components/Filters/__tests__/__snapshots__/Issue.spec.jsx.snap b/src/components/Filters/__tests__/__snapshots__/Issue.spec.jsx.snap
@@ -104,7 +104,6 @@ exports[`component:Issue snapshots only shows the first five items 1`] = `
           <button
             className="flex-all a-btn a-btn__link"
             onClick={[Function]}
-            title="Incorrect information on credit report"
           >
             <span>
               Incorrect information on credit report
@@ -155,7 +154,6 @@ exports[`component:Issue snapshots only shows the first five items 1`] = `
           <button
             className="flex-all a-btn a-btn__link"
             onClick={[Function]}
-            title="Cont'd attempts collect debt not owed"
           >
             <span>
               Cont'd attempts collect debt not owed

diff --git a/src/components/Filters/__tests__/__snapshots__/Product.spec.jsx.snap b/src/components/Filters/__tests__/__snapshots__/Product.spec.jsx.snap
@@ -65,7 +65,6 @@ exports[`component:Product snapshots only shows the first five items 1`] = `
           <button
             className="flex-all a-btn a-btn__link"
             onClick={[Function]}
-            title="Credit reporting, credit repair services, or other personal consumer reports"
           >
             <span>
               Credit reporting, credit repair services, or other personal consumer reports
@@ -116,7 +115,6 @@ exports[`component:Product snapshots only shows the first five items 1`] = `
           <button
             className="flex-all a-btn a-btn__link"
             onClick={[Function]}
-            title="Mortgage"
           >
             <span>
               Mortgage

diff --git a/src/components/List/ListPanel.less b/src/components/List/ListPanel.less
@@ -13,4 +13,12 @@
   h2 {
     padding: @gutter-normal;    
   }
+
+  .m-pagination {
+    .a-btn[disabled],
+    .m-pagination_label,
+    .m-pagination_btn-submit {
+      color: @gray;
+    }
+  }
 }
diff --git a/src/components/RefineBar/RefineBar.less b/src/components/RefineBar/RefineBar.less
@@ -6,7 +6,7 @@
     p {
       font-size: @size-vi;
       font-weight: 600;
-      color: @gray-40;
+      color: @gray;
     }
     button {
       &.selected {

diff --git a/src/components/__tests__/__snapshots__/FilterPanelToggle.spec.jsx.snap b/src/components/__tests__/__snapshots__/FilterPanelToggle.spec.jsx.snap
@@ -13,7 +13,6 @@ exports[`initial state renders without crashing 1`] = `
     <button
       className="a-btn"
       onClick={[Function]}
-      title="Filter results"
     >
       Filter results
     </button>

diff --git a/src/components/__tests__/__snapshots__/ListPanel.spec.jsx.snap b/src/components/__tests__/__snapshots__/ListPanel.spec.jsx.snap
@@ -73,7 +73,6 @@ exports[`component:ListPanel displays a message when an error has occurred 1`] =
         <button
           className="a-btn"
           onClick={[Function]}
-          title="Filter results"
         >
           Filter results
         </button>
@@ -377,7 +376,6 @@ exports[`component:ListPanel displays a message when only the narratives are sta
         <button
           className="a-btn"
           onClick={[Function]}
-          title="Filter results"
         >
           Filter results
         </button>
@@ -857,7 +855,6 @@ exports[`component:ListPanel displays a message when the data has issues 1`] = `
         <button
           className="a-btn"
           onClick={[Function]}
-          title="Filter results"
         >
           Filter results
         </button>
@@ -1337,7 +1334,6 @@ exports[`component:ListPanel displays a message when the data is stale 1`] = `
         <button
           className="a-btn"
           onClick={[Function]}
-          title="Filter results"
         >
           Filter results
         </button>
@@ -1792,7 +1788,6 @@ exports[`component:ListPanel displays a message when there are no results 1`] =
         <button
           className="a-btn"
           onClick={[Function]}
-          title="Filter results"
         >
           Filter results
         </button>
@@ -2077,7 +2072,6 @@ exports[`component:ListPanel only displays data message when both types are stal
         <button
           className="a-btn"
           onClick={[Function]}
-          title="Filter results"
         >
           Filter results
         </button>
@@ -2533,7 +2527,6 @@ exports[`component:ListPanel renders mobile filters without crashing 1`] = `
         <button
           className="a-btn"
           onClick={[Function]}
-          title="Filter results"
         >
           Filter results
         </button>
@@ -2988,7 +2981,6 @@ exports[`component:ListPanel renders without crashing 1`] = `
         <button
           className="a-btn"
           onClick={[Function]}
-          title="Filter results"
         >
           Filter results
         </button>

diff --git a/src/components/__tests__/__snapshots__/MapPanel.spec.jsx.snap b/src/components/__tests__/__snapshots__/MapPanel.spec.jsx.snap
@@ -73,7 +73,6 @@ exports[`component:MapPanel renders Print without crashing 1`] = `
         <button
           className="a-btn"
           onClick={[Function]}
-          title="Filter results"
         >
           Filter results
         </button>
@@ -295,7 +294,6 @@ exports[`component:MapPanel renders warning without crashing 1`] = `
         <button
           className="a-btn"
           onClick={[Function]}
-          title="Filter results"
         >
           Filter results
         </button>
@@ -480,7 +478,6 @@ exports[`component:MapPanel renders without crashing 1`] = `
         <button
           className="a-btn"
           onClick={[Function]}
-          title="Filter results"
         >
           Filter results
         </button>

diff --git a/src/components/__tests__/__snapshots__/ResultsPanel.spec.jsx.snap b/src/components/__tests__/__snapshots__/ResultsPanel.spec.jsx.snap
@@ -102,7 +102,6 @@ exports[`component:Results renders List print mode without crashing 1`] = `
           <button
             className="a-btn"
             onClick={[Function]}
-            title="Filter results"
           >
             Filter results
           </button>
@@ -597,7 +596,6 @@ exports[`component:Results renders Map print mode without crashing 1`] = `
           <button
             className="a-btn"
             onClick={[Function]}
-            title="Filter results"
           >
             Filter results
           </button>
@@ -797,7 +795,6 @@ exports[`component:Results renders list panel without crashing 1`] = `
           <button
             className="a-btn"
             onClick={[Function]}
-            title="Filter results"
           >
             Filter results
           </button>
@@ -1255,7 +1252,6 @@ exports[`component:Results renders map panel without crashing 1`] = `
           <button
             className="a-btn"
             onClick={[Function]}
-            title="Filter results"
           >
             Filter results
           </button>
@@ -1444,7 +1440,6 @@ exports[`component:Results renders trends panel without crashing 1`] = `
           <button
             className="a-btn"
             onClick={[Function]}
-            title="Filter results"
           >
             Filter results
           </button>

diff --git a/src/components/__tests__/__snapshots__/TrendsPanel.spec.jsx.snap b/src/components/__tests__/__snapshots__/TrendsPanel.spec.jsx.snap
@@ -73,7 +73,6 @@ exports[`component:TrendsPanel Snapshots renders Focus without crashing 1`] = `
         <button
           className="a-btn"
           onClick={[Function]}
-          title="Filter results"
         >
           Filter results
         </button>
@@ -489,7 +488,6 @@ exports[`component:TrendsPanel Snapshots renders area without crashing 1`] = `
         <button
           className="a-btn"
           onClick={[Function]}
-          title="Filter results"
         >
           Filter results
         </button>
@@ -872,7 +870,6 @@ exports[`component:TrendsPanel Snapshots renders company Overlay without crashin
         <button
           className="a-btn"
           onClick={[Function]}
-          title="Filter results"
         >
           Filter results
         </button>
@@ -1268,7 +1265,6 @@ exports[`component:TrendsPanel Snapshots renders date warning without crashing 1
         <button
           className="a-btn"
           onClick={[Function]}
-          title="Filter results"
         >
           Filter results
         </button>
@@ -1533,7 +1529,6 @@ exports[`component:TrendsPanel Snapshots renders external Tooltip without crashi
         <button
           className="a-btn"
           onClick={[Function]}
-          title="Filter results"
         >
           Filter results
         </button>
@@ -1916,7 +1911,6 @@ exports[`component:TrendsPanel Snapshots renders lineChart Overview without cras
         <button
           className="a-btn"
           onClick={[Function]}
-          title="Filter results"
         >
           Filter results
         </button>
@@ -2182,7 +2176,6 @@ exports[`component:TrendsPanel Snapshots renders mobile filters without crashing
         <button
           className="a-btn"
           onClick={[Function]}
-          title="Filter results"
         >
           Filter results
         </button>
@@ -2447,7 +2440,6 @@ exports[`component:TrendsPanel Snapshots renders print mode without crashing 1`]
         <button
           className="a-btn"
           onClick={[Function]}
-          title="Filter results"
         >
           Filter results
         </button>