From f0bbb29621b9f770ef8f0f9cf4c60d351595318e Mon Sep 17 00:00:00 2001
From: Frances Hocutt <fhocutt@wikimedia.org>
Date: Thu, 17 Dec 2015 16:18:11 -0800
Subject: [PATCH] [WIP] Improve style and turnitin report display

---
 static/style.css     | 11 +++++++++++
 templates/index.mako | 14 +++++++-------
 2 files changed, 18 insertions(+), 7 deletions(-)

diff --git a/static/style.css b/static/style.css
index ecc7dad..cb0fa11 100644
--- a/static/style.css
+++ b/static/style.css
@@ -63,6 +63,17 @@ div#info-box {
     margin: 10px 5px;
 }
 
+div#turnitin-container {
+    padding: 5px 10px;
+    margin: 15px 10px 10px 5px;
+}
+
+div#turnitin-title {
+    margin-bottom: -5px;
+    text-align: center;
+    font-weight: bold;
+}
+
 div#cv-result {
     padding: 5px;
     margin: 10px 5px;
diff --git a/templates/index.mako b/templates/index.mako
index 2795fea..dafc474 100644
--- a/templates/index.mako
+++ b/templates/index.mako
@@ -150,6 +150,7 @@
         </tr>
     </table>
 </form>
+
 % if result:
     <div id="generation-time">
         Results
@@ -166,14 +167,14 @@
     </div>
 
     % if query.turnitin:
-        <div id="turnitin-result" class="${'red' if query.turnitin_result else 'green'}-box">
-            <p>Turnitin results (this should be centered like "checked sources")</p>
-            % if query.turnitin_result:
-                Turnitin (through <a href="https://en.wikipedia.org/wiki/User:EranBot">EranBot</a>) found revisions that may have been plagiarized. Please review them. (Does this need some sort of p tag or something?)
+        <div id="turnitin-container" class="${'red' if query.turnitin_result.reports else 'green'}-box">
+            <div id="turnitin-title">Turnitin Results</div>
+            % if query.turnitin_result.reports:
+                <p>Turnitin (through <a href="https://en.wikipedia.org/wiki/User:EranBot">EranBot</a>) found revisions that may have been plagiarized. Please review them.</p>
 
                 %for report in turnitin_result.reports:
                 <ul>
-                    <li><a href="https://tools.wmflabs.org/eranbot/ithenticate.py?rid=${report.reportid}">Turnitin report</a>
+                    <li><a href="https://tools.wmflabs.org/eranbot/ithenticate.py?rid=${report.reportid}">Turnitin report ${report.reportid}</a>
                     <ul>
                     % for source in report.sources:
                           <li> ${source['percent']}% of revision text (${source['words']} words) found at <a href="${source['url']}">${source['url']}</a></li>
@@ -181,10 +182,9 @@
                     </ul></li>
                 %endfor
                 </ul>
-                ${turnitin_result}
 
             % else:
-                Turnitin (through <a href="https://en.wikipedia.org/wiki/User:EranBot">EranBot</a>) found no matching sources.
+                <p>Turnitin (through <a href="https://en.wikipedia.org/wiki/User:EranBot">EranBot</a>) found no matching sources.</p>
             % endif
         </div>
     % endif