tauthomas01

Results 12 comments of tauthomas01

Hi, the update is now published via [npm](https://www.npmjs.com/package/@shopify/theme-product-form) Let me know if there's anything else. Thank you

The icon looks good. Can you rebase your branch off master? Thank you

@onahkenneth We have changed our contributions guideline and we're no longer allowed to use `base64` image due to the file size. Could you provide the icon with SVG markups?

Hi @thejoshualewis , - The dimension is invalid. Please see the [contribution guideline](url) The "bitpay" currencies icon include 4 icons in 1 file, which does not follow the guideline. -...

Hi @thejoshualewis, If that's the case, all 4 icons should be in their own file, not together.

Hi @thejoshualewis I understand what you're trying to do, but in order to keep consistencies and the rules of this repository, we need to follow the guideline like any other...

Exactly, you'll create 4 individual icons. All instructions are included here: https://github.com/activemerchant/payment_icons/blob/master/CONTRIBUTING.md#guidelines-for-new-icons Here's the current list of icons, that will give you an idea how it looks like: https://github.com/activemerchant/payment_icons/tree/master/app/assets/images/payment_icons

Hi @LaNeraNura , it's okay to keep the `path` for usage, but what I meant is removing the `id` attributes [![alt](https://screenshot.click/09-09-8nhc4-zwvqj.png)](https://screenshot.click/09-09-8nhc4-zwvqj.png)

Hi @steve-paystack , There are attributes missing in the icon. Please follow the [contribution guidelines](https://github.com/activemerchant/payment_icons/blob/master/CONTRIBUTING.md#markup) for more information. Thank you.