Problem/Motivation

Often contrib modules need a way to indicate a settings textarea should never be WYSIWYG or have HTML code at all. This is very difficult to accomplish with BUEditor.

Proposed resolution

The de facto solution, implemented by the WYSIWYG module, is to support a #wysiwyg attribute on FAPI elements. If a textarea has #wysiwyg set to FALSE, then all editors should be suppressed on that element.

API changes

A simple check for #wysiwyg attribute in the _bueditor_textarea() function can bypass the editor for a specific field.

Original report by thePanz

Hi, will BUEditor support upcoming FAPI #wysiwyg setting?
Regards

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

thePanz’s picture

Component: User interface » Code
Status: Active » Reviewed & tested by the community
FileSize
691 bytes

This patch implements this features...
I'm still asking myself if this module is still maintained anymore...

thePanz’s picture

Re-Rolled against latest -dev (February) release.

ufku’s picture

Status: Reviewed & tested by the community » Closed (won't fix)

this won't go in because;
1- For 6.x there is no such api. It may be for 7.x
2- BUEditor is not a wysiwyg editor. There are usecases for plain-text fields where for example some characters not available on user's keyboard or some predefined phrases need to be inserted.

thePanz’s picture

Status: Closed (won't fix) » Postponed

Hi, thank you for your reply.

1. I know that a some modules still use this feature in D6 (I don't remember if FCK or other editors still do that), so why don't do BUEditor?

2. You're right, so maybe BUEditor (for predefined phrases) will have to filter available buttons? I mean, inserting an IMG tag where the textarea is only for plain text isn't right..

Regards

Dave Reid’s picture

Issue tags: +#wysiwyg

@ufku: I'd really like you to re-consider this. A *LOT* of contrib modules are already using this as an easy way to indicate a settings textarea should never be WYSIWYG or have HTML code at all.

a_c_m’s picture

Seems work is already ongoing : #429684: Add editor: BUEditor

ufku’s picture

Status: Postponed » Closed (won't fix)

Closing 6.x-1.x issues as this branch is no more supported.

jstoller’s picture

Version: 6.x-1.x-dev » 6.x-2.x-dev
Status: Closed (won't fix) » Needs review
FileSize
558 bytes

This had me pulling my hair out for days. I'm bumping the request to version 6.x-2.x-dev and providing a patch for your consideration.

jstoller’s picture

Title: Support for incoming #wysiwyg FAPI » Support #wysiwyg FAPI attribute

I updated the issue summery and title for clarity.

jstoller’s picture

Issue summary: View changes

Update issue summery with template.