Show recurring icon in payment methods #4299
Conversation
| .components-popover__content { | ||
| border-radius: 4px; | ||
| box-shadow: 0px 2px 6px 0px rgba( 0, 0, 0, 0.05 ); | ||
| padding: 5px 10px; | ||
| background-color: #000000; | ||
| color: #ffffff; | ||
| } |
There was a problem hiding this comment.
The styles here don't match what we're doing for the currency (i) popover. We should use the same styles or component.
There was a problem hiding this comment.
Waiting on confirmation from design about this.
There was a problem hiding this comment.
As per the confirmation from design, we are keeping these different as one if a tooltip and another is a popover. However, in the previous implementation, I was using the popover component for showing Supports recurring payments message. I have updated the component in facce12
|
Thanks @daledupreez, for the feedback. I have made some adjustments in my recent commits and it's ready for review again. 👀 |
| const Icon = styled.img` | ||
| height: 14px; | ||
| width: 14px; | ||
| cursor: pointer; |
There was a problem hiding this comment.
Do we need this? I am not sure that the item is clickable, so adding a pointer cursor doesn't feel right. Also, it doesn't seem to be visible for me! 🙃
Fixes STRIPE-192
Fixes #2831
Changes proposed in this Pull Request:
Testing instructions