68059 - remove extra items in context menu for objects

Status: NEW

(SeaMonkey :: Composer, defect)

Description

[sujay]

using 2/7 build of netscape

1) launch netscape
2) launch composer
3) insert object(image/h. line/link/table/anchor)
4) right mouse click on object.

Notice these extra menu items which don't need to be there:

Select All
Remove all Text Styles
Remove Links (should only be here after you've created the link)

all platforms.

Comment 1

[rubydoo123]

Select All should probably remain in the menu, the other options could remain in 
the menu, but should be disabled until such time where the selection pertains to 
 the choice.

Comment 2

[Charles Manske]

Probably will never change.
There's definitely nothing wrong with allowing "Select All" all the time.
As far as removing styles and links, it is an expensive calculation to do each
time UI is updated -- e.g., it would have to be done for each of the known
inline styles throughout the entire selection. So we decided it was not a
problem to always show those commands.
Any disagreement about this from others?

Comment 3

[Kathleen :Brade]

select all doesn't make sense if there isn't anything to select (imo)

as for the other commands, can't we do the "calculation" when the user pops up 
the context menu (rather than always showing them)?

btw, last I heard, in context menus items shouldn't be disabled, they should be 
removed

Comment 4

[Kathleen :Brade]

btw, this is similar (dependent?) on Simon's bug:  #29000

Comment 5

[Syd Logan]

spam composer change

Comment 6

[Charles Manske]

Please don't put "#" before bug number so we get a link! (e.g. bug 29000)

Comment 7

[Charles Manske]

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

Comment 8

[Arun Prasannan (polar)]

This bug is being marked EXPIRED as it has seen no activity in a very long time.

If you think that the issue reported might still be relevant, please test with a recent release of SeaMonkey and if the problem persists feel free to re-open the report. Thank you.

http://www.seamonkey-project.org/

Comment 9

[Mark Banner (:standard8)]

Bulk reopening incorrectly expired bugs - no activity does not constitute no bug - these need proper checking.