247064 - Editor: MIDAS - inserted HTML gets munged very badly

Status: UNCONFIRMED

(Core :: DOM: Editor, defect, P5)

Description

[osavill]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8a2) Gecko/20040610
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8a2) Gecko/20040610

When a component has Design Mode 'on' it is possible to paste HTML directly into
it. (See http://www.mozilla.org/editor/midasdemo/ and check the 'View HTML
Source' button.) However, when the 'View HTML Source' button is unchecked and
the WYSIWYG view is returned the newly pasted HTML is apparently parsed and
modified. The most striking thing is the number of  s ! Moreover the page
will no longer render as expected.

IE also modifies the HTML but the structure remains intact.

I will attach a before and after example.

Reproducible: Always
Steps to Reproduce:
1.
2.
3.




Clients are starting to look at MIDAS and view it as the 'best thing since
sliced bread'. At the moment both IE and Mozilla have some major problems. Sadly
IE just has the edge, but Mozilla is far better in many areas. Is any one
working on this ? Not a single report seems to have been addressed yet, which is
sad, but once again Mozilla / Fire Fox could so easily steal a lead over IE and
leave it standing in what could become quite important in CMS.

Comment 1

[osavill]

Okay, I see what is going on here. ALL white space, where ever it occurs, is
being replaced with  s. So, for e.g.

  <table>
   <tbody>
    <tr>
     <td>
      fred
     </td>
    </tr>
   </tbody>
  </table>

becomes:

&nbsp;&nbsp;<table>&nbsp;&nbsp;&nbsp;<tbody>&nbsp;&nbsp;&nbsp;&nbsp;<tr>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<td>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;fred&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</td>&nbsp;&nbsp;&nbsp;&nbsp;</tr>&nbsp;&nbsp;&nbsp;</tbody>&nbsp;&nbsp;</table>
    

which, to quote the prophet, is complete pants !!!

But replacing it with:

<table><tbody><tr><td>fred</td></tr></tbody></table>

works just fine.

I'll not now attach the HTML before and afters unless requested to do so.

I really, really, really want to see this part of Mozilla reach maturity :-)

Comment 2

[osavill]

Attachment: The original HTML snippet

Sorry about this, it is a little more complicated. The very simple example I
gave above does in fact work. The white space is simply removed. I will attach
three files to show the original, the corrupted and the one that enables MIDAS
to work correctly.

Comment 3

[osavill]

Attachment: The MIDAS corrupted result

Comment 4

[osavill]

Attachment: Removed all white space by hand before pasting in and MIDAS woks just fine

Comment 5

[Gervase Markham [:gerv]]

This is an automated message, with ID "auto-resolve01".

This bug has had no comments for a long time. Statistically, we have found that
bug reports that have not been confirmed by a second user after three months are
highly unlikely to be the source of a fix to the code.

While your input is very important to us, our resources are limited and so we
are asking for your help in focussing our efforts. If you can still reproduce
this problem in the latest version of the product (see below for how to obtain a
copy) or, for feature requests, if it's not present in the latest version and
you still believe we should implement it, please visit the URL of this bug
(given at the top of this mail) and add a comment to that effect, giving more
reproduction information if you have it.

If it is not a problem any longer, you need take no action. If this bug is not
changed in any way in the next two weeks, it will be automatically resolved.
Thank you for your help in this matter.

The latest beta releases can be obtained from:
Firefox:     http://www.mozilla.org/projects/firefox/
Thunderbird: http://www.mozilla.org/products/thunderbird/releases/1.5beta1.html
Seamonkey:   http://www.mozilla.org/projects/seamonkey/

Comment 6

[osavill]

This problems is still there. 

Comment 7

[john.talarico]

This is still very much an issue.

Using the following code, you actually get invalid conversion:

<pre>
 test1
 test2
 test3
</pre>

The linefeeds get converted to &nbsp; and the rendered text becomes:

<pre>&nbsp;test1&nbsp;test2&nbsp;test3</pre>

Comment 8

[Jens Stutte [:jstutte]]

Bulk-downgrade of unassigned, >=5 years untouched DOM/Storage bugs' priority.

If you have reason to believe this is wrong (especially for the severity), please write a comment and ni :jstutte.