657106 - Give Debug Plain Text Editor a context menu

Status: RESOLVED FIXED

(SeaMonkey :: Composer, defect)

Description

[Ian Neal]

Attachment: Add context menu for plain text editor patch v1.0 [Checked in: Comment 4]

At the moment the plain text editor available from the composer debug menu does not have a context menu.
This patch gives it a context menu.

Comment 1

[Philip Chee]

Can't test because I'm getting this:

XML Parsing Error: undefined entity
Location: chrome://debugqa/content/debugQATextEditorShell.xul
Line Number 113, Column 7:
      <menu id="viewMenu"
------^

Comment 2

[[e-mail]]

Comment on attachment 532420 [details] [diff] [review]
Add context menu for plain text editor patch v1.0 [Checked in: Comment 4]

>+  <script type="application/javascript"
>+          src="chrome://communicator/content/contentAreaClick.js"/>
Why do we need this script?
(Nit: communicator scripts belong between editor and global scripts.)

Comment 3

[Ian Neal]

(In reply to comment #2)
> Comment on attachment 532420 [details] [diff] [review] [review]
> Add context menu for plain text editor patch v1.0
> 
> >+  <script type="application/javascript"
> >+          src="chrome://communicator/content/contentAreaClick.js"/>
> Why do we need this script?
Without it you get, when right clicking on the sidebar, in the error console:
Error: contentAreaClick is not defined
Source File: chrome://debugqa/content/debugQATextEditorShell.xul
Line: 1

Comment 4

[Ian Neal]

Comment on attachment 532420 [details] [diff] [review]
Add context menu for plain text editor patch v1.0 [Checked in: Comment 4]

With communicator script load moved between editor and global loads.

http://hg.mozilla.org/comm-central/rev/d850703965b6