Opened 12 years ago
Closed 12 years ago
#23878 closed defect (bug) (fixed)
comment_form function has inconsistent spacing between HTML elements in $fields array
Reported by: | sim | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 3.6 | Priority: | normal |
Severity: | trivial | Version: | 3.0 |
Component: | Comments | Keywords: | has-patch commit |
Focuses: | Cc: |
Description
In function comment_form (wp-includes/comment-template.php) there are several default HTML strings which output the form fields for users entering comments to a post.
The first two, "author" and "email" have a space between the </label>
and the <input>
elements, whereas the following two fields "url" and "comment_field" (from $defaults) do not have spaces between these elements.
From comment-template.php
line 1524: '</label> ' .
line 1526: '</label> ' .
line 1528: '</label>' .
line 1535: '</label><textarea
Note the spaces after the label element in lines 1524 & 1526 which are lacking in lines 1528 and 1535.
Suggested fix: add a space after the label element on lines 1528 and 1535 for consistency.
In the Twenty Eleven and Twenty Twelve themes this does not manifest itself as a problem, since the label element is displayed as a block.
However, if your theme displays the label inline, it manifests itself as a difference in spacing between the first two and second two elements, which cannot be easily overcome with CSS.
While this is a trivial issue, it should be ideally fixed for consistency and to avoid the need for workarounds when comment labels are displayed inline.
I agree with the suggested fix.