--- 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;
--- 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;