[v4] [core] fix(Tag): fix colors for remove icon in minimal tags (#4992)

palantir · Oct 28, 2021 · 6965714 · 6965714
1 parent 315c5cd
commit 6965714
Showing 1 changed file with 19 additions and 9 deletions.
diff --git a/packages/core/src/blueprint.scss b/packages/core/src/blueprint.scss
@@ -79,7 +79,7 @@ $minimal-dark-tag-intent-colors-modern: (
   "primary": ($pt-intent-primary, $blue5, $blue6, $blue6),
   "success": ($pt-intent-success, $green5, $green6, $green6),
   "warning": ($pt-intent-warning, $orange5, $orange6, $orange6),
-  "danger": ($pt-intent-danger, $red5, $red1, $red6, $red6)
+  "danger": ($pt-intent-danger, $red5, $red6, $red6)
 );
 
 // Toast variables
@@ -452,6 +452,7 @@ table.#{$ns}-html-table {
   ) {
     background: $background-color;
     color: $text-color;
+
     &.#{$ns}-interactive {
       &:hover {
         background-color: $hover-color;
@@ -548,6 +549,14 @@ table.#{$ns}-html-table {
   // Contrast for default intent
   color: $white;
 
+  .#{$ns}-tag-remove {
+    &,
+    &:hover,
+    &:active {
+      opacity: 1;
+    }
+  }
+
   &.#{$ns}-interactive {
     &:hover {
       background: $dark-gray5;
@@ -610,6 +619,7 @@ table.#{$ns}-html-table {
     // Minimal style for default intent
     background-color: rgba($gray3, 0.15);
     color: $pt-text-color;
+
     &.#{$ns}-interactive {
       &:hover {
         background-color: rgba($gray3, 0.3);
@@ -620,17 +630,17 @@ table.#{$ns}-html-table {
         background-color: rgba($gray3, 0.35);
         color: $black;
       }
+    }
 
-      .#{$ns}-tag-remove > .#{$ns}-icon:first-child {
-        color: $gray1;
+    .#{$ns}-tag-remove > .#{$ns}-icon:first-child {
+      color: $gray1;
 
-        &:hover {
-          color: $dark-gray1;
-        }
+      &:hover {
+        color: $dark-gray5;
+      }
 
-        &:active {
-          color: $dark-gray1;
-        }
+      &:active {
+        color: $dark-gray5;
       }
     }