From ed1d975f20e0478bd26f904ce91641882ede7ec9 Mon Sep 17 00:00:00 2001
From: Jorran de Wit <jorrandewit@outlook.com>
Date: Mon, 22 Jan 2018 22:24:47 +0100
Subject: [PATCH] Revert table
---
.../static/scipost/assets/css/_reports.scss | 22 +++++++++++
.../static/scipost/assets/css/_tables.scss | 14 -------
.../submissions/_single_report_ratings.html | 38 ++++---------------
3 files changed, 30 insertions(+), 44 deletions(-)
diff --git a/scipost/static/scipost/assets/css/_reports.scss b/scipost/static/scipost/assets/css/_reports.scss
index 0f45a330d..871bb5acc 100644
--- a/scipost/static/scipost/assets/css/_reports.scss
+++ b/scipost/static/scipost/assets/css/_reports.scss
@@ -18,3 +18,25 @@
margin-top: 1rem;
}
}
+
+
+.ratings {
+ background-color: #f4f4f4;
+ display: inline-block;
+ border: 1px solid #ddd;
+ text-transform: capitalize;
+
+ > ul {
+ list-style: none;
+ display: flex;
+ padding: 0;
+ margin: 0;
+
+ li {
+ border: 1px solid #ddd;
+ margin: 2px 2px;
+ padding: 0.25rem 0.5rem;
+ background: $body-bg;
+ }
+ }
+}
diff --git a/scipost/static/scipost/assets/css/_tables.scss b/scipost/static/scipost/assets/css/_tables.scss
index 978a10b85..8873b93c3 100644
--- a/scipost/static/scipost/assets/css/_tables.scss
+++ b/scipost/static/scipost/assets/css/_tables.scss
@@ -28,17 +28,3 @@ table.contributor-info {
vertical-align: top;
}
}
-
-.ratings {
- background-color: $body-bg;
- padding: 0.5rem;
- display: inline-block;
- border: 1px solid #ddd;
- text-transform: capitalize;
-
- th {
- min-width: 120px;
- text-align: right;
- padding: 0.1rem 1rem;
- }
-}
diff --git a/submissions/templates/submissions/_single_report_ratings.html b/submissions/templates/submissions/_single_report_ratings.html
index 5eeb8dcf8..56f04fc1b 100644
--- a/submissions/templates/submissions/_single_report_ratings.html
+++ b/submissions/templates/submissions/_single_report_ratings.html
@@ -1,32 +1,10 @@
<div class="ratings">
- <table>
- <tr>
- <th>Validity</th>
- <td>{{ report.get_validity_display }}</td>
- </tr>
- <tr>
- <th>Significance</th>
- <td>{{ report.get_significance_display }}</td>
- </tr>
- <tr>
- <th>Validity</th>
- <td>{{ report.get_grammar_display }}</td>
- </tr>
- <tr>
- <th>Originality</th>
- <td>{{ report.get_originality_display }}</td>
- </tr>
- <tr>
- <th>Clarity</th>
- <td>{{ report.get_clarity_display }}</td>
- </tr>
- <tr>
- <th>Formatting</th>
- <td>{{ report.get_formatting_display }}</td>
- </tr>
- <tr>
- <th>Grammar</th>
- <td>{{ report.get_grammar_display }}</td>
- </tr>
- </table>
+ <ul>
+ <li>validity: {{report.get_validity_display}}</li>
+ <li>significance: {{report.get_significance_display}}</li>
+ <li>originality: {{report.get_originality_display}}</li>
+ <li>clarity: {{report.get_clarity_display}}</li>
+ <li>formatting: {{report.get_formatting_display}}</li>
+ <li>grammar: {{report.get_grammar_display}}</li>
+ </ul>
</div>
--
GitLab