payment_icons icon indicating copy to clipboard operation
payment_icons copied to clipboard

feat(icons): Add logo for Escrow.com Payments

Open AdiRishi opened this issue 10 months ago • 7 comments

Why are you adding this icons?

Escrow.com is preparing it's payments app integration with Shopify. As part of this work Escrow needs it's own branding available at checkout to provide a better experience.

Help us identify yourself

  • [X] I'm working/collaborating with the brand directly and they have provided the icons.

Link to the brand guidelines: Not available

Checklist to add new icons

  • [X] All icons have a corresponding entry in db/payment_icons.yml
  • [X] I have followed the icon guidelines detailed in the CONTRIBUTING.md file
  • [X] I have optimized the icon with SVGO
  • [X] I am confident that all icons are clear and easy to read/understand
  • [X] I have provided a link to the brand icon’s brand guidelines whenever possible.
  • [X] I have attached a screenshot comparison with the example icon provided in guidelines
  • [X] I recognize that if my icon is not approved by the Shopify Partners team it may not receive review nor merger.

Attach a screenshot of the icon along side the example Visa icon

Screenshot 2024-04-04 at 3 30 38 pm

AdiRishi avatar Apr 04 '24 04:04 AdiRishi

Hi @AdiRishi, could you please update the name to follow our contributing guidelines on naming?

hellicarusprime avatar Apr 08 '24 10:04 hellicarusprime

Hi @AdiRishi, could you please update the name to follow our contributing guidelines on naming?

@hellicarusprime my apologies, I missed that requirement. Updated now 👍

AdiRishi avatar Apr 08 '24 11:04 AdiRishi

Hi @AdiRishi, could you please update the name to follow our contributing guidelines on naming?

@hellicarusprime my apologies, I missed that requirement. Updated now 👍

No worries, we used to allow it.

hellicarusprime avatar Apr 08 '24 13:04 hellicarusprime

@AdiRishi One other small thing is that the border should be applied to the outside rather than the center.

You currently have a 37 x 23 px rectangle with the border applied to the center, but we'd prefer if the rectangle was 36 x 22 px with the border applied to the outside. I hope that makes sense.

hellicarusprime avatar Apr 08 '24 13:04 hellicarusprime

Escrow com icon

Hi @hellicarusprime , I'm Jeremy, Designer at Escrow. I made the rectangle 36 x 22 with the border on the outside. And also used the optimization tool. However, because the border is applid on the outside, the overal svg size becomes 38 x 24, please let me know if this is what you prefer.

Jeremyhzk avatar May 01 '24 01:05 Jeremyhzk

Hi @Lovedanihonjin Im not sure what you mean by expanding/outlining the stroke, did you mean making the outline thicker? or were you refering to the the shield logo of Escrow? Escrow com

Jeremyhzk avatar May 02 '24 07:05 Jeremyhzk

Hi @Lovedanihonjin Im not sure what you mean by expanding/outlining the stroke, did you mean making the outline thicker? or were you refering to the the shield logo of Escrow? Escrow com

Here is an example of outlining your stroke. You just need to make your stroke a flat vector. Uploading Screenshot 2024-05-09 at 9.38.21 AM.png…

Lovedanihonjin avatar May 09 '24 16:05 Lovedanihonjin

@Lovedanihonjin Hi, your screenshot is broken but I think I know what you meant. I made the stroke into a vector by outlining it with the pen tool, also put it through the optimization tool. I uploaded the optimized and non optimized version. Let me know if this works Escrow logo revised optimized Escrow logo revised

Jeremyhzk avatar May 14 '24 07:05 Jeremyhzk

Hi @Jeremyhzk , could you please rebase your branch against activemerchant:master to address the conflict. Thanks

adeniyiao avatar Jun 04 '24 10:06 adeniyiao

I've rebased the branch, used the updated optimized icon @Jeremyhzk provided, and added in the necessary additions like viewbox, id and title as per the contributing guidelines.

AdiRishi avatar Jun 05 '24 07:06 AdiRishi