Allow arguments to jsModal

demos/modal.php

@@ -51,7 +51,11 @@ public function init()
     $frame->add(['Header', 'Clicked row with ID = '.$_GET['id']]);
 });
 
-$t->on('click', 'tbody>tr', new \atk4\ui\jsModal(
+$t->onRowClick(new \atk4\ui\jsModal('Row Clicked', $frame, ['id'=>$t->jsRow()->data('id')]));
+
+// Old COMPAT code
+/*
+$t->on('click', 'tr', new \atk4\ui\jsModal(
     'Row Clicked',
     new \atk4\ui\jsExpression(
         '[]+"&id="+[]', [
@@ -61,3 +65,4 @@ public function init()
     )
 ));
 $t->addStyle('cursor', 'pointer');
+ */

src/Table.php

@@ -319,6 +319,35 @@ public function renderView()
         return View::renderView();
     }
 
+    /**
+     * Same as on('click', 'tr', $action), but will also make sure you can't
+     * click outside of the body. Additionally when you move cursor over the
+     * rows, pointer will be used and rows will be highlighted as you hover.
+     *
+     * @param jsChain|callable $action Code to execute
+     *
+     * @return jQuery
+     */
+    public function onRowClick($action)
+    {
+        $this->addClass('selectable');
+        $this->js(true)->find('tbody')->css('cursor', 'pointer');
+
+        return $this->on('click', 'tbody>tr', $action);
+    }
+
+    /**
+     * Use this to quickly access the <tr> and wrap in jQuery.
+     *
+     * $this->jsRow()->data('id');
+     *
+     * @return jQuery
+     */
+    public function jsRow()
+    {
+        return new jQuery(new jsExpression('this'));
+    }
+
     /**
      * Executed for each row if "totals" are enabled to add up values.
      */

src/jsModal.php

@@ -7,7 +7,7 @@
  */
 class jsModal extends jsExpression
 {
-    public function __construct($title, $url)
+    public function __construct($title, $url, $args = [])
     {
         if ($url instanceof VirtualPage) {
             $url = $url->getURL('cut');
@@ -29,7 +29,7 @@ public function __construct($title, $url)
 
         parent::__construct('
         var m=$("<div>").appendTo("body").addClass("ui fullscreen scrolling modal").html([content]);
-        m.modal({onHide: function() { m.children().remove(); return true; }}).modal("show").find(".content").load([url], function() { m.modal("refresh"); })',
-            ['content'=>$content, 'url'=>$url]);
+        m.modal({onHide: function() { m.children().remove(); return true; }}).modal("show").find(".content").load([url], [arg], function() { m.modal("refresh"); })',
+            ['content'=>$content, 'url'=>$url, 'arg'=>$args]);
     }
 }