- From: Jussi Kalliokoski <jussi.kalliokoski@gmail.com>
- Date: Sun, 21 Oct 2012 16:54:50 +0300
- To: Chris Wilson <cwilso@google.com>
- Cc: Thierry MICHEL <tmichel@w3.org>, olivier Thereaux <olivier.thereaux@bbc.co.uk>, "public-audio@w3.org Group" <public-audio@w3.org>
- Message-ID: <CAJhzemXx3Z9pWrKwH_D6H1snk5GZyQU4Xd07N1hDMUm5ZKP29w@mail.gmail.com>
Thanks for fixing the remaining issues, Chris! Interesting, it appears that the link validator has JS disabled, because respec.js adds those "missing" IDs and they were working for me. Cheers, Jussi On Fri, Oct 19, 2012 at 8:57 PM, Chris Wilson <cwilso@google.com> wrote: > Hi Thierry, > > I fixed the remaining issues, and made sure the doc validated cleanly for > HTML, CSS and link checks. It should be ready to go. > > -Chris > > > On Fri, Oct 19, 2012 at 8:06 AM, Thierry MICHEL <tmichel@w3.org> wrote: > >> Jussi, >> >> Thanks for your updates. >> There are still minor issues remaining with broken anchors: >> >> see checker result: >> http://validator.w3.org/**checklink?uri=https%3A%2F%** >> 2Fdvcs.w3.org%2Fhg%2Faudio%**2Fraw-file%2Ftip%2Fmidi%** >> 2Fspecification.html&hide_**type=all&depth=&check=Check<http://validator.w3.org/checklink?uri=https%3A%2F%2Fdvcs.w3.org%2Fhg%2Faudio%2Fraw-file%2Ftip%2Fmidi%2Fspecification.html&hide_type=all&depth=&check=Check> >> >> >> 1- Broken fragments: >> >> http://www.khronos.org/**registry/typedarray/specs/**latest/#7<http://www.khronos.org/registry/typedarray/specs/latest/#7>(line 130) >> >> That is curious, the URL is indeed correct. Seems like a bug in the >> validator ;-) >> >> 2- Broken fragments: >> >> error Lines: 153, 257, 261, 264, 330 https://dvcs.w3.org/hg/audio/** >> raw-file/tip/midi/**specification.html<https://dvcs.w3.org/hg/audio/raw-file/tip/midi/specification.html> >> Status: 200 Script output follows >> >> Some of the links to this resource point to broken URI fragments (such as >> index.html#fragment). >> >> >> * https://dvcs.w3.org/hg/audio/**raw-file/tip/midi/** >> specification.html#idl-def-**NavigatorMIDIAccess<https://dvcs.w3.org/hg/audio/raw-file/tip/midi/specification.html#idl-def-NavigatorMIDIAccess>(line 153) >> >> you have the following link <a href="#idl-def-**NavigatorMIDIAccess" >> but you are missing an id="idl-def-**NavigatorMIDIAccess" somewhere >> >> >> * https://dvcs.w3.org/hg/audio/**raw-file/tip/midi/** >> specification.html#idl-def-**NavigatorMIDIAccessError<https://dvcs.w3.org/hg/audio/raw-file/tip/midi/specification.html#idl-def-NavigatorMIDIAccessError>(line 257) >> >> Same here >> <a href="#idl-def-**NavigatorMIDIAccessError" >> but you are missing an id="idl-def-**NavigatorMIDIAccessError" somewhere >> >> >> * https://dvcs.w3.org/hg/audio/**raw-file/tip/midi/** >> specification.html#dom-**navigatormidiaccesserror-**PERMISSION_DENIED<https://dvcs.w3.org/hg/audio/raw-file/tip/midi/specification.html#dom-navigatormidiaccesserror-PERMISSION_DENIED>(lines 264, 330) >> >> You have a "title" attribute, should be an "id" attribute >> <code title="dom-**NavigatorMIDIAccessError-**PERMISSION_DENIED"> >> for the href >> <a href="#dom-**navigatormidiaccesserror-**PERMISSION_DENIED" >> >> Be careful about the Camel case used above, it is different than the one >> in the title attribute ... >> >> and then another >> <code title="dom-**NavigatorMIDIAccessError-**PERMISSION_DENIED"> >> >> you should use "id" instead of "title" and use a different id string value >> for >> <a href="#dom-**navigatormidiaccesserror-**PERMISSION_DENIED">PERMISSION_ >> **DENIED</a> >> >> >> * https://dvcs.w3.org/hg/audio/**raw-file/tip/midi/** >> specification.html#dom-**navigatormidiaccesserror-code<https://dvcs.w3.org/hg/audio/raw-file/tip/midi/specification.html#dom-navigatormidiaccesserror-code>(line 261) >> >> <code title="dom-**NavigatorMIDIAccessError-code"**> >> >> Same here should be an id attribute >> <a href="#dom-**navigatormidiaccesserror-code"**>code</a> >> >> Be careful about the Camel case used above, it is different than the one >> in the title attribute ... >> >> -------------- >> >> I have fixed the output document (without the scripts) for these broken >> anchors >> http://www.w3.org/2011/audio/**drafts/midi/1WD/Overview.html<http://www.w3.org/2011/audio/drafts/midi/1WD/Overview.html> >> >> The document is now ready for publication. >> >> But please fix these bugs in your source version, >> https://dvcs.w3.org/hg/audio/**raw-file/tip/midi/**specification.html<https://dvcs.w3.org/hg/audio/raw-file/tip/midi/specification.html> >> so we don't get these again at next publication version. >> >> Best >> >> Thierry >> >> >> >> >> On 19/10/2012 14:02, Jussi Kalliokoski wrote: >> >>> On Fri, Oct 19, 2012 at 9:35 AM, Thierry MICHEL <tmichel@w3.org> wrote: >>> >>> Chris, >>>> >>>> Done, changed to >>>> http://www.w3.org/TR/webmidi/ >>>> >>>> >>> Thanks, Thierry! >>> >>> >>> Could you also change the following in your MIDI source >>>> https://dvcs.w3.org/hg/audio/****raw-file/tip/midi/**** >>>> specification.html<https://dvcs.w3.org/hg/audio/**raw-file/tip/midi/**specification.html> >>>> <https://**dvcs.w3.org/hg/audio/raw-file/**tip/midi/specification.html<https://dvcs.w3.org/hg/audio/raw-file/tip/midi/specification.html> >>>> > >>>> >>>> >>> Yes: https://dvcs.w3.org/hg/audio/**rev/50a6b5936add<https://dvcs.w3.org/hg/audio/rev/50a6b5936add> >>> >>> >>> As we must publish valid documents (HTML/CSS/links,etc) >>>> >>>> 1- comment the following CSS selector >>>> >>>> /* tbody { height: 300px; overflow: auto; } */ >>>> >>>> >>> I removed the line, we are using source control after all. :) >>> >>> >>> 2- add an attribute type="text/css" to <style> >>>> >>>> >>> Done. >>> >>> >>> see CSS validator result >>>> http://jigsaw.w3.org/css-****validator/validator?uri=https%**** >>>> 3A%2F%2Fdvcs.w3.org%2Fhg%****2Faudio%2Fraw-file%2Ftip%** >>>> 2Fmidi%2Fspecification.html&****profile=css3&usermedium=all&** >>>> warning=1&vextwarning=&lang=**fr<http://jigsaw.w3.org/css-** >>>> validator/validator?uri=https%**3A%2F%2Fdvcs.w3.org%2Fhg%** >>>> 2Faudio%2Fraw-file%2Ftip%**2Fmidi%2Fspecification.html&** >>>> profile=css3&usermedium=all&**warning=1&vextwarning=&lang=fr<http://jigsaw.w3.org/css-validator/validator?uri=https%3A%2F%2Fdvcs.w3.org%2Fhg%2Faudio%2Fraw-file%2Ftip%2Fmidi%2Fspecification.html&profile=css3&usermedium=all&warning=1&vextwarning=&lang=fr> >>>> **> >>>> >>> >>> >>> Good for some French practise! ;) >>> >>> >>> 3- fix the following broken fragments: >>>> * http://www.khronos.org/****registry/typedarray/specs/****latest/#7<http://www.khronos.org/**registry/typedarray/specs/**latest/#7> >>>> <http://www.khronos.**org/registry/typedarray/specs/**latest/#7<http://www.khronos.org/registry/typedarray/specs/latest/#7>>(line >>>> 136) >>>> >>>> >>> The fragment is actually referring to the right place, i.e. an h2 element >>> with the id="7". Is this a bug in the validator or is there something >>> invalid about that id? Either way, I think we need to complain somewhere >>> else first. :P >>> >>> >>> * https://dvcs.w3.org/hg/audio/****raw-file/tip/midi/**<https://dvcs.w3.org/hg/audio/**raw-file/tip/midi/**> >>>> specification.html#idl-def-****NavigatorMIDIAccess<https://** >>>> dvcs.w3.org/hg/audio/raw-file/**tip/midi/specification.html#** >>>> idl-def-NavigatorMIDIAccess<https://dvcs.w3.org/hg/audio/raw-file/tip/midi/specification.html#idl-def-NavigatorMIDIAccess> >>>> >(**line 159) >>>> >>>> >>> Not sure what on earth happened with this one, I retyped it by hand with >>> replace mode character by character and now it works. There's some dark >>> magic at play here. :S >>> >>> >>> * https://dvcs.w3.org/hg/audio/****raw-file/tip/midi/**<https://dvcs.w3.org/hg/audio/**raw-file/tip/midi/**> >>>> specification.html#dom-****navigatormidiaccesserror-code<** >>>> https://dvcs.w3.org/hg/audio/**raw-file/tip/midi/** >>>> specification.html#dom-**navigatormidiaccesserror-code<https://dvcs.w3.org/hg/audio/raw-file/tip/midi/specification.html#dom-navigatormidiaccesserror-code> >>>> >**(line 267) >>>> >>>> * https://dvcs.w3.org/hg/audio/****raw-file/tip/midi/**** >>> specification.html#dom-<https://dvcs.w3.org/hg/audio/**raw-file/tip/midi/**specification.html#dom-> >>> >>>> **navigatormidiaccesserror-****permission_denied<https://** >>>> dvcs.w3.org/hg/audio/raw-file/**tip/midi/specification.html#** >>>> dom-navigatormidiaccesserror-**permission_denied<https://dvcs.w3.org/hg/audio/raw-file/tip/midi/specification.html#dom-navigatormidiaccesserror-permission_denied>>(lines >>>> 270, 336) >>>> * https://dvcs.w3.org/hg/audio/****raw-file/tip/midi/**** >>>> specification.html#**<https://dvcs.w3.org/hg/audio/**raw-file/tip/midi/**specification.html#**> >>>> navigatormidiaccesserror<https**://dvcs.w3.org/hg/audio/raw-** >>>> file/tip/midi/specification.**html#navigatormidiaccesserror<https://dvcs.w3.org/hg/audio/raw-file/tip/midi/specification.html#navigatormidiaccesserror> >>>> >**(line 263) >>>> >>>> >>> Fixed. >>> >>> >>> * http://dev.w3.org/html5/spec/****webappapis.html#function<http://dev.w3.org/html5/spec/**webappapis.html#function> >>>> <http**://dev.w3.org/html5/spec/**webappapis.html#function<http://dev.w3.org/html5/spec/webappapis.html#function>>(line >>>> 119) >>>> >>>> >>> Oh they've gone more language-agnostic, this is actually an important >>> catch >>> then. Apparently also WebIDL now has the type `callback` accordingly, so >>> I >>> removed the whole reference to `Function` and used callback instead, I >>> think that's the intention. >>> >>> >>> see link checker result >>>> http://validator.w3.org/****checklink?uri=https%3A%2F%** >>>> 2Fdvcs.w3.org%2Fhg%2Faudio%****2Fraw-file%2Ftip%2Fmidi%** >>>> 2Fspecification.html&hide_****type=all&depth=&check=Check<ht** >>>> tp://validator.w3.org/**checklink?uri=https%3A%2F%** >>>> 2Fdvcs.w3.org%2Fhg%2Faudio%**2Fraw-file%2Ftip%2Fmidi%** >>>> 2Fspecification.html&hide_**type=all&depth=&check=Check<http://validator.w3.org/checklink?uri=https%3A%2F%2Fdvcs.w3.org%2Fhg%2Faudio%2Fraw-file%2Ftip%2Fmidi%2Fspecification.html&hide_type=all&depth=&check=Check> >>>> > >>>> >>>> >>> Thanks for the resource, it's very useful! Although it seems to be doing >>> caching so can't see if it validates now... But I checked that all the >>> links it was complaining about go to the right place (for me) now. >>> >>> Cheers, >>> Jussi >>> >>> >>> Let me know when you are done and I will use your updated document for >>>> publication. >>>> >>>> Best, >>>> >>>> Thierry >>>> >>>> >>>> >>>> >>>> On 19/10/2012 00:56, Chris Wilson wrote: >>>> >>>> Looks great. I'd prefer the "webmidi" shortname, to be symmetric with >>>>> webaudio. >>>>> >>>>> >>>>> On Thu, Oct 18, 2012 at 3:02 PM, Thierry MICHEL <tmichel@w3.org> >>>>> wrote: >>>>> >>>>> >>>>> Olivier, >>>>>> >>>>>> I had published the MIDI spec in the temporary URL to check it before >>>>>> moving it to TR. >>>>>> >>>>>> http://www.w3.org/2011/audio/******drafts/midi/1WD/Overview.**html<http://www.w3.org/2011/audio/****drafts/midi/1WD/Overview.html> >>>>>> <http://www.w3.org/2011/**audio/**drafts/midi/1WD/**Overview.html<http://www.w3.org/2011/audio/**drafts/midi/1WD/Overview.html> >>>>>> > >>>>>> **<http://www.w3.org/2011/**audio/**drafts/midi/1WD/**Overview.html<http://www.w3.org/2011/audio/**drafts/midi/1WD/Overview.html> >>>>>> <http://www.w3.**org/2011/audio/drafts/midi/**1WD/Overview.html<http://www.w3.org/2011/audio/drafts/midi/1WD/Overview.html> >>>>>> > >>>>>> >>>>>> >>>>>>> >>>>>> >>>>>> I had used the short name >>>>>> http://www.w3.org/TR/midi-api/ >>>>>> >>>>>> but if that does not fit you, I can change it to >>>>>> http://www.w3.org/TR/webmidi/ >>>>>> >>>>>> Let me know ASAP and I will request it. >>>>>> >>>>>> For the document, >>>>>> >>>>>> I have updated the following: >>>>>> >>>>>> - SOTD section >>>>>> - Date and Title >>>>>> - previous, latest, editor's versions URIs, >>>>>> >>>>>> Also I have commented the following CSS selector >>>>>> /* tbody { height: 300px; overflow: auto; } */ >>>>>> >>>>>> which produces ugly tables, which large height cells >>>>>> >>>>>> The document is now ready. >>>>>> >>>>>> If I don't have objections by monday afternoon (French time), let's >>>>>> say >>>>>> *noon Boston time*, I will then request Transition and then >>>>>> Publication. >>>>>> >>>>>> Thierry >>>>>> >>>>>> >>>>>> >>>>>> >>>>>> >>>>>> >>>>>> >>>>>> On 18/10/2012 22:01, olivier Thereaux wrote: >>>>>> >>>>>> Dear all, >>>>>> >>>>>>> >>>>>>> At the teleconference yesterday, we discussed the Web MIDI API spec, >>>>>>> and >>>>>>> agreed we'd like to publish as First Public Working Draft next week >>>>>>> if >>>>>>> Chris found the time to make a few edits - which, as you can see in >>>>>>> his >>>>>>> e-mail he did. >>>>>>> >>>>>>> Publishing at this point will allow us to publicise the work at TPAC, >>>>>>> and >>>>>>> get more feedback from the public on a spec which is already quite >>>>>>> mature. >>>>>>> Kudos to the editors. >>>>>>> >>>>>>> If there is no objection by Tuesday morning (Boston time) we will >>>>>>> assume >>>>>>> consensus and will move forward with publication. >>>>>>> >>>>>>> Thierry will lead the process for publication, as usual. >>>>>>> Thierry, since this is a FPWD, can you get the green light for the >>>>>>> "webmidi" shortname? >>>>>>> >>>>>>> Thank you. >>>>>>> Olivier >>>>>>> >>>>>>> On 18 Oct 2012, at 00:42, Chris Wilson <cwilso@google.com> wrote: >>>>>>> >>>>>>> I have updated the Web MIDI API specification, as per the >>>>>>> discussion >>>>>>> in >>>>>>> >>>>>>> today's teleconference, with one addition. >>>>>>>> >>>>>>>> The changeset is here: https://dvcs.w3.org/hg/audio/*****<https://dvcs.w3.org/hg/audio/***> >>>>>>>> *rev/f4727ce84474 <https://dvcs.w3.org/hg/audio/** >>>>>>>> **rev/f4727ce84474<https://dvcs.w3.org/hg/audio/**rev/f4727ce84474> >>>>>>>> >< >>>>>>>> https://**dvcs.w3.org/hg/**audio/rev/**f4727ce84474<http://dvcs.w3.org/hg/audio/rev/**f4727ce84474> >>>>>>>> <https**://dvcs.w3.org/hg/audio/rev/**f4727ce84474<https://dvcs.w3.org/hg/audio/rev/f4727ce84474> >>>>>>>> > >>>>>>>> >>>>>>>>> >>>>>>>>> . >>>>>>>> >>>>>>>> The updated spec is here: https://dvcs.w3.org/hg/audio/******<https://dvcs.w3.org/hg/audio/****> >>>>>>>> <https://dvcs.w3.org/hg/**audio/**<https://dvcs.w3.org/hg/audio/**> >>>>>>>> > >>>>>>>> raw-file/tip/midi/******specification.html<https://** >>>>>>>> dvcs.w3.org/hg/audio/raw-file/****tip/midi/specification.html<http://dvcs.w3.org/hg/audio/raw-file/**tip/midi/specification.html> >>>>>>>> <**https://dvcs.w3.org/hg/audio/**raw-file/tip/midi/** >>>>>>>> specification.html<https://dvcs.w3.org/hg/audio/raw-file/tip/midi/specification.html> >>>>>>>> > >>>>>>>> >>>>>>>> >>>>>>>>> >>>>>>>> . >>>>>>>> >>>>>>>> I made the following edits: >>>>>>>> • I expanded the overview section, making it clear that >>>>>>>> this >>>>>>>> API >>>>>>>> is not intended to cover semantic controls through MIDI (i.e. a >>>>>>>> solution to >>>>>>>> the web of things problem), and also that this API is not concerned >>>>>>>> with >>>>>>>> Standard MIDI Files or General MIDI - that is, that it is concerned >>>>>>>> with >>>>>>>> input and output, not "playback" per se. >>>>>>>> • I greatly expanded the introduction section to provide a >>>>>>>> more >>>>>>>> table-of-contents style overview of the API, and also to describe in >>>>>>>> more >>>>>>>> detail how the API is intended to function. >>>>>>>> • I essentially rewrote the Security and Privacy >>>>>>>> considerations >>>>>>>> section to describe the fingerprinting and access concerns in more >>>>>>>> detail, >>>>>>>> and also (per conversation in telecon) to explicitly leave the model >>>>>>>> open. >>>>>>>> These three edits should resolve bug 19187. >>>>>>>> • I changed the sendMessage() method back to my suggested >>>>>>>> three-parameter form, and explicitly excluded sysex from >>>>>>>> sendMessage(). I >>>>>>>> expect further discussion on this point, but for our FPWD, I wanted >>>>>>>> to >>>>>>>> have >>>>>>>> it this way as I was brainstorming the security and privacy >>>>>>>> constraints, >>>>>>>> and I think it may be possible to use sysex as the "needs user >>>>>>>> permission" >>>>>>>> switch - that is, to require user permission ONLY in order to >>>>>>>> send/receive >>>>>>>> sysex. I'm not positive this will be enough, but it will be easier >>>>>>>> to >>>>>>>> change it back than it would be to break variadic usage later. ref: >>>>>>>> Bug >>>>>>>> 18764. >>>>>>>> • I explicitly made timestamps in MIDIMessage allowed to >>>>>>>> be >>>>>>>> set >>>>>>>> to zero, with the semantic of "send now", as per Bug 18760. >>>>>>>> • I added IDs to several elements in order to provide >>>>>>>> forward >>>>>>>> links, and I expanded some of the IDL constructs to better show the >>>>>>>> descriptions of individual method parameters or members. (no >>>>>>>> substantive >>>>>>>> changes.) >>>>>>>> -Chris >>>>>>>> >>>>>>>> >>>>>>>> >>>>>>> >>>>>>> >>>>> >>> >
Received on Sunday, 21 October 2012 13:55:18 UTC