Update tooling and instructions for ER and RDD #315
Conversation
palemieux
left a comment
palemieux
left a comment
There was a problem hiding this comment.
Tests should be added as well.
|
Can you create a sample RDD document under |
doc/main.html
Outdated
| <p>The <code>id</code> attribute shall be present on the <code>dl</code> element and equal to | ||
| <code>rdd-proponent</code>. | ||
|
|
||
| <p>If present, the <code>dl</code> element, as described in <a href="#sec-dl-element"></a>, shall use a single <code>dt</code> element for each company name designated as a Proponent of the document, and multiple lines of <code>dd</code> as needed for contact information for each company listed. The contact infomation should at least include an address and email, reach out to SMPTE HO for further deatails. </p> |
There was a problem hiding this comment.
Isn't a single
There was a problem hiding this comment.
... in fact, I am not convinced <dl> is the right approach if there are multiple proponents... can you provide an example of how this would work?
There was a problem hiding this comment.
<dl id="element-proponent">
<dt>Company name</dt>
<dd>Contact name</dd>
<dd>Contact email</dd>
<dd>Company Address</dd>
<dd>Company website</dd>
<dt>Company 2 name</dt>
<dd>Contact name</dd>
<dd>Contact email</dd>
<dd>Company 2 Address</dd>
<dd>Company 2 website</dd>
</dl>
Nothing prevents the usage of multiple dd for each dt, such as https://developer.mozilla.org/en-US/docs/Web/HTML/Element/dl#single_term_multiple_descriptions. This is frankly the least ugly way to present, besides trying to to the same with ul and then wrapping a style in CSS.
There was a problem hiding this comment.
I added the above example to the documentation as well.
Updated instructions, updated Foreword (and streamlined logic), added clause for contact info, updated validation.