<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://www.w3.org/Bugs/Public/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4"
          urlbase="https://www.w3.org/Bugs/Public/"
          
          maintainer="sysbot+bugzilla@w3.org"
>

    <bug>
          <bug_id>21971</bug_id>
          
          <creation_ts>2013-05-08 17:59:18 +0000</creation_ts>
          <short_desc>Paragraph elements in fieldset example are semantically incorrect.</short_desc>
          <delta_ts>2014-01-16 15:10:19 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>HTML WG</product>
          <component>HTML5 spec</component>
          <version>unspecified</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>CR</keywords>
          <priority>P3</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter>xaxiobrandish</reporter>
          <assigned_to name="Robin Berjon">robin</assigned_to>
          <cc>mike</cc>
    
    <cc>poccil</cc>
    
    <cc>public-html-admin</cc>
    
    <cc>public-html-wg-issue-tracking</cc>
    
    <cc>robin</cc>
    
    <cc>travil</cc>
          
          <qa_contact name="HTML WG Bugzilla archive list">public-html-bugzilla</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>87447</commentid>
    <comment_count>0</comment_count>
    <who name="">xaxiobrandish</who>
    <bug_when>2013-05-08 17:59:18 +0000</bug_when>
    <thetext>The paragraph elements located in the examples at the following location are semantically incorrect:

http://www.w3.org/TR/html5/forms.html#dom-fieldset-elements

As per posts on the mailing list, they serve to separate content rather than grouping it.

http://lists.w3.org/Archives/Public/public-html-comments/2013May/0012.html
http://lists.w3.org/Archives/Public/public-html-comments/2013May/0017.html

Specifically, in the three examples at the end of the section, all paragraph elements can be removed and still achieve the same semantic effect.  All of the elements in the forms are related, and separated by labels.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>87462</commentid>
    <comment_count>1</comment_count>
    <who name="Peter Occil">poccil</who>
    <bug_when>2013-05-08 20:55:54 +0000</bug_when>
    <thetext>The issue I see with that is that &quot;label&quot; is usually rendered as an inline element by default, rather than as a block element.

Accordingly, if the &quot;p&quot; elements are to be removed then I think
the specification should do one of these three things.

* Replace the &quot;p&quot; elements with another -- perhaps more appropriate --
block-level element, such as &quot;div&quot;, or if that&apos;s not acceptable,
* Include a CSS &quot;style=&apos;display:block&apos;&quot;, but I don&apos;t think that&apos;s ideal
because none of the other examples in Section 4.10 include a style
attribute; or if that&apos;s not acceptable,
* Add a note next to the examples that under CSS, label elements
are rendered as inline elements rather than as block elements by
default.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>87524</commentid>
    <comment_count>2</comment_count>
    <who name="">xaxiobrandish</who>
    <bug_when>2013-05-10 00:32:08 +0000</bug_when>
    <thetext>The first option you mention is the most acceptable, I believe.

In the first proposition, the &quot;fieldset&quot; element already groups the related field controls, as does the &quot;label&quot; element, so no more semantic meaning is necessary.  AS such, a &quot;div&quot; would be more appropriate since it adds no meaning to the content.  Because this is a rendering(presentation) issue as you state, it seems like the best option.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>87525</commentid>
    <comment_count>3</comment_count>
    <who name="">xaxiobrandish</who>
    <bug_when>2013-05-10 00:33:38 +0000</bug_when>
    <thetext>(In reply to comment #2)
&gt; The first option you mention is the most acceptable, I believe.
&gt; 
&gt; In the first proposition, the &quot;fieldset&quot; element already groups the related
&gt; field controls, as does the &quot;label&quot; element, so no more semantic meaning is
&gt; necessary.  AS such, a &quot;div&quot; would be more appropriate since it adds no
&gt; meaning to the content.  Because this is a rendering(presentation) issue as
&gt; you state, it seems like the best option.

*AS was not meant to be caps-locked.  Apologies.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>91103</commentid>
    <comment_count>4</comment_count>
    <who name="Travis Leithead [MSFT]">travil</who>
    <bug_when>2013-07-22 23:03:17 +0000</bug_when>
    <thetext>EDITOR&apos;S RESPONSE: This is an Editor&apos;s Response to your comment. If you are
satisfied with this response, please change the state of this bug to CLOSED. If
you have additional information and would like the Editor to reconsider, please
reopen this bug. If you would like to escalate the issue to the full HTML
Working Group, please add the TrackerRequest keyword to this bug, and suggest
title and text for the Tracker Issue; or you may create a Tracker Issue
yourself, if you are able to do so. For more details, see this document:


   http://dev.w3.org/html5/decision-policy/decision-policy.html

Status: Accepted
Change Description: Applied div for p + note.

Fixed in 5.1: https://github.com/w3c/html/commit/72d8faaa484bc35c3b4d208ef4a1c0b1c9b5c767

And in the latest CR editors draft:
https://github.com/w3c/html/commit/05d1f3f4f7002c0e12ce719c4acef39abfb5a85d</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>98577</commentid>
    <comment_count>5</comment_count>
    <who name="github bugzilla bot">sideshowbarker+gitzilla</who>
    <bug_when>2014-01-16 14:50:46 +0000</bug_when>
    <thetext>Commit pushed to merge-CR at https://github.com/w3c/html

https://github.com/w3c/html/commit/0f25f5a31acf5644bc9ce02781cff07644722fb7
Cherry-pick [Bug 21971]

Pulling:
https://github.com/w3c/html/commit/72d8faaa484bc35c3b4d208ef4a1c0b1c9b5c767
into the CR branch
(cherry picked from commit 05d1f3f4f7002c0e12ce719c4acef39abfb5a85d)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>98584</commentid>
    <comment_count>6</comment_count>
    <who name="github bugzilla bot">sideshowbarker+gitzilla</who>
    <bug_when>2014-01-16 15:10:19 +0000</bug_when>
    <thetext>Commits pushed to CR at https://github.com/w3c/html

https://github.com/w3c/html/commit/72d8faaa484bc35c3b4d208ef4a1c0b1c9b5c767
[Bug 21971] Paragraph elements in fieldset example are semantically incorrect

https://github.com/w3c/html/commit/0f25f5a31acf5644bc9ce02781cff07644722fb7
Cherry-pick [Bug 21971]</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>