Commit 0335953c authored by John Lee's avatar John Lee Committed by Commit Bot

WebUI: Remove --cr-actionable mixin, add pointer cursor manually

The --cr-actionable mixin wasn't too meaningful and only applied a
pointer cursor. We should use [actionable] in places where we can
and just manually add the pointer cursor to places where we can't.

Bug: 973674
Change-Id: Ie0e43f6e463276417701f1f172c9f91987b3a09c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1705016Reviewed-by: default avatarDemetrios Papadopoulos <dpapad@chromium.org>
Commit-Queue: John Lee <johntlee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#678374}
parent c82e1aa2
...@@ -44,7 +44,6 @@ ...@@ -44,7 +44,6 @@
} }
#advancedToggle { #advancedToggle {
@apply --cr-actionable;
--ink-color: currentColor; --ink-color: currentColor;
align-items: center; align-items: center;
background: transparent; background: transparent;
......
...@@ -77,12 +77,12 @@ ...@@ -77,12 +77,12 @@
border: none; border: none;
box-sizing: border-box; box-sizing: border-box;
color: var(--paper-grey-800); color: var(--paper-grey-800);
cursor: pointer;
font: inherit; font: inherit;
min-height: 32px; min-height: 32px;
padding: 0 24px; padding: 0 24px;
text-align: start; text-align: start;
width: 100%; width: 100%;
@apply --cr-actionable;
} }
.list-item:focus { .list-item:focus {
......
...@@ -8,11 +8,11 @@ ...@@ -8,11 +8,11 @@
<template> <template>
<style> <style>
button.action-link { button.action-link {
@apply --cr-actionable;
-webkit-appearance: none; -webkit-appearance: none;
background: none; background: none;
border: none; border: none;
color: var(--cr-link-color); color: var(--cr-link-color);
cursor: pointer;
display: inline-block; display: inline-block;
font-family: inherit; font-family: inherit;
text-decoration: none; text-decoration: none;
......
...@@ -6,7 +6,7 @@ ...@@ -6,7 +6,7 @@
<template> <template>
<style> <style>
[is='action-link'] { [is='action-link'] {
@apply --cr-actionable; cursor: pointer;
display: inline-block; display: inline-block;
text-decoration: none; text-decoration: none;
} }
......
...@@ -49,7 +49,7 @@ ...@@ -49,7 +49,7 @@
} }
:host ::slotted(.dropdown-item:not([disabled])) { :host ::slotted(.dropdown-item:not([disabled])) {
@apply --cr-actionable; cursor: pointer;
} }
:host ::slotted(.dropdown-item:focus) { :host ::slotted(.dropdown-item:focus) {
......
...@@ -22,7 +22,7 @@ ...@@ -22,7 +22,7 @@
} }
[actionable] { [actionable] {
@apply --cr-actionable; cursor: pointer;
} }
.subpage-arrow, .subpage-arrow,
...@@ -72,7 +72,7 @@ ...@@ -72,7 +72,7 @@
@apply --cr-selectable-focus; @apply --cr-selectable-focus;
} }
[selectable]:not(cr-radio-group) > * { [selectable]:not(cr-radio-group) > * {
@apply --cr-actionable; cursor: pointer;
} }
.cr-container-shadow { .cr-container-shadow {
......
...@@ -112,10 +112,6 @@ ...@@ -112,10 +112,6 @@
* set for both light and dark modes and use a single variable below. */ * set for both light and dark modes and use a single variable below. */
html { html {
--cr-actionable: {
cursor: pointer;
}
--cr-button-edge-spacing: 12px; --cr-button-edge-spacing: 12px;
--cr-button-height: 32px; --cr-button-height: 32px;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment