Hi James,
Many thanks for taking the time to review and comment, it’s very much
appreciated.
On 15-Oct-2009, at 20:18, James Holderness wrote:
Some comments on the current draft:
1. Get your own namespace for the new elements. I'm almost positive
you're not going to be able to extend the existing Atom namespace
the way you're proposing, and I don't see any advantage to doing so
even if you could. Having your own namespace doesn't stop you
reusing Atom elements where needed.
Noted, will tweak.
2. In your draft spec, the group element has collapsed attribute,
but in your example you're using an expanded attribute. I'm assuming
one or the other is out of date.
Ah, example is incorrect. It should be “collapsed” (with omission
being collapsed="no").
3. The spec seems to indicate that the collapsed attribute is in the
atom namespace. However in your example file it's in the default
namespace which would make more sense.
Spec’s wrong on that front.
4. Why restrict the author element in the feedset to no more than
one? That may be all that is required for your needs, but you need
to consider how others might use the format too. Given you're using
Atom elements it would seem logical to follow Atom's lead and allow
both multiple authors and contributors.
Spec’s wrong there, too—it was supposed to follow Atom’s lead in
general, so I’m more than happy to defer.
I’ll revise both spec/examples later on this evening to fix the above.
All the best,
Mo.
--
mo mcroberts
http://nevali.net
iChat: [email protected] Jabber/GTalk: [email protected] Twitter:
@nevali
Run Leopard or Snow Leopard? Set Quick Look free with DropLook -
http://labs.jazzio.com/DropLook/
