Use a new class instead of inline styling to apply cursor: pointer

FredrikNoren · Sep 26, 2020 · 406a4b0 · 406a4b0
1 parent 0dcb1e2
commit 406a4b0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/components/textdiff/textdiff.js b/components/textdiff/textdiff.js
@@ -181,9 +181,7 @@ class TextDiffViewModel {
                 return this.getPatchCheckBox(capture, index, this.patchLineList()[index++]);
               }
             );
-            return `<td class="${tdClass}" ${
-              this.editState() === 'patched' ? 'style="cursor:pointer"' : ''
-            } data-bind="click: toggleCheckboxFromRowClick">${content}</td>`;
+            return `<td class="${tdClass}" data-bind="click: toggleCheckboxFromRowClick, css: { patched: editState() === 'patched' }">${content}</td>`;
           }
         );
       }

diff --git a/components/textdiff/textdiff.less b/components/textdiff/textdiff.less
@@ -3,4 +3,8 @@
     padding: 10px 0;
     text-align: center;
   }
+
+  td.patched {
+    cursor: pointer;
+  }
 }