Re: [w3c/push-api] Expose pushManager on Window (PR #393)

@annevk commented on this pull request.



> @@ -170,9 +170,19 @@ <h2>
         </h2>
         <p>
           A <dfn>push subscription</dfn> is a message delivery context established between the
-          <a>user agent</a> and the <a>push service</a> on behalf of a web application. Each
-          <a>push subscription</a> is associated with a <a>service worker registration</a> and a
-          <a>service worker registration</a> has at most one <a>push subscription</a>.
+          <a>user agent</a> and the <a>push service</a> on behalf of a web application.
+        </p>
+        <p>
+          A [=push subscription=] has an associated <dfn for="push subscription">scope</dfn>, which
+          is a [=/URL=].
+        </p>
+        <p>
+          A [=push subscription=] is considered to have a <dfn for=
+          "push subscription">window-accessible scope</dfn> when its [=push subscription/scope=] is
+          a [=/list=] of [=list/size=] 1 and [=push subscription/scope=][0] is the empty string.

Indeed, this should look at the scope's path instead of scope.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/w3c/push-api/pull/393#discussion_r1969888221
You are receiving this because you are subscribed to this thread.

Message ID: <w3c/push-api/pull/393/review/2641263476@github.com>