Code review comment for lp:~edwin-grubbs/launchpad/bug-436507-distro-sourcepackage-link

Revision history for this message
Edwin Grubbs (edwin-grubbs) wrote :

> Hi Edwin.
>
> The link text is a nice improvement. Without an icon, it looks surprising. I
> expect the link to have class="sprite info". You can land these changes with
> the addition of the icon.

Hi Curtis,

This stylesheet overrides the background, which prevents the sprite from showing up. I also don't see any other see-all links with icons, so I assumed that it was intended to just be text.

.see-all a {
    padding-left: 8px;
    background: inherit;
    font-size: 72%;
    text-decoration: underline;
    }

-Edwin

« Back to merge proposal