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

Split the definition of web-based protocol handler from the definition of registerProtocolHandler #9678

Open
SoniEx2 opened this issue Sep 3, 2023 · 8 comments

Comments

@SoniEx2
Copy link

SoniEx2 commented Sep 3, 2023

Recent PWA support for protocol handlers would benefit from splitting it off of registerProtocolHandler. A web-based protocol handler should just be an endpoint that follows the protocol handler template - but it need not be passed to registerProtocolHandler to be considered a web-based protocol handler.

@annevk
Copy link
Member

annevk commented Sep 3, 2023

I thought that specification already integrated with this one?

@SoniEx2
Copy link
Author

SoniEx2 commented Sep 3, 2023

It would genuinely clarify things so much to separate a "web-based protocol handler" from "a call to registerProtocolHandler".

@annevk
Copy link
Member

annevk commented Sep 3, 2023

Can you point me to the definition that is confusing? I agree that they shouldn't directly invoke this method, that is unlikely to work.

@SoniEx2
Copy link
Author

SoniEx2 commented Sep 3, 2023

Sure. The title of this section: https://html.spec.whatwg.org/multipage/system-state.html#custom-handlers is: "Custom scheme handlers: the registerProtocolHandler() method", which directly references registerProtocolHandler() and is pretty much entirely defined by registerProtocolHandler().

@annevk
Copy link
Member

annevk commented Sep 3, 2023

I'm familiar with that section, I was asking about the other specification trying to reuse it.

@SoniEx2
Copy link
Author

SoniEx2 commented Sep 3, 2023

Well we also wanna reference web-based protocol handlers in our talks without having to spend 10 minutes separating them from registerProtocolHandler...

@annevk
Copy link
Member

annevk commented Sep 4, 2023

A talk is not sufficient motivation to change a standard.

@SoniEx2
Copy link
Author

SoniEx2 commented Sep 4, 2023

It is if it highlights clarity issues with the standard.

Like, this isn't even a functional change, it's merely an editorial one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants
@annevk @SoniEx2 and others