Re: [w3c/browser-payment-api] editorial: attr/method intros are notes (closes #379) (#438)

marcoscaceres commented on this pull request.



> @@ -787,28 +790,42 @@
         <h2>
           <dfn>shippingAddress</dfn> attribute
         </h2>
-        <p>
-          <a>shippingAddress</a> is populated when the user provides a shipping
-          address. It is null by default. When a user provides a shipping
-          address, the <a>shipping address changed algorithm</a> runs.
+        <p class="note">

> Hmm. I am less sure about the attributes than about the methods.

Agree. Removed "note".  Making statements of fact with non-RFC2119 words seems good (even if they mostly benefit developers). 

> Oh, also note that PaymentResponse currently uses a completely different style for its attributes, instead of having a heading per attribute :-/.

We should switch those to be headings (separate bug) - personally, I find it frustrating when attributes are not in the ToC - one should be able to see/access the API shape there too.  

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/w3c/browser-payment-api/pull/438#discussion_r103142058

Received on Monday, 27 February 2017 07:30:26 UTC