Code review comment for lp://qastaging/~evfool/scratch/lp1309291

Revision history for this message
Danielle Foré (danrabbit) wrote :

Thanks for your branch Robert!

I think maybe we should just use a text button here. I don't think any themes ship with a "replace all" icon. While we're at it, maybe we should use a text button for "replace" as well. I think we're probably not going to get the best level of clarity out of an icon for this action.

review: Needs Fixing

« Back to merge proposal