Opened 16 years ago

Closed 13 years ago

#2522 closed Bug (invalid)

Bug report: no empty value on firefox

Reported by: Almog Owned by:
Priority: Normal Milestone:
Component: Core : Output Data Version: FCKeditor 2.6.4
Keywords: HasPatch Cc:

Description

I used on the last ver of the fckeditor(Night build), and when i set the value to ""[empty] on firefox the value become to "<br>".

This bug makes some problems, start by the empty checking(& I have to "igonore" it by php) and continue to char counting.

I hope to find an any solution.

Best regards, Almog Baku,

Israel

Attachments (1)

2522.patch (546 bytes) - added by Noah Botimer 16 years ago.
Patch to handle dangling BR tags

Download all attachments as: .zip

Change History (5)

comment:1 Changed 16 years ago by Alfonso Martínez de Lizarrondo

Keywords: break line <br? removed
Priority: HighNormal

comment:2 Changed 16 years ago by Artur Formella

Keywords: Pending WorksForMe added
Summary: Bug report: NO EMPTY VALUE ON FIREFOXBug report: no empty value on firefox

Could you try reproduce it with current Night build? How you set an empty value?

Changed 16 years ago by Noah Botimer

Attachment: 2522.patch added

Patch to handle dangling BR tags

comment:3 Changed 16 years ago by Noah Botimer

Keywords: Confirmed HasPatch added; Pending WorksForMe removed
Version: FCKeditor 2.6.4

Attached is a patch to FCKXHtml._AppendNode(). It ensures that, when EnterMode = br, the padding node is not returned by FCK.GetXHTML(). This fixes two specific symptoms:

  1. Editors with no content in WYSIWYG mode are submitted with a single BR tag
  2. FCK.GetXHTML() is dirtying the content by tracking the padding node improperly

This is also related to ticket #1150. The attached patch does not resolve the issue where Select All followed by a backspace while EnterMode = br leaves a dangling BR that is not the padding node.

This scenario can be exercised with Sample 12. Steps to test:

  1. Select EnterMode = br.
  2. Select Source mode
  3. Remove all content
  4. Return to WYSIWYG mode
  5. Click Submit

With this patch, there should be no submitted content on the result page. Without it, there will be a stray <br />, which has been stripped of the bogus node attributes (as a result of #1150 / [2219]).

comment:4 Changed 13 years ago by Jakub Ś

Resolution: invalid
Status: confirmedclosed

FCKeditor was retired and is no longer supported. All active development was moved to its successor, CKEditor 3.x, that is a fully mature and far superior product. We recommend you upgrade as soon as possible.

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy