Permalink
Browse files

Improve the look between sections of the review requestbox.

The dividers are now flush up against the border of the box. Spacing in
each section is now consistent across the sections, instead of being a
bit off in some as before.

The "Required" asterisk is a bit smaller as well, so it doesn't affect
sizing and is a bit more subtle.
  • Loading branch information...
1 parent 2a35fb1 commit 341cd095c54ab704aefd8d50bb5411fd46d2db4f @chipx86 chipx86 committed Sep 1, 2012
@@ -25,6 +25,7 @@
@box-inner-border-radius: @box-border-radius - 1px;
@box-title-bg: @light-blue;
@box-shadow: 1px 1px 2px rgba(0, 0, 0, .15);
+@box-padding: 10px;
@draft-bg-color: #CDFF9C;
@draft-border-color: darken(@draft-bg-color, 50%);
@@ -49,8 +49,7 @@
}
.content {
- margin: 5px;
- padding: 0;
+ padding: @box-padding;
&.sliding {
margin-top: -5px;
@@ -62,7 +61,6 @@
label {
display: block;
- margin-top: 20px;
min-height: 15px;
img {
@@ -85,12 +83,13 @@
.main {
clear: both;
+ padding: 0;
}
#required-field-legend {
color: darkRed;
font-weight: bold;
- padding: 12px 0;
+ padding: @box-padding;
}
.controls {
@@ -119,7 +118,7 @@
}
.header {
- margin: 5px;
+ padding: @box-padding;
}
.star {
@@ -131,8 +130,7 @@
border-bottom: 1px #c2c1b0 solid;
border-spacing: 8px;
border-top: 1px #c2c1b0 solid;
- margin-top: 5px;
- padding: 5px 0;
+ padding: @box-padding;
width: 100%;
.caption {
@@ -155,12 +153,15 @@
}
#review-request-warning {
- background-color: #F5A9A9;
- border: 1px solid #FF0000;
display: none;
- font-weight: bold;
- padding: 5px;
- text-align: center;
+
+ td {
+ background-color: #F5A9A9;
+ border: 1px solid #FF0000;
+ font-weight: bold;
+ padding: 5px;
+ text-align: center;
+ }
}
}
@@ -311,7 +312,7 @@
.required-flag {
color: darkRed;
- font-size: 14px;
+ font-size: 10px;
font-weight: bold;
padding: 0 2px;
}
@@ -475,7 +475,7 @@ function setDraftField(field, value) {
.delay(6000)
.fadeOut(400, function() {
$(this).hide();
- });
+ });
/* Wrap each term in quotes or a leading 'and'. */
$.each(rsp.fields[field], function(key, value) {
@@ -504,7 +504,10 @@ function setDraftField(field, value) {
}
}
- $("#review-request-warning").html(message).show();
+ $("#review-request-warning")
+ .show()
+ .find("td")
+ .html(message);
}
var func = gEditorCompleteHandlers[field];
@@ -41,8 +41,8 @@
<col width="0%" />
<col width="50%" />
</colgroup>
- <tr>
- <td id="review-request-warning" colspan="4"></td>
+ <tr id="review-request-warning">
+ <td colspan="4"></td>
</tr>
<tr>
<td class="label"><label for="submitter">{% trans "Submitter" %}:</label></td>

0 comments on commit 341cd09

Please sign in to comment.