207531 - Allow editor to edit xhtml files (as html)

Status: RESOLVED FIXED

(Core :: DOM: Editor, defect)

Description

[timeless]

This is not a panacea, it won't switch composer into an xml editing mode, nor
will it prevent composer from using htmlisms, nor will it enable composer to
save as xml/xhtml.

all i want is to be able to load an xhtml file, edit it (as html), and save as
html (or text). and that's all my patch does.

Comment 1

[timeless]

Attachment: nothing fancy

Comment 2

[Simon Fraser [no longer active]]

Will composer make appropriate noises about switching the DOCTYPE of your
document under you? I doubt it.

Comment 3

[rbs]

see bug 78128 which has another patch.

Comment 4

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

Working on it for Nvu.

Comment 5

[Kathleen :Brade]

Comment on attachment 124477 [details] [diff] [review]
nothing fancy

This patch seems ok to me but I'm concerned that it may conflict with what
Daniel has done.  Passing review to him.

Timeless--does the doctype get altered when this is opened/saved in Composer or
is it left untouched?

Comment 6

[steve.swanson]

Note there's a similar mimetype check in the patch for bug 133827.  (You need 
this to get Copy/Paste to work.)  That patch is now part of the trunk.

Comment 7

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

timeless, I am a bit reluctant to give you an r= on this patch because it's
conflicting with my own changes. The basis of your code addition is absolutely
right, don't misunderstand me. But I'm doing more than that to allow to publish
with the correct mimetype, trigger the xml parser and not the html one, and so on.

Comment 8

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

Comment on attachment 124477 [details] [diff] [review]
nothing fancy

Since Nvu 1.0 codebase is schedule to die and since this is needed for Mozilla Composer, r=[e-mail]

Please note it _may_ haver some side effect on Composer in Seamonkey : the document may load but since some commands check the doctype of the document, they could fail.

Comment 9

[Chris Pearce [:cpearce (Not reading bugmail)]]

Comment on attachment 124477 [details] [diff] [review]
nothing fancy

This patch fixes bug 401990 (which is basically a dupe). Rob, can you sr+ this?

Comment 12

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 124477 [details] [diff] [review]
nothing fancy

This can land because it's basically a dup of a blocker.

Comment 13

[Mike Connor [:mconnor]]

Comment on attachment 124477 [details] [diff] [review]
nothing fancy

a=mconnor on behalf of drivers

Comment 14

[Reed Loden [:reed]]

timeless landed this at 2007-11-13 14:08

mozilla/editor/composer/src/nsEditingSession.cpp 	1.56