fix: added ux fixes

upfluence · Jan 9, 2025 · da91af9 · da91af9
1 parent 47416a4
commit da91af9
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 4 deletions.
diff --git a/addon/components/hyper-table/cell-renderers/list.hbs b/addon/components/hyper-table/cell-renderers/list.hbs
@@ -10,8 +10,9 @@
         <div class="expandable-list">
           {{#each this.value as |item|}}
             <div
-              class="expandable-list__item fx-row fx-xalign-center fx-gap-px-6"
-              {{on "click" (action "goToUrl" (concat this.url item.id) bubbles=false)}}
+              class="expandable-list__item
+                {{if item.locked 'expandable-list__item-disabled '}}fx-row fx-xalign-center fx-gap-px-6"
+              {{on "click" (action "goToUrl" (concat this.url item.id) item.locked bubbles=false)}}
             >
               {{item.name}}
               {{#if item.locked}}

diff --git a/addon/components/hyper-table/cell-renderers/list.js b/addon/components/hyper-table/cell-renderers/list.js
@@ -17,8 +17,8 @@ export default Component.extend(CellRendererMixin, {
   }),
 
   actions: {
-    goToUrl(url, event) {
-      if (!this.url) {
+    goToUrl(url, locked, event) {
+      if (!this.url || locked) {
         event.preventDefault();
         event.stopPropagation();
         return;

diff --git a/app/styles/cells.less b/app/styles/cells.less
@@ -295,6 +295,12 @@
     padding: var(--spacing-px-12) var(--spacing-px-18);
     cursor: pointer;
 
+    &-disabled {
+      &:hover {
+        cursor: not-allowed;
+      }
+    }
+
     &:hover {
       background-color: var(--color-gray-100);
     }