Add Text Style to Context Menu in Composer

RESOLVED WONTFIX

Status

SeaMonkey
Composer
--
enhancement
RESOLVED WONTFIX
16 years ago
14 years ago

People

(Reporter: Chris Shaffer, Assigned: Chris Shaffer)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments, 3 obsolete attachments)

(Assignee)

Description

16 years ago
To reduce the number of 'mouse miles', it would be nice to have the 'Text Style'
sub menu as part of the Context Menu when Text is selected.
(Assignee)

Comment 1

16 years ago
Reassinged to myself (cherokeestargazer@yahoo.com)
Assignee: syd → cherokeestargazer
(Assignee)

Comment 2

16 years ago
Created attachment 104553 [details]
Patches chrome/comm.jar for Text Style in context Menu

I was not really sure what format I should have provided this 'patch' in.  I
archived the two files I changed into comm_Patch.jar.  The two files in the my
archive are:
content/editor/EditorContextMenuOverlay.xul
content/editor/EditorContextMenu.js

These file replace the files of the same name in chrome/comm.jar.  If I need to
provide this patch in another format, please email me and let me know.

Changes in "EditorContextMenuOverlay.xul"
-----------------------------------------
Added menu "fontStyleMenu_cm" to popupset "editorContentContextSet"

Changes in "EditorContextMenu.js"
-----------------------------------------
Added function call to SetElementEnabledById to enable Text Style sub menu if
object selected is NOT an image.
(Assignee)

Comment 3

16 years ago
Created attachment 104556 [details]
Applies 'Text Style' Menu to Context menu in editor

Added call to SetElementEnabledById to enable menu item 'fontStyleMenu_cm' if
object selected was NOT an image.
Attachment #104553 - Attachment is obsolete: true
(Assignee)

Comment 4

16 years ago
Created attachment 104558 [details]
Added tags to add Text Style Menu to Context Menu in Editor

Added tags for the 'Text Style' menu item to be added to the Context Menu with
in editor.  

Menu item added was 'fontStyleMenu_cm'.
cmanske: could you take a look at this "patch" and review it?

reporter: please attach a patch in patch format, by using "cvs diff -u >
name_of_patch_file" if you are using CVS.
(Assignee)

Comment 6

16 years ago
Created attachment 104716 [details]
EditorContextMenu.js - Inculdes edits to enable Text Styles in Context Menu

Applied edit to current CVS version of file to add 'Text Style' submenu to the
Context menu in Composer.  Used call to SetElementEnabledById() to enable the
submenu if objext selected was NOT an image
Attachment #104556 - Attachment is obsolete: true
(Assignee)

Comment 7

16 years ago
Created attachment 104717 [details] [diff] [review]
EditorContextMenu.js.dif - Difference between modded version and CVS version

'cvs diff -U EditorContextMenu.js' output
(Assignee)

Comment 8

16 years ago
Created attachment 104718 [details]
EditorContextMenuOverlay.xul - added tags for Text Styles Submenu 

Added tags to add 'Text Style' to Context menu in Composer
Attachment #104558 - Attachment is obsolete: true
(Assignee)

Comment 9

16 years ago
Created attachment 104719 [details] [diff] [review]
EditorContextMenuOverlay.xul.dif - Diff b/w local copy and CVS version

'cvs diff -U EditorContextMenuOverlay.xul' output
(Assignee)

Updated

16 years ago
Attachment #104716 - Attachment description: Inculdes edits to enable Text Styles in Context Menu → EditorContextMenu.js - Inculdes edits to enable Text Styles in Context Menu
(Assignee)

Updated

16 years ago
Attachment #104717 - Attachment description: Difference between modded version and CVS version → EditorContextMenu.js.dif - Difference between modded version and CVS version
Reporter, there is no need for attaching the whole document. Just attach one
patch with the changes to both files, and mark it "patch".

Maybe someone should CONFIRM this bug? I won't :-D
Attachment #104717 - Attachment is patch: true
Attachment #104717 - Attachment mime type: application/octet-stream → text/plain
Attachment #104719 - Attachment is patch: true
Attachment #104719 - Attachment mime type: application/octet-stream → text/plain

Updated

16 years ago
Attachment #104553 - Attachment is patch: false
Attachment #104553 - Attachment mime type: text/plain → application/zip

Comment 11

16 years ago
I'm not so sure this is a good idea! Context menus are crowded already. 
Brade?
I disagree with having this in the contextual menu. What we should have instead is
(un)dockable menus.

I recommend a WONTFIX for this bug.
(Assignee)

Comment 13

16 years ago
You do acculate alot of 'mouse miles' by having to back through the menu system
to get to some type of special style (like CODE...)  Perhaps a drop down list on
the toolbar would work...  I'll try that...

Comment 14

15 years ago
per Glazman, wontfix.
Status: UNCONFIRMED → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → WONTFIX
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.