adjust reviews page for UI debt (bug 1189761)

mozilla · Jan 29, 2016 · 4523258 · 4523258
1 parent 5fc3f36
commit 4523258
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 11 deletions.
diff --git a/src/media/css/detail/reviews.styl b/src/media/css/detail/reviews.styl
@@ -21,7 +21,7 @@
 
 .reviews-summary-large {
     text-align: center;
-    padding: 30px 0;
+    padding: 0 0 20px;
 
     .stars-wrapper {
         transform: scale(2.5);
@@ -68,6 +68,14 @@
         // Offset the margin-bottom from reviews.
         margin-bottom: -60px;
     }
+    [data-page-type~="reviews"] {
+        .reviews-summary-large {
+            padding: 30px 0;
+        }
+        .detail-flex-wrap {
+            flex-direction: row-reverse;
+        }
+    }
 }
 
 .reviews--guideline.reviews--guideline {

diff --git a/src/media/css/reviews.styl b/src/media/css/reviews.styl
@@ -262,6 +262,9 @@ mkt-prompt[data-modal="flag-review"] {
     .review-buttons li {
         margin-top: 0;
     }
+    .review-buttons {
+        margin-bottom: 30px;
+    }
 }
 
 .review .review-has-flagged {

diff --git a/src/templates/ratings/main.html b/src/templates/ratings/main.html
@@ -12,12 +12,12 @@
     {% defer (url=apiParams('reviews', {'app': slug}), pluck='objects',
               id='ratings', paginate='.reviews-wrapper') %}
       <div class="detail-flex-wrap">
-        <div class="detail-left-side">
-          {% if not settings.meowEnabled %}
-            <h3>{{ _('Reviews') }}</h3>
-          {% endif %}
+        <div class="detail-right-side">
           {% include "_includes/reviews_summary.html" %}
-
+          {% set is_review_listing = true %}
+          {% include "_includes/reviews_sidebar.html" %}
+        </div>
+        <div class="detail-left-side">
           <ul class="reviews-wrapper">
             {% for _review in this %}
               {{ review(_review, slug=slug,
@@ -29,11 +29,6 @@ <h3>{{ _('Reviews') }}</h3>
             {% endif %}
           </ul>
         </div>
-
-        <div class="detail-right-side">
-          {% set is_review_listing = true %}
-          {% include "_includes/reviews_sidebar.html" %}
-        </div>
       </div>
     {% placeholder %}
       <p class="spinner alt padded"></p>