Re: [Launchpad-reviewers] [Merge] lp:~wallyworld/launchpad/sprite-icon-spacing-1007263 into lp:launchpad

2012-06-04 Thread Ian Booth
I have introduced a new css class 'actionicon' which provides the required left margin. Where nbsp; were removed before the anchors in the markup, I have added the new css class to the anchors. I have also done some drive bt fixes on the product index page - the programming languages edit icon

Re: [Launchpad-reviewers] [Merge] lp:~wallyworld/launchpad/sprite-icon-spacing-1007263 into lp:launchpad

2012-06-04 Thread Curtis Hovey
Review: Approve code I think the values are interpolated in the wrong order on line 39. Fix the line before merging your code. -- https://code.launchpad.net/~wallyworld/launchpad/sprite-icon-spacing-1007263/+merge/108291 Your team Launchpad code reviewers is subscribed to branch lp:launchpad.

[Launchpad-reviewers] [Merge] lp:~wallyworld/launchpad/sprite-icon-spacing-1007263 into lp:launchpad

2012-06-04 Thread Curtis Hovey
The proposal to merge lp:~wallyworld/launchpad/sprite-icon-spacing-1007263 into lp:launchpad has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~wallyworld/launchpad/sprite-icon-spacing-1007263/+merge/108291 --

Re: [Launchpad-reviewers] [Merge] lp:~wallyworld/launchpad/sprite-icon-spacing-1007263 into lp:launchpad

2012-06-01 Thread Curtis Hovey
Review: Needs Information code What sprite is not an action? I suppose it its sprites on non-anchors. The crux of the issue is text + anchor is too close. The text is often wrapped in a span so maybe we can use a sibling selector span + a.sprite {margin-left: 6px;} We might want to try this