Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clarify order of appinstalled event and prompt() result #35

Open
beaufortfrancois opened this issue Sep 3, 2021 · 0 comments
Open

Clarify order of appinstalled event and prompt() result #35

beaufortfrancois opened this issue Sep 3, 2021 · 0 comments
Labels
install-prompt https://wicg.github.io/manifest-incubations/index.html#installation-prompts

Comments

@beaufortfrancois
Copy link

Spec is not clear on whether the appinstalled event should fire before or after prompt() promise resolves. This may lead to compatibility issues if not addressed.

https://wicg.github.io/manifest-incubations/index.html#dfn-present-an-install-prompt
https://wicg.github.io/manifest-incubations/index.html#dfn-steps-to-install-the-web-application

I would expect the events to fire in order, user responds to the prompt, then the install to happen.

  1. beforeinstallprompt event fires
  2. prompt() resolves
  3. appinstalled event fires

Chromium issue: https://bugs.chromium.org/p/chromium/issues/detail?id=1094025

@phoglenix phoglenix added the install-prompt https://wicg.github.io/manifest-incubations/index.html#installation-prompts label Sep 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
install-prompt https://wicg.github.io/manifest-incubations/index.html#installation-prompts
Projects
None yet
Development

No branches or pull requests

2 participants