-
Notifications
You must be signed in to change notification settings - Fork 510
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
I think the page is outdated #3139
Comments
Thanks for filing this issue. I've added it to our backlog. We'll take a look and try to make the page clearer/more up to date. |
The manifest file is correctly linked to from the docs: https://github.com/MicrosoftEdge/Demos/blob/main/1DIV/dist/manifest.json Can you elaborate on what doesn't make sense for you here?
The app.js file does use the So, thinking about this again, I'm not sure what's the source of the confusion here. Can you please elaborate on what changes you'd like us to make in order to make this article clearer? |
Thank you @captainbrosset for clarifying. I was of the impression that the manifest file should be on top like in an Electron-app, it was weird to find the manifest in the distributable. The navigator was still not accessible from my Electron scripts, is there a way for an Electron app to access the Window Controls Overlay? |
No, the Window Controls Overlay is a PWA feature only. |
dist
folder. That does not make sense.I was not able to access this API.
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.
AB#50475498
The text was updated successfully, but these errors were encountered: