Code review comment for lp:~gmb/launchpad/subscribers-timeout-bug-471974

Hi Graham,

Thanks for this fix. I note there are two issues. One, the icon and subscriber name now appear on different lines and for the life of me I cannot figure out why.

Also I'm bothered that a change of this size has no tests. You could do a view test to check the results of your new view. What about Windmill tests?

Perhaps it is just the current state of affairs but the fragility of this change bothers me.

I've removed the 'js' portion from this review as you may want to get an expert to do that portion.