Re: [w3c/payment-request] Attach 'payment request is showing boolean' to top-level browsing context (#811)

ianbjacobs commented on this pull request.



> @@ -609,12 +609,21 @@ <h2>
           member is set.
         </p>
       </div>
-      <p data-link-for="PaymentRequest">
-        Because the simultaneous display of multiple <a>PaymentRequest</a> user
-        interfaces might confuse the user, this specification limits the
-        <a>user agent</a> to displaying one at a time via the <a>show()</a>
-        method. This is ensured by a <dfn>payment request is showing</dfn>
-        boolean.
+      <p>
+        A <var>request</var>'s <dfn>payment-relevant browsing context</dfn> is
+        that <a>PaymentRequest</a>'s <a data-cite=
+        "HTML#concept-relevant-global">relevant global object</a>'s browsing
+        context's <a>top-level browsing context</a>. Every <a>payment-relevant
+        browsing context</a> has a <dfn>payment request is showing</dfn>

Hi @marcoscaceres,

The above sentence is quite dense. Is it correct? Should it be:

<pre>
       A <var>request</var>'s <dfn>payment-relevant browsing context</dfn> is
       that <a>PaymentRequest</a>'s <a data-cite=
        "HTML#concept-relevant-global">relevant global object</a>'s <a>top-level browsing context</a>.
</pre>

Or even better:

<pre>
    The <dfn>payment-relevant browsing context</dfn> of a <a>PaymentRequest</a>
    is the <a>top-level browsing context</a> of the <a data-cite=
        "HTML#concept-relevant-global">relevant global object</a>.
</pre>



-- 
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-request/pull/811#pullrequestreview-193347605