fix: a11y colors

adobe · Aug 1, 2024 · 1da2479 · 1da2479
1 parent 1734740
commit 1da2479
Show file tree

Hide file tree

Showing 10 changed files with 16 additions and 25 deletions.
diff --git a/blocks/accordion/accordion.css b/blocks/accordion/accordion.css
@@ -1,5 +1,5 @@
 .accordion details {
-  border: 1px solid var(--dark-color);
+  border: 1px solid #dadada;
 }
 
 /* stylelint-disable-next-line no-descending-specificity */
@@ -56,6 +56,6 @@
 }
 
 .accordion details[open] .accordion-item-body {
-  border-top: 1px solid var(--dark-color);
+  border-top: 1px solid #dadada;
   background-color: var(--background-color);
 }
diff --git a/blocks/cards/cards.css b/blocks/cards/cards.css
@@ -8,7 +8,7 @@
 }
 
 .cards > ul > li {
-  border: 1px solid var(--light-color);
+  border: 1px solid #dadada;
   background-color: var(--background-color);
 }
 

diff --git a/blocks/carousel/carousel.css b/blocks/carousel/carousel.css
@@ -77,7 +77,7 @@
   margin: 0;
   padding: 0;
   border-radius: 50%;
-  background-color: rgba(19 19 19 / 25%);
+  background-color: #dadada;
   transition: background-color 0.2s;
 }
 

diff --git a/blocks/footer/footer.css b/blocks/footer/footer.css
@@ -13,14 +13,6 @@ footer .footer p {
   margin: 0;
 }
 
-footer .footer a:any-link {
-  color: var(--link-hover-color);
-}
-
-footer .footer a:hover {
-  color: var(--link-color);
-}
-
 @media (width >= 900px) {
   footer .footer > div {
     padding: 40px 32px 24px;

diff --git a/blocks/header/header.css b/blocks/header/header.css
@@ -300,7 +300,7 @@ header .breadcrumbs ol {
 /* stylelint-disable-next-line no-descending-specificity */
 header .breadcrumbs ol li {
   padding: 0;
-  opacity: 0.6;
+  color: var(--dark-color);
 }
 
 header .breadcrumbs ol li:not(:last-child)::after {
@@ -310,7 +310,7 @@ header .breadcrumbs ol li:not(:last-child)::after {
 
 /* stylelint-disable-next-line no-descending-specificity */
 header .breadcrumbs ol li[aria-current] {
-  opacity: 1;
+  color: var(--text-color);
 }
 
 @media (width >= 900px) {

diff --git a/blocks/search/search.css b/blocks/search/search.css
@@ -34,7 +34,7 @@
 }
 
 .search ul.search-results > li {
-  border: 1px solid var(--light-color);
+  border: 1px solid #dadada;
 }
 
 .search ul.search-results > li > a {

diff --git a/blocks/table/table.css b/blocks/table/table.css
@@ -23,12 +23,12 @@
 }
 
 .table table thead tr {
-  border-top: 2px solid;
-  border-bottom: 2px solid;
+  border-top: 2px solid #dadada;
+  border-bottom: 2px solid #dadada;
 }
 
 .table table tbody tr {
-  border-bottom: 1px solid;
+  border-bottom: 1px solid #dadada;
 }
 
 .table table th {
@@ -52,7 +52,7 @@
 
 /* no header variant */
 .table.no-header table tbody tr {
-  border-top: 1px solid;
+  border-top: 1px solid #dadada;
 }
 
 /* striped variant */
@@ -63,5 +63,5 @@
 /* bordered variant */
 .table.bordered table th,
 .table.bordered table td {
-  border: 1px solid;
+  border: 1px solid #dadada;
 }
diff --git a/blocks/tabs/tabs.css b/blocks/tabs/tabs.css
@@ -21,7 +21,7 @@
 .tabs .tabs-list button {
   flex: 0 0 max-content;
   margin: 0;
-  border: 1px solid var(--dark-color);
+  border: 1px solid #dadada;
   border-radius: 0;
   padding: 0.5em;
   background-color: var(--light-color);
@@ -49,7 +49,7 @@
 .tabs .tabs-panel {
   margin-top: -1px;
   padding: 24px;
-  border: 1px solid var(--dark-color);
+  border: 1px solid #dadada;
   overflow: auto;
 }
 

diff --git a/blocks/video/video.css b/blocks/video/video.css
@@ -46,13 +46,12 @@
 }
 
 .video .video-placeholder-play button {
-  box-sizing: border-box;
   position: relative;
   display: block;
   width: 44px;
   height: 44px;
   border-radius: 50%;
-  border: 2px solid;
+  outline: 2px solid;
   padding: 0;
 }
 

diff --git a/styles/styles.css b/styles/styles.css
@@ -13,7 +13,7 @@
  :root {
   /* colors */
   --background-color: white;
-  --light-color: #f3f3f3;
+  --light-color: #f8f8f8;
   --dark-color: #505050;
   --text-color: #131313;
   --link-color: #3b63fb;