launchpad-reviewers team mailing list archive

[Guruprasad <mp+447200@xxxxxxxxxxxxxxxxxx>]

Diff comments:

> diff --git a/lib/canonical/launchpad/icing/style.css b/lib/canonical/launchpad/icing/style.css
> index 125f723..04dd339 100644
> --- a/lib/canonical/launchpad/icing/style.css
> +++ b/lib/canonical/launchpad/icing/style.css
> @@ -663,12 +692,18 @@ table.diff .line-no.active, table.diff .ss-line-no.active {
>    background: url(/@@/add) #f6f6f6 center left no-repeat;
>  }
>  table.diff .diff-chunk, table.diff .diff-file, table.diff .diff-header {
> - font-weight: bold;
> +  background: #f4f6f9;
> +  font-weight: 600;
>  }
> -table.diff .diff-added { background-color: #92ED92; }

+1 on this feedback.

> -table.diff .diff-removed { background-color: #FF7F7F; }
> +table.diff .diff-added { background-color: #e1ffe7; }
> +table.diff .diff-removed { background-color: #ffe6e4; }
>  table.diff .inline-comments > td > div {
> -  margin: 0 1em 1.5em 1em;
> +  margin: 0;
> +}
> +
> +table.diff .inline-comment {
> +    border-top: 1px solid #ddd;
> +    border-bottom: 1px solid #ddd;
>  }
>  table.diff .inline-comments .yui3-ieditor { padding-right:0!important; }
>  table.diff .inline-comments .yui3-ieditor-multiline .yui3-ieditor-btns


-- 
https://code.launchpad.net/~petermakowski/launchpad/+git/launchpad/+merge/447200
Your team Launchpad code reviewers is requested to review the proposed merge of ~petermakowski/launchpad:update-diff-view-ui into launchpad:master.