43307 - [UI] Implement appropriate items in context menu

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect, P4)

Description

[Judson Valeski]

create a table, right click it to bring up the context menu, select properties
and nothing happens.

Comment 1

[Kathleen :Brade]

*** Bug 43308 has been marked as a duplicate of this bug. ***

Comment 2

[Kathleen :Brade]

This is also a problem on Macintosh.

Comment 3

[rubydoo123]

setting to m17 and nominate for nsbeta2

Comment 4

[Charles Manske]

Changing summary to relect larger issue: table items in the context menus
are not complete. 
1. The "Insert", "Delete", and "Select" table submenus need to be added instead
of the single "Table" item. Command handlers need to be hooked up in the XUL.
2. Clicking on "Table Properties..." currently crashes on Windows.
3. Table items should only be visible when selection is or is inside a table.

Comment 5

[Jay Patel [:jay]]

Please create separate bugs for each issue mentioned and resubmit any 
nominations.

Comment 6

[Jay Patel [:jay]]

Putting on [nsbeta2-] radar and adding nsbeta3 keyword. Not critical to beta2. 
Yes, it is a regression and yes it looks bad, but we have no indication that any 
other sites show this behavior.

Comment 7

[rubydoo123]

we need to finalize our context menu spec and then go back through and finalize 
the menu items in the app. The spec will be completed during the UI review

Comment 8

[rubydoo123]

setting to nsbeta3+

Comment 9

[rubydoo123]

adding priority to status whiteboard

Comment 10

[Kathleen :Brade]

reassign to Charley

Comment 11

[Simon Fraser [no longer active]]

We decided on this layout in a UI meeting:

        Undo
        Redo
        ------------
        Cut
        Copy
        Paste
        Clear
        ------------
        Select All
        ------------
        Remove text styles      
        Create link... (text or image)
        Remove link     (If in a link)
        -------------
        Foo properties...
        -------------
        Table insert >   (If in a table, these)
        Table select >
        Table delete >
        Join cells
        Split cells


Remove preview in browser

Comment 12

[Lakespur Roca]

In build 2000-08-09-09m18 I see that if you try and select the table item (of 

course you can't really do that, but you highlight it and then click off when you 

nect invoke this contect menu the table item is already selected and as you mouse 

over you can have more than one highlighted at a time. I actually expect that 

this revamp of the menu will fix this too but I thought I'd mention it.

Comment 13

[leger]

Per PDT, menu items that appear must work or be pulled out ASAP. 

Comment 14

[leger]

Updating [PDT 9/1] to [PDT Check]

Comment 15

[rubydoo123]

*** Bug 49846 has been marked as a duplicate of this bug. ***

Comment 16

[rubydoo123]

*** Bug 50327 has been marked as a duplicate of this bug. ***

Comment 17

[Charles Manske]

This is simple but tedious work and should rightly be left for last priority, 
but must be done before shipping.

Comment 18

[rubydoo123]

*** Bug 49848 has been marked as a duplicate of this bug. ***

Comment 19

[Charles Manske]

Work is done -- waiting for review.

Comment 20

[Charles Manske]

checked in 9/5/00

Comment 21

[sujay]

wired up but I still see uncessary items for different objects.

For example, when I insert an image and right mouse click on it
to bring up the context menu, I see these options listed:

Join with Cell to the Right
Table or Cell Background Color

I will file separate bug

Comment 22

[Kathleen :Brade]

*** Bug 55745 has been marked as a duplicate of this bug. ***