Re: [heycam/webidl] Add [InNamespace] to allow defining an interface in a namespace (#425)

Thanks for the PR!

A comment and some bikeshedding.

My understanding was the wanted to avoid such constructs in favor of waiting for modules instead, but maybe things have changed?

If we do go this route though, shouldn't we go for dedicated syntax here instead of (ab)using extended attributes? E.g.: something like:

```webidl
namespace Foo { };

interface Foo.Bar {
};

interface Foo.Baz : Foo.Bar {
};
```
And if we do choose extended attributes, maybe `[Namespace=Foo]` reads better than `[InNamespace=Foo]`.



-- 
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/pull/425#issuecomment-324746689