- From: Boris Zbarsky <notifications@github.com>
- Date: Tue, 17 Jul 2018 11:40:18 -0700
- To: heycam/webidl <webidl@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Tuesday, 17 July 2018 18:40:41 UTC
Hmm. So namespaces only allow readonly attributes. But `[Replaceable]` could in theory still make sense. That said, in practice it doesn't make sense to me; `[Replaceable]` is basically a backwards-compat hack, and there's no need for that on namespaces.... There is definitely a spec bug here, though. https://heycam.github.io/webidl/#define-the-attributes can have a namespace as _definition_ and in step 3 does https://heycam.github.io/webidl/#dfn-attribute-setter with that namespace as _target_ if that happens, but https://heycam.github.io/webidl/#dfn-attribute-setter assumes that _target_ is not a namespace. We need to fix that. Probably by just having step 1 of https://heycam.github.io/webidl/#dfn-attribute-setter return **undefined** if _target_ is a namespace. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/heycam/webidl/issues/572#issuecomment-405685553
Received on Tuesday, 17 July 2018 18:40:41 UTC