Supplying a string like this: <head><title>My Page</title></head> to CreateContextualFragment yields a nsDOMDocumentFragment with a nsHTMLTitleElement child node, but no textnode for the title contents. In Composer, use View | HTML Source, then add a title tag and contents in the source editor. Switch back to "Normal Edit Mode" and break in nsHTMLEditor::ReplaceHeadContentsWithHTML(), nsHTMLEditor.cpp, to debug. I'm no sure about component or who to assign to, but vidur knows about the issue and will redirect.
BTY: This does not block Composer's HTML Source editing from working, so this can be "futured".
In previous discussions with Vidur and RigkG, they seemed to know exactly what causes this problem and how to fix it. I have discovered another situation which *does* impact greatly on composer: all <style> .... </style> and <script> ... </script> content in a Composer page will be lost after using HTML Source editing. This is not good!
Keywords: dataloss, nsbeta3
Re-assigning to Johnny. Marking nsbeta3-. Nominating for rtm because not fixing this is going to result in users lising script and style content in the head element when they edit it.
Assignee: vidur → jst
Mass update of qa contact
QA Contact: gerardok → janc
Priority: P3 → P1
Whiteboard: [nsbeta3-][egk] → [egk]
Harish and Vidur are gonna work on this together. marking nsbeta3+
Assignee: jst → harishd
Whiteboard: [egk] → [nsbeta3+]
PDT: Either we get this fixed or consider removing html editing, the data corruption is too severe. Agree, P1. Not a ship stopper for PR3, though, we CAN release note this.
Whiteboard: [nsbeta3+] → [nsbeta3+][PDTP1]
I'm pretty close to a fix for this ( may be a day more ).
Status: NEW → ASSIGNED
Whiteboard: [nsbeta3+][PDTP1] → [nsbeta3+][PDTP1][Fix in hand]
Created attachment 15218 [details] [diff] [review] patch version 1.2 ( includes support for textarea )
Whiteboard: [nsbeta3+][PDTP1][Fix in hand] → [nsbeta3+][PDTP1][Fix in hand][ETA 09/22/00 ]
I've tested the patch and it works great. The only problem I see is formatting of DOM source into HTML source (doen's break after <meta...> like it should, and extra linefeeds are inserted inside a <style> ... </style> tag. but that's a different bug.
I saw the extra linefeeds too...but I'm almost sure that it has got nothing to do with my change.
Patch version 1.3 has been r=jst & a=waterson. PDT, could you please ++ this bug?
Whiteboard: [nsbeta3+][PDTP1][Fix in hand][ETA 09/22/00 ] → [nsbeta3+][PDTP1][Fix in hand]
Marking relnote and rtm+. Harish will check in the fix to the trunk. We will wait until after PR3 ship to check this fix into the branch.
Whiteboard: [nsbeta3+][PDTP1][Fix in hand] → [nsbeta3+][PDTP1][Fix in hand][rtm+]
Marking nsbeta3- to get this off the beta 3 radar.
Whiteboard: [nsbeta3+][PDTP1][Fix in hand][rtm+] → [nsbeta3-][PDTP1][Fix in hand][rtm+]
Changes are in the trunk.
Why don't we want to checkin to the nsbeta branch now so this gets some testing before rtm?
Because the branch hasn't been tagged for PR3 yet. We don't want this fix to go into beta 3, only rtm.
PDT marking [rtm++]
Whiteboard: [nsbeta3-][PDTP1][Fix in hand][rtm+] → [nsbeta3-][PDTP1][Fix in hand][rtm++]
Fix is on the TRUNK too. Marking FIXED.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
*** Bug 49873 has been marked as a duplicate of this bug. ***
verified: 2000-11-08-01-MN6: Win32
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.