[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