Now that this is an `<a>`, I wonder if we should add the `href="#"` - probably doesn't matter but seems like we should be consistent with the other icons. Also, was there a particular reason why you switched the attribute order? (Doesn't really matter, just curious.)