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

Replace appmanifest's launch algorithm with one that has launch_handler behaviors #79

Merged
merged 5 commits into from Nov 8, 2022

Conversation

alancutter
Copy link
Collaborator

@alancutter alancutter commented Oct 28, 2022

#67 #69 #84

This builds on top of w3c/manifest#1056 which adds a launch web application algorithm to the manifest spec. This updates the launch handler spec to replace that algorithm with its own. This enables other webapp specs (not in WICG) to gain the launch_handler behaviour without directly referencing this WICG spec.

Additionally this replaces the concept of "web app launch client" with the existing "application context" concept from the manifest spec which is effectively the same thing.

@alancutter
Copy link
Collaborator Author

I'll wait for w3c/manifest#1056 to land before merging this one.

Copy link
Collaborator

@fallaciousreasoning fallaciousreasoning left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, with one comment

index.html Outdated Show resolved Hide resolved
Copy link
Member

@mgiuca mgiuca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks good, no comments on the code (though see separate bugs here and here).

Just some comments on the algorithm structure / redundancy.

index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
a9ac7fc
Branch: replace-launch-algorithm

5e7a975
Clean up

7444e95
clean up variables

cd6973a
Add missing return
@alancutter
Copy link
Collaborator Author

The manifest exports need a bit of time to propagate through, will hold off landing until then to make sure the xrefs work correctly.

@alancutter
Copy link
Collaborator Author

The red squiggles remaining will be fixed after w3c/manifest#1062 lands and the xrefs have time to propagate through.

@alancutter alancutter merged commit 6860bf5 into WICG:main Nov 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants