- From: T.V Raman <raman@google.com>
- Date: Wed, 13 Sep 2006 16:43:48 -0700
- To: noah_mendelsohn@us.ibm.com
- Cc: raman@google.com, www-tag@w3.org
Noah comments in-lined. You caught a lot of the typos I usually catch with a perl linter that I run at the very end --- I'll describe what I did for your various comments: noah_mendelsohn@us.ibm.com writes: > Raman, > > Mostly this looks fine and ready to go. I did notice a few more purely > editorial issues, though the first two are significant and need to be > fixed. That one is that the status section has a statement "An informal > guide to previous discussion of this topic is available and may be useful > to reviewers of this draft." The link is in fact to the draft itself. I > strongly suspect that this entire sentence traces to your > perhaps having Good guess, it's gone. > adopted the metadataInUri draft finding as a starting point. It had such > a link to informal discussion, so my guess is that this entire sentence is > to be deleted. Anyway, the link is bogus. > > The other significant comment is that you also have carried over the usual > reference to RFC 2119, but in fact the formal uppercase SHOULD, MUST, MUST > NOT, etc. are not used in the finding. I suggest that the > sentence in the Nuked that reference, and turned the "Should Not" at the end to > lower case by rewriting that sentence. > status, and the corresponding bibliography entry be dropped. (Warning: I > do comment later on one place in which the finding uses the odd > capitalization Should Not; my recommendation is to lowercase that one, > but if you uppercase it, then obviously you should keep the RFC 2119 > reference after all.) > > Other less significant comments follow. I think most of these are worth > fixing, but I leave that entirely to you. There are a fair number of > them. If you want to fix only the two above and ship, I'm fine with that. > I don't think the suggestions below are the sort of thing that would > trigger another round of formal reviews. Most are typos or barely above > that in significance. I'm afraid these are in the order I discovered > them, which is not necessarily document order. > > If you find that you do want to fix these but that the editing is > burdensome, I'll be glad to share the load. We can pick a time when the > CVS copy is clean and I'll be glad to help patch up some of these, > particularly where the issues are just punctuation and spacing. > > --- > From 2.1.1 list item 5: > > "In contrast, links meant for machine consumption, e.g., Atom/RSS feeds > might use the HTML link element." > > I think that this is hard to parse, partly due to the position > of the this wasn't quite saying what I watned it to, rewrote it -- please look it over. > commas and partly due to the wording. I think what you meant was that > Atom/RSS feeds are examples of resources to which an HTML document might > link for purposes of machine consumption. My eye tends to hang up on the > comma after the e.g., and tries to parse as "Atom/RSS feeds might use the > HTML link element", before backtracking and realizing that's not intended. > Proposed rewording: > > "In contrast, links meant for machine consumption, e.g. links to Atom/RSS > feeds, might use the HTML link element." > > --- > From 2.1.1: > > > "If no content negotiation is in place, Serve a canonical representation > of the content at http://example.com/ubiquity/resource" > Fixed > The word "Serve" should be lowercase. > > --- > The punctuation that terminates list items is inconsistent throughout the > finding. > > For example in the abstract, there's a list of: This was a consequence of specxml structuring, I used a slist there, and ended u using "," at each bullet; now nuked. > > Representations appropriate for different delivery contexts, > Representations in different languages, > > I'm not sure why these end with commas. The style guides I've seen mainly > say to use periods for list items if and only if they are each complete > sentences, and otherwise to use no punctuation at all. In any case, the > commas look suspect. > --- > In chapter 3: > > "As can be seen from the use-cases and suggested solutions enumerated in > the previous section, pointers to Web Resources (URIs) can either: > Fixed. nuked Can > * Be canonical URIs, i.e., have no context hard-wired. > * Can encapsulate partial context, e.g., encapsulate language, > * Encapsulate multiple context bits, e.g., language and device > profile, > * Capture all context, i.e., the creator of the URI guarantees that > all state is completely captured by the URI." > > I think the word "Can" should be deleted from the second item in the list. > --- > > In the abstract: > Fixed. > "This document explores the issues that arise in this context, and > attempts to define best practices that help toward: > > * Preserve the One Web while enabling content publishing to a > multiplicity of delivery contexts. > * Enable automatic discovery of the available representations. > * Enable the creation of RESTful URIs that remain representation > agnostic while delivering the correct end-user experience." > > I think it would be better grammar to delete the word "toward", so the > list can be interpreted as "practices that help Preserve the One Web", > "practices that Enable automatic discovery", etc. > Remaining typos fixed as you suggested. > --- > > In the very first sentence of the introduction: > > "There has always been a need to serve user-agent specific contents for a > given URI --- thus highlighting the distinction..." > > you have approximated an emdash with three hyphens, which looks sort of > bad. For your convenience, this is a real emdash ô you can copy it if you > like, or else use the entity —. A useful guide to dashes and their > use in HTML is at [1]. > > -- > > I hesitate to raise this, because it's more a question of patching around > bugs than doing things right, but both Firefox and IE tend to render the > text in <code> elements too small relative to the text around them. I've > seen this discussed on the Web, but I'm not sure why the problem occurs. > Anyway, to get around this, I reluctantly made the drafts of metadata in > URI point to a stylesheet (which happens to be metadataInURI.xsl in the > same doc directory as your finding) which has in it: > > code {font-family: monospace; > font-size: 130%;} > pre {font-family: monospace; > font-size: 120%;} > > in the generated HTML <style>. I find this makes the text sizes line up > better. Others may disagree. > > --- > > Chapter 3: > > "URIs are cheap, we suggest creating as many distinctive URIs as is > meaningful." > > I believe that the correct punctuation to join the independent clauses is > a semicolon: > > "URIs are cheap; we suggest creating as many distinctive URIs as is > meaningful." > > --- > > Similar issue also in chapter 3: > > "The hyperlink structure of the Web is crucial for content discovery; When > creating..." > > One option is to leave the semicolon, I think, but then the word "When" > following the semicolon should be lowercase. Otherwise, I think you could > make the semicolon a period, and leave the "When" in caps. > > --- > > Also in that same list, the 3rd item has: > > "in the absence of user context; Or equivalently" > > There too, I think the "Or" should be lowercase. > > --- > Chapter 3, same list: > > "Contrast these findings with the metadata in URIs and state finding which > each enumerate use cases where user context Should Not be encapsulated by > URIs" > > That "Should Not" has only initial caps. I think it should either be all > caps or no caps. My vote, as signaled above, would be to make them > lowercase, as nowhere else in the finding do we use RFC 2119 directives. > If you want to make them uppercase, then obviously you should not follow > my advice above on getting rid of the RFC 2119 reference in the status and > biblio. > > --- > Chapter 4: > > "URIs are cheap, create them as needed, publish them to the Web, and > ensure that they are appropriately linked in to the rest of the Web. Thus, > each representation of interest should get it's own URI and there should > be one additional URI representing the generic resource." > > More independent clause punctuation. I think that should either be "URIs > are cheap. Create..." with a period, or else "URIs are cheap; create..." > with a semicolon. The comma seems incorrect to me. > --- > Chapter 4: > > "Thus, given one of the alternatives for a resource,ensure " > > Missing space before the word ensure. > --- > > That's it. Sorry I didn't catch more of these before. Again, I'm very > happy with the substance of the finding. Thanks for the hard work on > this. > > Noah > > [1] http://alistapart.com/stories/emen/ > > -------------------------------------- > Noah Mendelsohn > IBM Corporation > One Rogers Street > Cambridge, MA 02142 > 1-617-693-4036 > -------------------------------------- > > > > -- Best Regards, --raman Title: Research Scientist Email: raman@google.com WWW: http://emacspeak.sf.net/raman/ Google: tv+raman GTalk: raman@google.com, tv.raman.tv@gmail.com PGP: http://emacspeak.sf.net/raman/raman-almaden.asc
Received on Wednesday, 13 September 2006 23:44:06 UTC