Item9956: Wysiwyg editor breaks the sample color in WebPreferences

pencil
Priority: Urgent
Current State: Closed
Released In: 1.1.2
Target Release: patch
Applies To: Extension
Component: TinyMCEPlugin
Branches:
Reported By: GeorgeClark
Waiting For:
Last Change By: KennethLavrsen
The <span> used to display the color sample gets confused by the TinyMCE editor.

Quick fix is to add a <sticky block for that line.

-- GeorgeClark - 04 Nov 2010

We are working towards letting TinyMCE be as strict as it wants to be with the markup. This is moxiecode's strategy for dealing with browser quirks - as I'm sure we are all aware, the number of failure modes on different browsers (and families of browser versions) is truly staggering. Ensuring proper markup helps them reduce the number of failure modes.

That said, I do recognize that wiki authors probably expect that the markup we are having trouble with here, shouldn't requier special treatment.

Moxiecode seem reluctant to fix bugs that only affect "unsupported" configurations - and the further away from the "recommended" config we stray, the more likely we are to struggle with a buggy Foswiki integration.

MichaelTempest already did good work removing our reliance on the forced_root_block : false setting; we should also work on removing our reliance on the legacyoutput plugin, which is causing a number of bugs at the moment.

So I don't think we should turn down TinyMCE's markup cleansing.

Perhaps one option is for WysiwygPlugin to "auto-sticky" stuff that uses macros inside the tag markup...

-- PaulHarvey - 04 Nov 2010

For what it's worth, the cleansing is happening on TinyMCE startup - not save. The few config options I tried to disable the cleansing behaviour, didn't help.

-- PaulHarvey - 04 Nov 2010

The sticky solution works fine. Closing this task.

-- GeorgeClark - 04 Nov 2010

 

ItemTemplate edit

Summary Wysiwyg editor breaks the sample color in WebPreferences
ReportedBy GeorgeClark
Codebase 1.1.1, 1.1.0, trunk
SVN Range
AppliesTo Extension
Component TinyMCEPlugin
Priority Urgent
CurrentState Closed
WaitingFor
Checkins distro:155c38a24220 distro:3eaea61d8b35
TargetRelease patch
ReleasedIn 1.1.2
Topic revision: r6 - 10 Nov 2010, KennethLavrsen
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License    Legal Imprint    Privacy Policy