Item9409: legacyoutput breaks WYSIWYG paragraph outdenting

pencil
Priority: Normal
Current State: No Action Required
Released In: n/a
Target Release: minor
Applies To: Extension
Component: TinyMCEPlugin
Branches:
Reported By: KennethLavrsen
Waiting For:
Last Change By: CrawfordCurrie
In trunk try and edit a simple page in Wysiwyg

Note that the indent button is still active when you have the cursor on a line with normal text.

You can indent the text. But the indent is not saved. Well, we do not really have a way to indent in TML so the correct way would be that the button was not enabled at all.

The unindent button is not active. And even when you indent normal text the unindent remains inactive.

When you add a bullet the indent and unindent are active and seem to work correctly.

So the bug I see is that the indent button should be disabled except when you are on a bullet.

-- KennethLavrsen - 29 Jul 2010

This is a normal feature of the editor, apparent on TinyMCE 3.2.7 that shipped with Foswiki 1.0.9 as well. I am reluctant to override core functionality of TinyMCE editor unless we have a good reason.

SupportBlockquoteAndIndenting is ready to be implemented.

I would like to implement post-Foswiki 1.1 (1.2 "editors special" as Crawford said).

I am reluctant to add even more button state logic to our foswikibuttons plugin because it is sometimes slow on IE6 to move the cursor around in large documents (in my VM at least).

Kenneth, if this is a new bug then I will give it some time.

Otherwise, please let's change to enhancement and target for Foswiki 1.2 where we can add the TML syntax to support this feature instead of cripple it

-- PaulHarvey - 30 Jul 2010

In 1.0.9 the bug is also there. It behaves a little differently. In 1.0.9 the un-undent is not disabled. So the button logic has changed a little bit.

But in both 1.0.9 and 1.1.0 the indent is seen in Wysiwyg but when you save it is ignored and the text is saved as normal text.

I am OK to lower this to Normal and wait for the enhancement that will add indented text.

The visible buttons is a bug and not an enhancement. But if we fix the bug by adding the indentation enhancement then it makes sense to live with the bug for another release round. How come the unindent is now disabled? Did you add this?

-- KennethLavrsen - 30 Jul 2010

Seems the outdent/indent feature works properly on the public moxiecode demo, so it must be something wrong with our integration. I will look into it. Edit: seems it's due to legacyoutput plugin. This is needed so that TinyMCE will understand non-XHTML-strict elements such as <u>, <strike>, <b> etc.

A major feature of TinyMCE 3.3 is a re-write of the way it handles detection, and application of custom formats.

You will notice that the Foswiki formats select-box should be much more reliable across the different browsers. It should reliably reflect that you are in a heading format, verbatim block, protected, etc. This was hard to get right in 3.2.7 on Foswiki 1.0.9

Re-titled the bug. I will see if I can get a patch to moxiecode, I've been swimming around the formatter engine stuff recently developing TinyMCECleanStickyBitsPlugin

-- PaulHarvey - 30 Jul 2010

Tasks.Item2515 should be used to get WysiwygPlugin ready to produce XHTML-strict elements so that we can stop shipping with legacyoutput enabled

-- PaulHarvey - 30 Jul 2010

Tasks.Item2516 for new TML syntax for indented paragraphs

-- PaulHarvey - 30 Jul 2010

Filed 3037077 with TinyMCE project

-- PaulHarvey - 30 Jul 2010

TinyMCE has a new bug tracker: http://www.tinymce.com/develop/bugtracker_view.php?id=3075

Marking this deferred for Foswiki 1.2.1

-- GeorgeClark - 15 Mar 2015

Cannot reproduce the original report in 4.5.3, so closing.

-- Main.CrawfordCurrie - 24 Mar 2017 - 15:52
 

ItemTemplate edit

Summary legacyoutput breaks WYSIWYG paragraph outdenting
ReportedBy KennethLavrsen
Codebase trunk
SVN Range
AppliesTo Extension
Component TinyMCEPlugin
Priority Normal
CurrentState No Action Required
WaitingFor
Checkins
TargetRelease minor
ReleasedIn n/a
CheckinsOnBranches
trunkCheckins
masterCheckins
ItemBranchCheckins
Release02x01Checkins
Release02x00Checkins
Release01x01Checkins
Topic revision: r9 - 24 Mar 2017, CrawfordCurrie
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