[wp-trac] [WordPress Trac] #62361: Set filter "activate_tinymce_for_media_description" to "true" is breaking meadia_descripton by running it through "htmlspecialchars()"
WordPress Trac
noreply at wordpress.org
Wed Nov 13 21:52:32 UTC 2024
#62361: Set filter "activate_tinymce_for_media_description" to "true" is breaking
meadia_descripton by running it through "htmlspecialchars()"
-----------------------------------------+------------------------
Reporter: dagobert24 | Owner: joedolson
Type: defect (bug) | Status: accepted
Priority: normal | Milestone: 6.8
Component: Media | Version: 6.6.2
Severity: normal | Resolution:
Keywords: has-patch 2nd-opinion close | Focuses: ui
-----------------------------------------+------------------------
Changes (by azaozz):
* keywords: has-patch => has-patch 2nd-opinion close
Comment:
Frankly I think that the change from #60158 should be reverted. It doesn't
seem a good idea to add support for TinyMCE to the content of attachment
pages just to have paragraph tags. Seems that missing paragraph tags there
should be treated as a bug when using the (so called) Text editor,
`quicktags`. I think this used to work before.
There are few reasons to remove this:
- TinyMCE in WordPress is in "deep maintenance" mode meaning it is not
supported any more except for eventual security fixes.
- The attachment pages are not used/shown any more. They are disabled for
by default new(er) installs.
Also the fact that this bug is being reported several months after this
functionality was introduced seems to signify there is very little
interest.
Thinking this type of functionality is best left for plugins that will
also ensure attachment pages are used in the theme/at the front-end.
Without that this is useless.
Imho instead of fixing this it seems better to revert [58372], make a
follow-up ticket for #60158 and look at fixing the Text editor to create
paragraphs.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/62361#comment:7>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list