Code review comment for lp:~edwin-grubbs/launchpad/bug-526608-milestone-ajax-plus-icon

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

> The content of the anchors should abut the tags to prevent odd underlining: <a
> ..><img ../></a>

I ended up changing the addicon into a sprite, since Firefox would apply underlining to the icon since it was in the link with "Target to milestone". I didn't change the editicon into a sprite, since the browsers would hide the <a> unless there was a &nbsp; in it, but then that would be underlined weirdly.

« Back to merge proposal