WordPress.org

Make WordPress Core

Opened 3 years ago

Last modified 2 years ago

#37420 new enhancement

Enable TinyMCE for attachment descriptions

Reported by: SergeyBiryukov Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Media Keywords: 2nd-opinion
Focuses: Cc:
PR Number:

Description

Enabling TinyMCE for attachment descriptions was briefly mentioned in #24472, but it seems like it was never really discussed as a default option before.

[23083] introduced the current text editor with quick tags, before that it was just a simple textarea.

[28944] introduced the wp_editor_settings filter, so it should be possible to override 'tinymce' => false on the attachment editor instance.

Any reasons not to enable it by default?

Change History (3)

This ticket was mentioned in Slack in #core-media by desrosj. View the logs.


2 years ago

#2 follow-up: @desrosj
2 years ago

@SergeyBiryukov did you have any use cases in mind?

Talked this over in the media bug scrub today with @joemcgill. Are there any locations where the description is being sanitized and would cause something to break when HTML is included?

It is also a poor user experience that the attachment edit screen supports this (https://cloudup.com/c9dhHFtzLtl+) but the modal does not (https://cloudup.com/cT4IGkRXoEm+).

#3 in reply to: ↑ 2 @SergeyBiryukov
2 years ago

Replying to desrosj:

@SergeyBiryukov did you have any use cases in mind?

Must be a project I worked on or a support topic I answered at the time, but I could not find any additional details.

Talked this over in the media bug scrub today with @joemcgill. Are there any locations where the description is being sanitized and would cause something to break when HTML is included?

The field already has an HTML editor since [23083], it's just that quicktags.js is not as convenient as TinyMCE :)

Note: See TracTickets for help on using tickets.