Repository: spark
Updated Branches:
  refs/heads/master db45f5ad0 -> 5f27ae16d


[SPARK-4255] Fix incorrect table striping

This commit stripes table rows after hiding some rows, to
ensure that rows are correct striped to alternate white
and grey even when rows are hidden by default.

Author: Kay Ousterhout <[email protected]>

Closes #3117 from kayousterhout/striping and squashes the following commits:

be6e10a [Kay Ousterhout] [SPARK-4255] Fix incorrect table striping


Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/5f27ae16
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/5f27ae16
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/5f27ae16

Branch: refs/heads/master
Commit: 5f27ae16d5b016fae4afeb0f2ad779fd3130b390
Parents: db45f5a
Author: Kay Ousterhout <[email protected]>
Authored: Thu Nov 6 00:03:03 2014 -0800
Committer: Kay Ousterhout <[email protected]>
Committed: Thu Nov 6 00:03:03 2014 -0800

----------------------------------------------------------------------
 .../resources/org/apache/spark/ui/static/additional-metrics.js  | 2 ++
 core/src/main/resources/org/apache/spark/ui/static/table.js     | 5 -----
 2 files changed, 2 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/5f27ae16/core/src/main/resources/org/apache/spark/ui/static/additional-metrics.js
----------------------------------------------------------------------
diff --git 
a/core/src/main/resources/org/apache/spark/ui/static/additional-metrics.js 
b/core/src/main/resources/org/apache/spark/ui/static/additional-metrics.js
index c5936b5..badd85e 100644
--- a/core/src/main/resources/org/apache/spark/ui/static/additional-metrics.js
+++ b/core/src/main/resources/org/apache/spark/ui/static/additional-metrics.js
@@ -39,6 +39,8 @@ $(function() {
         var column = "table ." + $(this).attr("name");
         $(column).hide();
     });
+    // Stripe table rows after rows have been hidden to ensure correct 
striping.
+    stripeTables();
 
     $("input:checkbox").click(function() {
         var column = "table ." + $(this).attr("name");

http://git-wip-us.apache.org/repos/asf/spark/blob/5f27ae16/core/src/main/resources/org/apache/spark/ui/static/table.js
----------------------------------------------------------------------
diff --git a/core/src/main/resources/org/apache/spark/ui/static/table.js 
b/core/src/main/resources/org/apache/spark/ui/static/table.js
index 32187ba..6bb0301 100644
--- a/core/src/main/resources/org/apache/spark/ui/static/table.js
+++ b/core/src/main/resources/org/apache/spark/ui/static/table.js
@@ -28,8 +28,3 @@ function stripeTables() {
         });
     });
 }
-
-/* Stripe all tables after pages finish loading. */
-$(function() {
-    stripeTables();
-});


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to