<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#ffffff">
That wouldn't be a valid complaint from a theme dev anyway. Only
users without the 'unfiltered_html' cap are limited to those HTML
tags. Any users with the 'unfiltered_html' cap can add any HTML
they want. Otherwise, we'd never get those tags to work in the
theme unit test data.<br>
<br>
On 5/3/2011 9:49 PM, Philip Walton wrote:
<blockquote cite="mid:4DC0BEC3.6080903@philipwalton.com" type="cite">
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
I could see a theme developer complaining that lists aren't even
supposed to be in comments based on the following:<br>
<br>
"You may use these HTML tags and attributes: <a href=""
title=""> <abbr title=""> <acronym title="">
<b> <blockquote cite=""> <cite> <code>
<del datetime=""> <em> <i> <q cite="">
<strike> <strong>"<br>
<br>
Obviously most themes don't actually strip out all but those tags,
but I'm still curious to hear how you'd respond to a comment like
that on a trac ticket (hypothetically).<br>
<br>
<br>
On 5/3/11 7:35 PM, Chip Bennett wrote:
<blockquote
cite="mid:BANLkTimVEvwXxjfJpHJsp1heVuQ1t9Snog@mail.gmail.com"
type="cite">The li.comment children lists and list items are one
of the primary reasons to check all HTML tags within comments.
All too often, the developer will either not properly specify
styles for li.comment, or else will overlook resetting the LI
styling.
<div> <br>
</div>
<div>For everything else, as with the Layout Test post, HTML tag
styling should be appropriate within the design intent. Some
things are fairly discrete (EM, BIG, STRONG, SUB, SUP, etc.),
but others are much more subjective.</div>
<div><br>
</div>
<div>But yes: comments *do* need to display HTML tags correctly.</div>
<div><br>
</div>
<div>Chip<br>
<br>
<div class="gmail_quote">On Tue, May 3, 2011 at 8:44 PM,
Philip Walton <span dir="ltr"><<a moz-do-not-send="true"
href="mailto:philip@philipwalton.com">philip@philipwalton.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt
0.8ex; border-left: 1px solid rgb(204, 204, 204);
padding-left: 1ex;">In the Theme Test Unit data, there is
a comment with the same markup as page 3 of the "Layout
Test".<br>
<br>
I've run across a number of themes that don't include the
styles necessary to have the comments display the same way
as the layout test does. One really common problem is the
lists often have borders or padding around them because
the comments themselves are list items.<br>
<br>
How much should comment styling really be taken into
consideration? Has it been decided that poor styling of
complex markup with a comment is grounds for rejection?<br>
_______________________________________________<br>
theme-reviewers mailing list<br>
<a moz-do-not-send="true"
href="mailto:theme-reviewers@lists.wordpress.org"
target="_blank">theme-reviewers@lists.wordpress.org</a><br>
<a moz-do-not-send="true"
href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers"
target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
</blockquote>
</div>
<br>
</div>
<pre wrap=""><fieldset class="mimeAttachmentHeader"></fieldset>
_______________________________________________
theme-reviewers mailing list
<a moz-do-not-send="true" class="moz-txt-link-abbreviated" href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a>
<a moz-do-not-send="true" class="moz-txt-link-freetext" href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a>
</pre>
</blockquote>
<pre wrap="">
<fieldset class="mimeAttachmentHeader"></fieldset>
_______________________________________________
theme-reviewers mailing list
<a class="moz-txt-link-abbreviated" href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a>
<a class="moz-txt-link-freetext" href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a>
</pre>
</blockquote>
</body>
</html>