WICG / manifest-incubations

Before install prompt API for installing web applications
https://wicg.github.io/manifest-incubations/
Other
98 stars 30 forks source link

Add borderless mode explainer #73

Closed sonkkeli closed 1 year ago

mgiuca commented 1 year ago

Hi @sonkkeli

Thanks for writing this up. I had a read over it. I will do a drive-by and give some in-line comments in a bit, but my overall thoughts is that the proposal makes sense, the write-up needs to be less Chrome and ChromeOS-focused.

Understanding that this is being primarily driven by ChromeOS, we still need to write an explainer that considers the API in the wider web ecosystem, given that it is being proposed as a W3C standard. I'll put direct comments in-line. I think a lot of this text could go into an internal design doc (public, if desired).

Note that it is still OK to reference specific browsers and OSes in explainers, but they need to be by way of example, not the exclusive focus of the document.

sonkkeli commented 1 year ago

Hey Matt and thanks for all the comments, they made the explainer much better I think. There's still a few unresolved comments with some open Qs but in general I think it is ready for another PTAL . :)

sonkkeli commented 1 year ago

I don't have the "power" to merge this btw. Could someone else click the button?