change `<code>` to `<pre>` in wp-includes/comment-template.php

[ritterml]

Change <code> to <pre> in "allowed tags" on line 2002 to avoid "nowrap" errors in Bootstrap. <code> block expands beyond container block when white-space:nowrap is applied.

Notes:

• <code> is for in-line, not block-level
• these tags should be in an unordered list

Considered adding a foreach loop to wrap each allowed_tags() in a <li> and <code>, which is more semantic.

    foreach(allowed_tags() as $t)
    return "<li><code>$t</code></li>"

[ritterml]

Now that I think of it, the foreach loop I mentioned should probably be wrapped into allowed_tags() to keep it out of the View.

[SergeyBiryukov]

Rather than focus on the solution, could you describe what the issue is, and how to reproduce it?

[DrewAPicture]

@ritterml: Care to followup here? See comment:2

[joedolson]

This is somewhat interesting, but it would have significant breakages in themes, and I don't see a lot of value in catering specifically to Bootstrap issues. There are no accessibility concerns on this that aren't 100% controllable in themes (avoiding overflow of containers), so I recommend closing this issue.

I'm removing the accessibility focus, at any rate.

[johnbillion]

Please re-open if there is a strong case for this.