[wp-trac] [WordPress Trac] #27609: change `<code>` to `<pre>` in wp-includes/comment-template.php

[WordPress Trac]

#27609: change `<code>` to `<pre>` in wp-includes/comment-template.php
--------------------------+------------------------------
 Reporter:  ritterml      |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  General       |     Version:  3.8.1
 Severity:  normal        |  Resolution:
 Keywords:                |     Focuses:  accessibility
--------------------------+------------------------------

Old description:

> 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>"

New description:

 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>"
 }}}

--

Comment (by SergeyBiryukov):

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

--
Ticket URL: <https://core.trac.wordpress.org/ticket/27609#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform