Ticket #1613 (closed Bug: fixed)

Opened 19 months ago

Last modified 17 months ago

Extra div wrapping the editor instance

Reported by: alfonsoml Owned by: alfonsoml
Priority: Normal Milestone: FCKeditor 2.6
Component: General Version:
Keywords: Review+ Cc:

Description

This is not about the content generated by the editor, but just the HTML that does in fact becomes the editor. The call to FCKeditor.prototype.CreateHtml returns a <div> that wraps some hidden inputs and a iframe or textarea.

From my point of view that div isn't necessary so it could be removed as it makes a little harder to get 100% height in standards mode and doesn't provide anything useful.

Attachments

1613.patch (7.0 KB) - added by alfonsoml 17 months ago.
Proposed SVN patch

Change History

Changed 19 months ago by dustball

Comment - this makes sense. I cannot think of any reason to have it.

After all, people can easily wrap the output of CreateHtml with their own DIV, if they really want it.

Changed 17 months ago by alfonsoml

  • owner set to alfonsoml
  • status changed from new to assigned

Changed 17 months ago by alfonsoml

Proposed SVN patch

Changed 17 months ago by alfonsoml

  • keywords Review? added

The patch does remove the div generated with the different integration methods.

Changed 17 months ago by fredck

  • keywords Review+ added; Review? removed
  • milestone set to FCKeditor 2.6

I'm sure this things had some sense in the past. I recall we were used to not have that <div>, but for some reason it has been introduced, in the early beginnings (so, prone to mistakes).

Unfortunately we didn't have SVN to help us explaining anything about it. It makes no sense to me too, so let's drop it.

Please include the changelog entry for it.

Changed 17 months ago by alfonsoml

  • status changed from assigned to closed
  • resolution set to fixed

Fixed with [1503]

Note: See TracTickets for help on using tickets.