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

Revision history for this message
j.c.sackett (jcsackett) wrote :

Looks good to me, assuming the text show/hide tool is tested already and therefor not showing in the diff. My memory tells me it is.

I'm getting Curtis to follow up; if it needs it, I imagine he can provide a UI review as well.

review: Approve (code*)

« Back to merge proposal