- From: mark a. foltz <mfoltz@google.com>
- Date: Wed, 21 Jan 2015 13:31:31 -0800
- To: "Kostiainen, Anssi" <anssi.kostiainen@intel.com>
- Cc: "Rottsches, Dominik" <dominik.rottsches@intel.com>, "public-secondscreen@w3.org" <public-secondscreen@w3.org>
- Message-ID: <CALgg+HHc++xxD8ri1dTpoH841XMHnB=U52eHeut-UHd4Y2tHqQ@mail.gmail.com>
Dominik, Thank you for posting an improved pull request. Overall it's a big improvement in structure. I made some specific suggestions in the PR thread. m. On Wed, Jan 21, 2015 at 2:14 AM, Kostiainen, Anssi < anssi.kostiainen@intel.com> wrote: > Hi Dominik, All, > > > On 21 Jan 2015, at 11:25, Rottsches, Dominik < > dominik.rottsches@intel.com> wrote: > > > > Hi Anssi, > > > > thanks for the PR - it was very helpful in highlighting a couple of > > inconsistencies and aligning with the conventional pattern / > > organization of "explanations below IDL" which other specifications use. > > Dominik - thanks for the updates! I like how the spec now establishes > common idioms to be used across the spec for consistency. Also, aligning > with the established spec style should help people from other groups to > review our work more easily. > > >> Dominik indicated he's working on the first pass review and expansion. > >> My expectation is he will share his results with the group shortly, > >> and before doing so will update the PR, to ease the review. > > > > In https://github.com/w3c/presentation-api/pull/43 > > I tried to address your comments and follow your suggestions for > > organizing the document. > > > > I closed the .XXX items, except one, which I transformed into an open > > issue: > > > > "Do we want to keep the attribute readonly session of > > NavigatorPresentation?" How can we consistently define what this > > attribute represents? I think this attribute easily gets confusing to a > > client of our API. > > All - please let us know if you are aware of good use case(s) for this > attribute. > > Dominik - please feel free to open an issue to track this. > > > It would be great if Anssi and Mark could review this updated version. > > If it looks good, I'll merge this and we can continue editing. > > All - please review the improvements Dominik landed in PR#43. The HTML > preview with Dominik's changes baked in is at: > > http://rawgit.com/drott/presentation-api/anssi_reorg/index.html > > Personally I feel that these changes improve the readability of the spec. > If we hear no concerns from the group, I'd suggest the editor will merge > these changes. > > Thanks, > > -Anssi
Received on Wednesday, 21 January 2015 21:32:19 UTC