81069 - Add <title> tag when blank document is created in composer

Status: VERIFIED INVALID

(SeaMonkey :: Composer, defect, P3)

Description

[shrirang khanzode]

Launch a blank page in composer, view source.
Observe that the  '<title></title>' tag is not included unless u try to 
save/save the file

Comment 1

[rubydoo123]

moving to 1.0

Comment 2

[Ryan Cassin]

-> rcassin

Comment 3

[Kathleen :Brade]

Does adding something like this around line 1780 in editor.js work?

  // add a title to new pages
  if (headelement)
  {
    element = domdoc.createElement("title");
    if (element)
      headelement.appendChild(element);
  }

Comment 4

[Kathleen :Brade]

-->brade
I have the fix for this

Comment 5

[Syd Logan]

Kathy, good, can you attach a patch (after you finish driving home that is :-)

Comment 6

[Kathleen :Brade]

Attachment: small patch to add title

Comment 7

[Kathleen :Brade]

please disregard the last part of that patch (part of another bug which I will
not be checking in as part of this fix).

Comment 8

[kinmoz]

Does the method you are adding that code to (EditorSetDefaultPrefsAndDoctype())
get called even when we're loading existing documents that may already have a
title? If so, wouldn't you be adding a 2nd title under head?

Comment 9

[Charles Manske]

Kin: No, that shouldn't happen 'cause of this code above the new code:
  /* only set default prefs for new documents */
  if ( !newDoc  )
    return;

BUT: In my debug build as of 9/20/01, when I switch to HTML Source mode, I 
already *do* see <title></title> in the source, thus the new code *does* add a 
second title!

Comment 10

[Kathleen :Brade]

interesting... there are now two title tags in my build.  That is what I get for 
holding on to this patch for so long.  I'm guessing that the dtd or something 
else is forcing the creation of the title tag now.

I'm going to resolve this bug as invalid since we no longer seem to need to add 
the title tag.

Comment 11

[shrirang khanzode]

v