Skip to content
This repository has been archived by the owner on Jul 30, 2019. It is now read-only.

Update semantics-forms.include #834

Merged
merged 3 commits into from
Mar 29, 2017
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Update semantics-forms.include
removed code causing layout issues in example
stevefaulkner authored Mar 28, 2017
commit 1f92e03d956c463616b509ba71b2145104912a36
2 changes: 1 addition & 1 deletion sections/semantics-forms.include
Original file line number Diff line number Diff line change
@@ -1209,7 +1209,7 @@
<pre highlight="html">
&lt;form&gt;
&lt;table&gt;
&lt;caption&gt;Example &lt;<code>label</code>:gt;'s <code>for</code> attribute with &lt;<code>label</code>&gt;&lt;/caption&gt;
&lt;caption&gt;Example &lt;label:gt;'s for attribute with &lt;label&gt;&lt;/caption&gt;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There was a typo on that line as well as the <code> tags... So the closing angle backet for the 1st label currently says :gt; and this should be replaced with &gt; (just before 's)

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be nice to have a colon after example, and "with label" seems totally redundant.

(Yes, this is nitpicking editorial comment…)

&lt;tr&gt;
&lt;th&gt;&lt;label for="name"&gt;Customer name: &lt;/label&gt;&lt;/th&gt;
&lt;td&gt;&lt;input name="name" id="name"&gt;&lt;/td&gt;