search

w3c / payment-handler

Payment Handler API
https://w3c.github.io/payment-handler/
Other
72 stars 42 forks source link

PaymentInstruments .set() and icons #233

Open marcoscaceres opened 6 years ago

marcoscaceres commented 6 years ago

The .set() algorithm has a whole bunch prose about icon processing. However, I don't think that is the right place to do any of the processing specified there.

We should have a dedicated section for displaying icons, but storage of a payment instrument should just store the instrument, and not do any icon processing.

rsolomakhin commented 6 years ago

When should icon processing happen?

marcoscaceres commented 6 years ago

At display time, I think. We should do some light validation checks here. Working on a proposal.

rsolomakhin commented 6 years ago

We would probably want to have some kind of caching to reduce paymentRequest.show() latency due to loading the icons.

marcoscaceres commented 6 years ago

Potentially, yeah... that's a tricky one. We need to check what the Notifications API does.

rsolomakhin commented 6 years ago

Sounds good. I'm all for re-using existing paradigms, if Notification API has it all figured out already.

agektmr commented 6 years ago

Let me cross reference to a related issue here. cc: @kenchris