refactor

Author: odeimaiz

services/static-webserver/client/source/class/osparc/jobs/RunsTable.js

@@ -166,8 +166,9 @@ qx.Class.define("osparc.jobs.RunsTable", {
       this.addListener("cellTap", e => {
         const rowIdx = e.getRow();
         const target = e.getOriginalTarget();
-        if (target.closest(".qx-material-button") && (target.tagName === "IMG" || target.tagName === "DIV")) {
-          const action = target.closest(".qx-material-button").getAttribute("data-action");
+        const closestItems = osparc.ui.table.cellrenderer.ButtonRenderer.getClosestItems(target);
+        if (closestItems && (target.tagName === "IMG" || target.tagName === "DIV")) {
+          const action = closestItems.getAttribute("data-action");
           if (action) {
             this.__handleButtonClick(action, rowIdx);
           }

services/static-webserver/client/source/class/osparc/jobs/SubRunsTable.js

@@ -136,8 +136,9 @@ qx.Class.define("osparc.jobs.SubRunsTable", {
       this.addListener("cellTap", e => {
         const row = e.getRow();
         const target = e.getOriginalTarget();
-        if (target.closest(".qx-material-button") && (target.tagName === "IMG" || target.tagName === "DIV")) {
-          const action = target.closest(".qx-material-button").getAttribute("data-action");
+        const closestItems = osparc.ui.table.cellrenderer.ButtonRenderer.getClosestItems(target);
+        if (closestItems && (target.tagName === "IMG" || target.tagName === "DIV")) {
+          const action = closestItems.getAttribute("data-action");
           if (action) {
             this.__handleButtonClick(action, row);
           }

services/static-webserver/client/source/class/osparc/ui/table/cellrenderer/ButtonRenderer.js

@@ -38,6 +38,15 @@ qx.Class.define("osparc.ui.table.cellrenderer.ButtonRenderer", {
     }
   },
 
+  statics: {
+    BUTTON_CLASS: "qx-material-button",
+    // BUTTON_CLASS: "form-button",
+
+    getClosestItems: function(target, className = "qx-material-button") {
+      return target.closest("."+className)
+    }
+  },
+
   members: {
     // Override
     _getContentHtml: function(cellInfo) {
@@ -46,7 +55,7 @@ qx.Class.define("osparc.ui.table.cellrenderer.ButtonRenderer", {
 
       // Return the button with the image
       return `
-        <div class="qx-material-button"
+        <div class="${this.self().BUTTON_CLASS}"
           data-action="${clickAction}" data-row="${cellInfo.row}"
           style="cursor:pointer; padding:2px 2px; width:26px; height:18px; display:flex; align-items:center; justify-content:center;"
         >