fix(select): unset cursor of header items (#571)

* fix(select): unset cursor of header items * fix(select) clarify comment
Refinitiv · Jan 20, 2023 · eff8a75 · eff8a75
1 parent d1c9bc2
commit eff8a75
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 3 deletions.
diff --git a/packages/elemental-theme/src/custom-elements/ef-select.less b/packages/elemental-theme/src/custom-elements/ef-select.less
@@ -31,8 +31,6 @@
   }
   // #endregion
 
-  cursor: pointer;
-
   &[readonly] {
     cursor: default;
   }

diff --git a/packages/elements/src/select/index.ts b/packages/elements/src/select/index.ts
@@ -129,6 +129,7 @@ export class Select extends ControlElement implements MultiValue {
         right: 0;
         bottom: 0;
         left: 0;
+        cursor: pointer;
       }
       #select {
         position: absolute;

diff --git a/packages/halo-theme/src/custom-elements/ef-select.less b/packages/halo-theme/src/custom-elements/ef-select.less
@@ -41,7 +41,6 @@
     }
   }
 
-  cursor: pointer;
 
   [part=list] {
     color: @control-text-color;

diff --git a/packages/solar-theme/src/custom-elements/ef-select.less b/packages/solar-theme/src/custom-elements/ef-select.less
@@ -9,6 +9,8 @@
 
   // #region - Extend from ef-button styles
   &:extend(:host);
+  // pointer cursor style would be applied internally to trigger element instead
+  cursor: unset;
   &:not([readonly]):hover {
     &:extend(:host:hover);
   }