Foswiki on GitHub is open for business! Next release meeting: Monday September 1, 1300Z

Item11337: Delete of a Heading 1 in WYSIWYG mangles the headings

Priority: CurrentState: AppliesTo: Component: WaitingFor:
Normal Closed Extension TinyMCEPlugin  

Heading delete bug in the WYSIWYG.

How to reproduce:
  • Make an empty topic.
  • Make heading1, followed by heading2 and normal. WIKI text will be like this:

---+ Heading 1
---++ Heading 2

normal

WYSIWYG will show

screenshot_01.jpg

Now, delete the Heading1, so:
  • select the heading1 line (triple click) and
  • press backspace
screenshot_02.jpg

screenshot_02.jpg

You will get this:

screenshot_03.jpg

As you can see:
  • the "visual representation" is OK - it is like heading2
  • but the STYLE showing "Heading 1".
The biggest surprise is after showing the code:
---+ <span style="color: #339900; font-size: 21px; line-height: 23px;">Heading 2</span>

normal

whereas should be:
---++ Heading 2

normal

-- JozefMojzis - 07 Dec 2011

Thank you for this most excellent bug report!

This is fixed in trunk. I'll work on releasing a new TinyMCEPlugin after Foswiki 1.1.4 is out.

-- PaulHarvey - 08 Dec 2011

This is NOT fixed is 1.1.5 - so, need reopen.

-- JozefMojzis - 03 May 2012

More precisely: in the Firefox - OK, Safari, Chrome - NOT.

-- JozefMojzis - 03 May 2012

There are two issues here. First - fixed in 1.1.5, is the bogus span, and rendering the heading 1 with heading 2 markup. The 1 vs. 2 is a bit difficult to notice - just a color change, but with 1.1.5 it all seems correct.

With 1.1.5 (and trunk / TMCE 3.4.9 too), Deleting the Heading 1 line causes the Heading 2 to take on the Heading 1 attributes. It is all done consistently and correctly, <h1> Heading 2 </h1>, and the saved TML is a ---+ heading 1.

So this issue falls pretty clearly into the TMCE space now.

-- GeorgeClark - 03 May 2012
 

ItemTemplate edit

Summary Delete of a Heading 1 in WYSIWYG mangles the headings
ReportedBy JozefMojzis
Codebase 1.1.4 RC1, 1.1.4 beta2, 1.1.4 beta1, 1.1.3, 1.1.3 RC1, 1.1.3 beta1
SVN Range
AppliesTo Extension
Component TinyMCEPlugin
Priority Normal
CurrentState Closed
WaitingFor
Checkins
TargetRelease patch
ReleasedIn 1.1.5
CheckinsOnBranches
trunkCheckins
Release01x01Checkins
Topic attachments
I Attachment Action Size Date Who Comment
screenshot_01.jpgjpg screenshot_01.jpg manage 29.5 K 07 Dec 2011 - 11:11 JozefMojzis  
screenshot_02.jpgjpg screenshot_02.jpg manage 37.6 K 07 Dec 2011 - 11:11 JozefMojzis  
screenshot_03.jpgjpg screenshot_03.jpg manage 26.2 K 07 Dec 2011 - 11:11 JozefMojzis  
Topic revision: r6 - 03 May 2012, GeorgeClark
 
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. see CopyrightStatement. Creative Commons License