[wp-meta] [Making WordPress.org] #20: Possible for user's to break layout when adding forum posts

Making WordPress.org noreply at wordpress.org
Thu Jan 30 05:48:49 UTC 2020


#20: Possible for user's to break layout when adding forum posts
-------------------------------------+-----------------------------
 Reporter:  siobhyb                  |       Owner:  SergeyBiryukov
     Type:  defect                   |      Status:  reopened
 Priority:  high                     |   Milestone:
Component:  Support Forums           |  Resolution:
 Keywords:  has-patch needs-testing  |
-------------------------------------+-----------------------------
Changes (by dd32):

 * status:  closed => reopened
 * resolution:  fixed =>


Comment:

 Replying to [comment:16 SergeyBiryukov]:
 > In [changeset:"9438" 9438]:
 > {{{
 > #!CommitTicketReference repository="" revision="9438"
 > Support Theme: Prevent standalone `<li>` tags from breaking the theme
 layout.
 >
 > If a `<li>` tag is not preceded by `<ul>` or `<ol>`, prepend it with
 `<ul>` and let `force_balance_tags()` do the rest.
 >
 > Fixes #20. See #bb2357.
 > }}}

 FWIW This seems to not work 100% with lists that have multiple items. I
 think the regex needs to either support a closing LI element preceding it
 or something like that.

 This is the best example I could come up with:
 https://wordpress.org/support/topic/this-is-a-test-topic/#post-12379821
 The three items should be part of the OL, but instead the second and third
 LI gets assigned to their own ULs (Which is probably better than the
 original problem)

-- 
Ticket URL: <https://meta.trac.wordpress.org/ticket/20#comment:17>
Making WordPress.org <https://meta.trac.wordpress.org/>
Making WordPress.org


More information about the wp-meta mailing list