Closed ninjaofawesome closed 6 years ago
lgtm! one suggestion I'd have is for any additions of new css styles, it could be helpful to drop a screenshot of what it looks like in the PR overview. For ex, I'm not sure what 'colorway' means aside from the web definition of "an arrangement of colors", but the picture itself could clarify in that instance.
@amaravora Thanks! Duly noted- I should but I got lazy as most of us on the FE team knows vaguely what these look like. Shouldn't rest on my laurels. :)
OVERVIEW
This PR adds a dark black colorway to the existing buttons, which is necessary for CMS updates.
SOLUTION
btn-vimeo-alt
to the list of buttons