Prevent enabling a text editor if a text format has one or more FILTER_TYPE_MARKUP_LANGUAGE filters enabled

Created on 5 March 2013, almost 12 years ago
Updated 16 January 2023, almost 2 years ago

Problem/Motivation

Nonsensical situations that are currently not prevented by the UI:

  1. Enabling CKEditor for a Markdown-powered text format.
  2. Enabling CKEditor while the filter_autop (or filter_url) filter is enabled.

(If it doesn't make sense to you that using filter_autop and filter_url precludes you from using CKEditor, then please read #1911884: Enable CKEditor in the Standard install profile β†’ and πŸ“Œ Prevent enabling a text editor if a text format has one or more FILTER_TYPE_MARKUP_LANGUAGE filters enabled Closed: works as designed FIRST.)

Proposed resolution

As of #1782838: WYSIWYG in core: round one β€” filter types β†’ , filters can have types. filter_autop and filter_url are marked as FILTER_TYPE_MARKUP_LANGUAGE, because they are in effect extremely minimal markup languages.

Whenever >=1 FILTER_TYPE_MARKUP_LANGUAGE-type filter is enabled, we should disallow the enabling of a text editor.

Remaining tasks

  1. Get feedback. Figure out how to deal with edge cases, like http://oscargodson.github.com/EpicEditor/ β€” a Markdown "WYSIWYG" editor.
  2. Write the code. Should probably use Drupal core's dialog and/or #states.

User interface changes

TBD

API changes

None.

πŸ“Œ Task
Status

Closed: works as designed

Version

10.1 ✨

Component
EditorΒ  β†’

Last updated 9 days ago

Created by

πŸ‡§πŸ‡ͺBelgium wim leers Ghent πŸ‡§πŸ‡ͺπŸ‡ͺπŸ‡Ί

Live updates comments and jobs are added and updated live.
  • wysiwyg

    Involves an in-browser What-You-See-Is-What-You-Get content editor.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

No activities found.

Production build 0.71.5 2024