37078 - Edit menu items in the history window do not work.

Status: RESOLVED FIXED

(Core :: DOM: Navigation, defect, P2)

Description

[John Burns]

Bug : None of the Edit menu Items in the History window work.

Steps to repro:
1. Launch the Address Book component of Netscape 6
2. Click on the "Tasks" ("Aufgaben") menu
3. Click "Tools" ("Extras")
4. Click "History" ("History")
5. Click on the "Edit" ("Bearbeiten")
6. Select any option on the list.

Nothing happens regardless of the option choosen

Comment 1

[lchiang]

This is a bug in the core product (not de version) as well.  
http://bugzilla.mozilla.org/show_bug.cgi?id=32021

You should probably mark it a dup of that one.

Comment 2

[Laura Slevin]

Bug in the US product, so reassigning it

Comment 3

[lchiang]

If this is in the history book, it doesn't belong to mail/news.  Need to 
reassign.

Does this bug happen in the history window from the browser?

Comment 4

[scottputterman]

reassigning to don.

Comment 5

[verah (gone)]

Reassigning as per Don

Comment 6

[leger]

Putting on [nsbeta2+] radar for beta2 fix. 

Comment 7

[don]

Move to M18 target milestone.

Comment 8

[Doron Rosenberg (IBM)]

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

Comment 9

[Steve Lamm]

I added the "View", "Tasks", and "Help" menus. The edit menu items still need
some work and the "Search" menu needs to be added (should go in
utilityOverlay.xul).

Comment 10

[don]

Added "rjc" to status whiteboard since we need Robert's help to fix this for
beta 2 while slamm is on sabbatical.

Comment 11

[Robert John Churchill]

In its current state, I'm removing the "nsbeta2+" status for PDT to reconsider
as I don't think that this is critical for beta2.

Comment 12

[Jay Patel [:jay]]

Putting on [nsbeta2-] radar. Not critical to beta2.  Adding "nsbeta3" keyword 
for consideration of a fix for that milestone. 

Comment 13

[Chris McAfee]

m18

Comment 14

[verah (gone)]

Nav triage team: [nsbeta3-]; Claudius is writing a new bug to remove the 
nonworking "Find in History" and "Find Again" items in the History window Edit 
menu.

Comment 15

[Rafael Ebron (:rebron)]

nominating rtm.  This is annoying.

Comment 16

[sairuh (rarely reading bugmail)]

over to don --who should own this now that slamm's gone?

and this seems to have missed the rtm boat. so unless this has been
fixed/whatever, am adding relnoteRTM.

Comment 17

[joki (gone)]

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

Comment 18

[Alec Flett]

Mine!

Comment 19

[german]

UI comment: History is read-only in a sense that should not be able to paste into 
history (Not to self: update history spec to not that fact). Removing items and 
putting items onto the clipboard should be enabled though. I think this works as 
it should last Alec checked on a trunk build.
Further suggestion: we should somehow tie things better in with the pref that 
allows you to erase all history, at least common wording etc.

Comment 20

[Viswanath Ramachandran]

nav triage team: not a beta stopper. 

Comment 21

[Asa Dotzler [:asa]]

this should not be targetted on mozilla 0.9 if Netscape has determined that
alecf will not be working on it between now and nsbeta1.

Comment 22

[Alec Flett]

Attachment: fix, along with the fix for bug 63613

Comment 23

[Alec Flett]

the proposed patch fixes some minor bugs in the nsTreeController's cut method,
as well as removing the paste menu items (bug 63613) and fixing up "cut" by
adding the appropriate observer, and so forth.

Comment 24

[timeless]

looks good.

Comment 25

[Robert John Churchill]

As mentioned in bug # 63613, the "Paste" menu item should be disabled on Mac 
instead of removed.

Comment 26

[Alec Flett]

as mentioned in that other bug, we'll handle the mac-specific case in another bug.

Comment 27

[Robert John Churchill]

I'd suggest looking at how this was solved for bookmarks instead of checking in 
the "Paste" menu item "fix" and then opening up yet another bug;  basically, 
check the userAgent, and if the platform isn't Mac, then remove the "Paste" menu 
item node from the content model.

Comment 28

[Ben Goodger (use ben at mozilla dot org for email)]

seems OK. 

a=[e-mail]

Comment 29

[Alec Flett]

fix is in.