Re: [w3c/manifest] BREAKING CHANGE: install event renamed appinstalled (closes #502) (#503)

mgiuca commented on this pull request.

lgtm with additional change.

@dominickng what are your thoughts on this rename? (See #502 for the issue).

> @@ -419,37 +419,38 @@ <h3 id="installation-sec">
         </p>
         <pre class="idl">
           partial interface Window {
-            attribute EventHandler oninstall;
+            attribute EventHandler onappinstalled;
           };
         </pre>
         <div class="example">
           <p>
             This example shows two ways of handling the install event.

s/install/appinstalled

-- 
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/manifest/pull/503#pullrequestreview-3246616