Move to mixins in base-styles

WordPress · Feb 29, 2024 · 6ade827 · 6ade827
1 parent af0f332
commit 6ade827
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/packages/base-styles/_mixins.scss b/packages/base-styles/_mixins.scss
@@ -367,6 +367,15 @@
 	}
 }
 
+@mixin link-reset {
+	&:focus {
+		color: var(--wp-admin-theme-color--rgb);
+		/* stylelint-disable-next-line declaration-property-value-disallowed-list */
+		box-shadow: 0 0 0 var(--wp-admin-border-width-focus) var(--wp-admin-theme-color, #007cba);
+		border-radius: $radius-block-ui;
+	}
+}
+
 // The editor input reset with increased specificity to avoid theme styles bleeding in.
 @mixin editor-input-reset() {
 	font-family: $editor-html-font !important;

diff --git a/packages/dataviews/src/style.scss b/packages/dataviews/src/style.scss
@@ -219,7 +219,6 @@
 	color: $gray-900;
 	text-overflow: ellipsis;
 	white-space: nowrap;
-	overflow: hidden;
 	display: block;
 	width: 100%;
 
@@ -235,11 +234,7 @@
 		&:hover {
 			color: $gray-900;
 		}
-		&:focus {
-			color: var(--wp-admin-theme-color--rgb);
-			box-shadow: 0 0 0 var(--wp-admin-border-width-focus) var(--wp-components-color-accent, var(--wp-admin-theme-color, #007cba));
-			border-radius: $radius-block-ui;
-		}
+		@include link-reset();
 	}
 
 	button.components-button.is-link {