Remove generic payment method icons (Stripe CC + Cheque) #2968
Conversation
- icons should only be used for recognisable brands - generic icons (e.g. credit card) are not recommended
Size Change: +3.9 kB (0%) Total Size: 1.66 MB
ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This works fine for me, but we're completely removing the ability for other extensions to register an icon, I think it's best if we just remove icons from our own payment methods but leave the attribute there.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Chatted with @LevinMedia in slack (p1597154267257400-slack-C8X6Q7XQU) and here's a summary:
- confirm that generic icons get removed
- please keep the icon attribute as it will still be needed for things like paypal etc that will get applied in future pulls around payment method select structure.
This reverts commit 1990028.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Summary of slack discussion:
|
We no longer need a dev note for this. |
Fixes #2669
This fixes a couple of issues by removing the generic icons for some payment methods.
The
PaymentMethodLabel
component now no longer needs support for icons - so I've removed this. We might simplify this further - remove the component and allow payment methods to provide a string or custom component for their label. cc @nerrad - this is an extensibility API changeScreenshots
How to test the changes in this Pull Request:
Changelog
May need a dev note - this changes the API/docs for implementing a payment method - specifically the
PaymentMethodLabel
interface has changed.