fix(FEC-13022): fix link and selection accessibility (#740)

solves FEC-13022
kaltura · Mar 15, 2023 · 10ddcc8 · 10ddcc8
1 parent fb7159a
commit 10ddcc8
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 6 deletions.
diff --git a/src/components/icon/icon.scss b/src/components/icon/icon.scss
@@ -144,7 +144,6 @@
 
 // Dynamic Colored Icons
 .player {
-  --playkit-icon-check-active-url: #{icon(check, $brand-color)};
   --playkit-icon-vr-stereo-full-url: #{icon(vrStereoFull, $brand-color)};
   --playkit-icon-chromecast-url: #{icon(chromecast, $brand-color)};
   --playkit-icon-quality-HD-active-url: #{icon(qualityHdActive, $brand-color, '16', '0 0 16 16')};
@@ -197,7 +196,7 @@
 }
 
 .icon-check-active {
-  background-image: var(--playkit-icon-check-active-url);
+  background-image:  icon(check, '#fff');
 }
 
 .icon-language {

diff --git a/src/components/language/_language.scss b/src/components/language/_language.scss
@@ -1,7 +1,7 @@
 .player {
   a:not([href]) {
     &.advanced-captions-menu-link {
-      color: $primary-brighter-color;
+      color: $tone-1-color;
       text-decoration: underline;
     }
   }

diff --git a/src/styles/_dropdown.scss b/src/styles/_dropdown.scss
@@ -109,8 +109,6 @@
     }
 
     &.active {
-      color: $primary-color;
-
       .menu-icon-container {
         opacity: 1;
       }

diff --git a/src/utils/themes-manager.js b/src/utils/themes-manager.js
@@ -19,7 +19,6 @@ const cssVarNames = {
 const ACCENT_AND_ACKNOWLEDGEMENT_COLORS = ['primary', 'secondary', 'success', 'danger', 'warning'];
 
 const dynamicColoredIconsSvgUrlVars = [
-  '--playkit-icon-check-active-url',
   '--playkit-icon-data-url',
   '--playkit-icon-chromecast-url',
   '--playkit-icon-quality-HD-active-url',