# HG changeset patch
# User Steve Losh <steve@stevelosh.com>
# Date 1276411553 14400
# Node ID ee8d78fbf746d7df0dd79b0d7d5b0ec7fbfdf031
# Parent  92e5e6c7159552630b1973be001fd2e7d7cf3809
web: more styles

diff -r 92e5e6c71595 -r ee8d78fbf746 review/static/style.css
--- a/review/static/style.css	Sun Jun 13 02:38:28 2010 -0400
+++ b/review/static/style.css	Sun Jun 13 02:45:53 2010 -0400
@@ -152,10 +152,6 @@
 body .disabled {
   display: none;
 }
-body .avatar img {
-  height: 52px;
-  width: 52px;
-}
 form .field {
   position: relative;
 }
@@ -407,7 +403,7 @@
   margin-bottom: 14px;
 }
 #changeset .content .item-listing .comment, #changeset .content .item-listing .signoff {
-  padding: 10px 10px 12px;
+  padding: 8px 10px;
   border-top: 1px solid #fff;
   border-bottom: 1px solid #ddd;
 }
@@ -420,34 +416,38 @@
 #changeset .content .item-listing .comment img, #changeset .content .item-listing .signoff img {
   float: right;
 }
-#changeset .content .item-listing .comment .signoff-opinion, #changeset .content .item-listing .signoff .signoff-opinion {
+#changeset .content .item-listing .comment .message, #changeset .content .item-listing .signoff .message {
+  font-family: Monaco, Consolas, "Courier New", monospace;
+  font-size: 12px;
+  width: 660px;
+  padding-top: 3px;
+}
+#changeset .content .item-listing .comment .avatar img, #changeset .content .item-listing .signoff .avatar img {
+  height: 30px;
+  width: 30px;
+  margin-top: 5px;
+}
+#changeset .content .item-listing .signoff .signoff-opinion {
   float: right;
-  font: bold 52px/1 "Helvetica Neue", HelveticaNeue, Arial, Helvetica, sans-serif;
+  font: bold 30px/1 "Helvetica Neue", HelveticaNeue, Arial, Helvetica, sans-serif;
   text-transform: capitalize;
   margin-right: 15px;
+  margin-top: 5px;
 }
-#changeset .content .item-listing .comment .signoff-opinion.yes, #changeset .content .item-listing .signoff .signoff-opinion.yes {
+#changeset .content .item-listing .signoff .signoff-opinion.yes {
   color: #568019;
 }
-#changeset .content .item-listing .comment .signoff-opinion.no, #changeset .content .item-listing .signoff .signoff-opinion.no {
+#changeset .content .item-listing .signoff .signoff-opinion.no {
   color: #c00;
 }
-#changeset .content .item-listing .comment .signoff-opinion.neutral, #changeset .content .item-listing .signoff .signoff-opinion.neutral {
+#changeset .content .item-listing .signoff .signoff-opinion.neutral {
   color: #bab994;
 }
-#changeset .content .item-listing .comment .author .opinion, #changeset .content .item-listing .signoff .author .opinion {
+#changeset .content .item-listing .signoff .author .opinion {
   font-weight: bold;
 }
-#changeset .content .item-listing .comment .message, #changeset .content .item-listing .signoff .message {
-  background-color: #fefefd;
-  border: 1px solid #eee;
-  font-family: Monaco, Consolas, "Courier New", monospace;
-  font-size: 12px;
-  padding: 10px;
-  width: 650px;
-}
 #changeset .content .item-listing .signoff .message {
-  width: 500px;
+  width: 620px;
 }
 #changeset .content .add-review-comment {
   margin-bottom: 40px;
@@ -517,6 +517,7 @@
 }
 #changeset .content .diff table td.comment .avatar {
   float: right;
+  margin-top: 2px;
 }
 #changeset .content .diff table td.comment .avatar img {
   height: 30px;
diff -r 92e5e6c71595 -r ee8d78fbf746 review/static/style.less
--- a/review/static/style.less	Sun Jun 13 02:38:28 2010 -0400
+++ b/review/static/style.less	Sun Jun 13 02:45:53 2010 -0400
@@ -182,10 +182,6 @@
     .disabled {
         display: none;
     }
-    .avatar img { 
-        height: 52px;
-        width: 52px;
-    }
 }
 
 form {
@@ -331,7 +327,7 @@
         margin-bottom: 14px;
 
         .comment, .signoff {
-            padding: 10px 10px 12px;
+            padding: 8px 10px;
             border-top: 1px solid #fff;
             border-bottom: 1px solid #ddd;
 
@@ -344,11 +340,25 @@
             img {
                 float: right;
             }
+            .message {
+                font-family: @font-mono;
+                font-size: 12px;
+                width: 660px;
+                padding-top: 3px;
+            }
+            .avatar img {
+                height: 30px;
+                width: 30px;
+                margin-top: 5px;
+            }
+        }
+        .signoff {
             .signoff-opinion {
                 float: right;
-                font: bold 52px/1 @font-normal;
+                font: bold 30px/1 @font-normal;
                 text-transform: capitalize;
                 margin-right: 15px;
+                margin-top: 5px;
 
                 &.yes { color: saturate(@c-green - #111, 20%); }
                 &.no { color: #c00; }
@@ -360,17 +370,9 @@
                 }
             }
             .message {
-                background-color: lighten(@c-soft-cream, 4%);
-                border: 1px solid #eee;
-                font-family: @font-mono;
-                font-size: 12px;
-                padding: 10px;
-                width: 650px;
+                width: 620px;
             }
         }
-        .signoff .message {
-            width: 500px;
-        }
     }
     .add-review-comment {
         margin-bottom: 40px;
@@ -455,6 +457,7 @@
 
                     .avatar {
                         float: right;
+                        margin-top: 2px;
 
                         img {
                             height: 30px;