206470 - editing text files shouldn't have HTML features in menus

Status: VERIFIED FIXED

(SeaMonkey :: Composer, defect)

Description

[timeless]

Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.4b) Gecko/20030519

Steps:
1. load a plain text file (text/plain, e.g. a bugzilla patch)
2. select file>edit page
3. try examine the menubar

Here's my list:
File
 Export to Text...
 Browse Page

Edit
 Paste without Formatting *1

View
 Show/Hide
  Format Toolbar
  Edit Mode Toolbar
 Normal Edit Mode
 HTML Tags
 HTML Source
 Preview

4. Look at the status bar, notice that it has glazou's html tag thingy, this
should be removed too.

*1 this feature could be useful if Paste actually pasted *with* formatting

Comment 1

[Kathleen :Brade]

I'm guessing we're just missing some checks for "!isHTML" (or whatever function
we have for this) for these menu items but maybe the mimetype isn't propagating
back properly or something.

Comment 2

[timeless]

Attachment: hide them

Comment 3

[jag (Peter Annema)]

Comment on attachment 124168 [details] [diff] [review]
hide them

+	     HideItem("cmd_viewFormatToolbar", false);
+	     HideItem("cmd_viewEditModeToolbar", false);

That shouldn't have a second parameter.

sr=jag with that fixed.

Comment 4

[Charles Manske]

Comment on attachment 124168 [details] [diff] [review]
hide them

looks good to me

Comment 5

[timeless]

checked in.
I'll solve the statusbar item later.

Comment 6

[Chris Petersen]

Verified in the 2003-05-29-08 Macho and Win32 trunk builds.