Closed mtruj013 closed 1 month ago
Thanks @britneywwc! Addressed almost everything and left couple of qs for the designers for the bits I wasn't sure about, could you take another look?
Just one more thing, there should be some padding around the card here
@britneywwc it has padding already which is provided by the p-card--overlay
class
It looks like the figma calls for an additional .5 rem though. I can update, but really it sounds like the card component should be officially upstreamed on Vanilla's side as every version we build of this card will need the additional custom style. Issue here
Done
QA
Issue / Card
Fixes https://warthogs.atlassian.net/browse/WD-12347