Re: [w3c/payment-handler] Add a constructor of PaymentRequestEvent and PaymentRequestEventInit. (#162)

marcoscaceres requested changes on this pull request.



> @@ -1314,6 +1314,16 @@
           Promise<WindowClient> openWindow(USVString url);
           void respondWith(Promise<PaymentAppResponse>appResponse);
         };
+
+        dictionary PaymentRequestEventInit : ExtendableEventInit {
+          DOMString topLevelOrigin;
+          DOMString paymentRequestOrigin;
+          DOMString paymentRequestId;
+          sequence<PaymentMethodData> methodData;
+          PaymentItem total;
+          sequence<PaymentDetailsModifier> modifiers;
+          DOMString instrumentKey;
+        };

Try: 

```
<p>
  The <dfn>topLevelOrigin</dfn>, <dfn>paymentRequestOrigin</dfn>, <dfn>paymentRequestId</dfn>, <dfn> methodData</dfn>, <dfn>total</dfn>, <dfn> modifiers</dfn>, and <dfn>instrumentKey</dfn> members share their definitions with those defined for <a>PaymentRequestEvent</a>
</p>
```

Also, don't include `PaymentRequestEventInit` in the same `<pre>` block. Move it to its own `pre` and `section` - otherwise, it won't appear in the TOC. 
  

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/w3c/payment-handler/pull/162#pullrequestreview-40684641