search

WICG / isolated-web-apps

Repository for explainers and other documents related to the Isolated Web Apps proposal.
Other
223 stars 19 forks source link

Incorporate most IsolatedContext review feedback #43

Closed robbiemc closed 3 months ago

robbiemc commented 4 months ago

This incorporates most of @domfarolino's feedback from issue #42. It doesn't address the biggest issue about going from environment settings object to browsing context group, which will be in another PR.

Preview | Diff

robbiemc commented 4 months ago

@domfarolino, can you PTAL as well?

robbiemc commented 3 months ago

I'm going to go ahead and land this and #44 to avoid merge conflicts with other changes. @domfarolino, I'm happy to continue working on this when you're back from vacation.