jenkins-bot has submitted this change and it was merged.

Change subject: Use stylesheet for main stats view in rt server
......................................................................


Use stylesheet for main stats view in rt server

Change-Id: Ib2f6cba4e08fc140e745a0e8a4890dfd20a8b584
---
M tests/server/static/style.css
M tests/server/views/index.html
2 files changed, 45 insertions(+), 16 deletions(-)

Approvals:
  Marcoil: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/tests/server/static/style.css b/tests/server/static/style.css
index 9441dc6..0902179 100644
--- a/tests/server/static/style.css
+++ b/tests/server/static/style.css
@@ -1,3 +1,35 @@
+/* Main stats page */
+
+.overview tr {
+       font-weight: bold
+}
+
+.overview td {
+       padding-left: 20px
+}
+
+.overview td + td {
+       text-align: right
+}
+
+#graph tr {
+       height: 60px;
+}
+
+#graph td[title='Perfect / no diffs'] {
+       background: green
+}
+
+#graph td[title='Syntactic diffs'] {
+       background: yellow
+}
+
+#graph td[title='Semantic diffs'] {
+       background: red
+}
+
+/* results table views */
+
 .results th {
        padding: 0 10px;
 }
diff --git a/tests/server/views/index.html b/tests/server/views/index.html
index b5b0681..6acee21 100644
--- a/tests/server/views/index.html
+++ b/tests/server/views/index.html
@@ -8,32 +8,29 @@
 </ul>
 <p></p>
 
-<table>
+<table id='graph'>
        <tbody>
-               <tr height="60px">
-                       <td width="{{graphWidths.perfect}}" 
style="background:green"
-                               title="Perfect / no diffs"></td>
-                       <td width="{{graphWidths.syntacticDiff}}" 
style="background:yellow"
-                               title="Syntactic diffs"></td>
-                       <td width="{{graphWidths.semanticDiff}}" 
style="background:red"
-                               title="Semantic diffs"></td>
+               <tr>
+                       <td width="{{graphWidths.perfect}}" title="Perfect / no 
diffs"></td>
+                       <td width="{{graphWidths.syntacticDiff}}" 
title="Syntactic diffs"></td>
+                       <td width="{{graphWidths.semanticDiff}}" 
title="Semantic diffs"></td>
                </tr>
        </tbody>
 </table>
 
 <p>
        Latest revision:
-       <table>
+       <table class='overview'>
                <tbody>
                {{#each latestRevision}}
-               <tr style="font-weight:bold">
-                       <td style="padding-left:20px">
+               <tr>
+                       <td>
                                {{description}}
                                {{#if ../prefix}}
                                        ({{../../prefix}})
                                {{/if}}
                        </td>
-                       <td style="padding-left:20px; text-align:right">
+                       <td>
                        {{#if url}}
                                <a href='{{url}}'>
                        {{/if}}
@@ -50,17 +47,17 @@
 
 <p>
        Averages (over the latest results):
-       <table>
+       <table class='overview'>
                <tbody>
                        {{#each averages}}
-                       <tr style="font-weight:bold">
-                               <td style="padding-left:20px">
+                       <tr>
+                               <td>
                                        {{description}}
                                        {{#if ../prefix}}
                                                ({{../../prefix}})
                                        {{/if}}
                                </td>
-                               <td style="padding-left:20px; 
text-align:right">{{round value}}</td>
+                               <td>{{round value}}</td>
                        </tr>
                        {{/each}}
                </tbody>

-- 
To view, visit https://gerrit.wikimedia.org/r/109722
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib2f6cba4e08fc140e745a0e8a4890dfd20a8b584
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/services/parsoid
Gerrit-Branch: master
Gerrit-Owner: Bebirchall <bebirch...@gmail.com>
Gerrit-Reviewer: Marcoil <marc...@wikimedia.org>
Gerrit-Reviewer: Subramanya Sastry <ssas...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to