launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #28572
Re: [Merge] ~lgp171188/launchpad:diff-viewer-fix-text-overflow into launchpad:master
Diff comments:
> diff --git a/lib/canonical/launchpad/icing/style.css b/lib/canonical/launchpad/icing/style.css
> index 92f2d6f..1be36de 100644
> --- a/lib/canonical/launchpad/icing/style.css
> +++ b/lib/canonical/launchpad/icing/style.css
> @@ -636,7 +637,7 @@ table.diff {
> margin-bottom: 0.5em;
> }
> table.unidiff {
> - table-layout: fixed;
> + table-layout: auto;
I am a little nervous about this change as I am not sure if this will have any unintended side-effects in some corner cases.
> }
>
> table.diff .nav-cursor:before {
--
https://code.launchpad.net/~lgp171188/launchpad/+git/launchpad/+merge/424302
Your team Launchpad code reviewers is requested to review the proposed merge of ~lgp171188/launchpad:diff-viewer-fix-text-overflow into launchpad:master.
References