Closed djhmateer closed 6 years ago
@djhmateer Thanks for this PR. A subtle change if you wouldn't mind. Thanks!
@djhmateer Just a small issue if you wouldn't mind addressing so we can merge? (See Steve's comment)
We'll leave this one as-is and pull out the icon if recommended after the rest of the UX review.
@djhmateer - One thing I didn't press during the codeathon as I didn't want to push too much to people too fast was linking the PR to the issue. If you put something like Fixes #2291 in the PR comment it will link it to the original issue. This is great for reviewers as it automatically adds a hyperlink and also, once the PR is merged, the issue is auto-closed.