Copy/Paste/Cut (etc) disabled within Browser

VERIFIED FIXED in M14

Status

()

Core
XUL
P1
blocker
VERIFIED FIXED
19 years ago
18 years ago

People

(Reporter: Eli Goldberg, Assigned: Simon Fraser)

Tracking

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [PDT+] 1/28)

(Reporter)

Description

19 years ago
In today's builds (checked Linux, Mac & NT 4), all Editing-related items under
the Edit menu are disabled.

Of course, this renders it impossible to verify, write, or follow up on any copy/
paste bugs.

If this could be fixed as soon as possible, it would be most appreciated.

Updated

19 years ago
Assignee: trudelle → saari
Priority: P3 → P1
Target Milestone: M11

Comment 1

19 years ago
Chris, could you look at this?  reassigning to saari as p1 for M11.

Updated

19 years ago
Blocks: 14356

Updated

19 years ago
Blocks: 13873

Updated

19 years ago
Assignee: saari → don

Comment 2

19 years ago
paste works, but the menu item is not shown enabled. Now then...

There are two nodes with id="cmd_copy". This is a bug. There is one in
navigator.xul (and other xul files), and one in globalOverlay.xul

Don, I leave it to one of your team to figure out which is the right node to
keep. But it looks like you want the one in globalOverlay.xul, based on looking
at the XUL and tests.

Updated

19 years ago
No longer blocks: 13873

Updated

19 years ago
Assignee: don → davidm

Comment 3

19 years ago
David, can you figure out what's going on here ...
(Reporter)

Updated

19 years ago
Blocks: 12541

Comment 4

19 years ago
adding myself to cc: list.

Comment 5

19 years ago
talked to hangas and we have to wait for an ender controller and then this
should magically work. He also mention he thought Radha was doing this also.

Updated

19 years ago
QA Contact: beppe → elig
(Reporter)

Updated

18 years ago
Blocks: 12691

Updated

18 years ago
Summary: Copy/Paste/Cut (etc) disabled in today's build → [DOGFOOD] Copy/Paste/Cut (etc) disabled in today's build

Comment 6

18 years ago
Added "[DOGFOOD]" to get this bug on PDT radar. When I'm trying to use SeaMonkey
for daily work, I have to copy the current url from the page, load 4.7, and then
copy out some big piece of text to go into a mail message, etc.

Updated

18 years ago
Depends on: 12022

Comment 7

18 years ago
add dependency

Updated

18 years ago
Whiteboard: [PDT+]

Comment 8

18 years ago
Putting on [PDT+] radar.

Updated

18 years ago
Target Milestone: M11 → M13

Comment 9

18 years ago
m13 due to dependancy

Comment 10

18 years ago
*** Bug 17577 has been marked as a duplicate of this bug. ***

Comment 11

18 years ago
OK, if buster can get bug #12022 fixed (and I mean more than a day) before M12,
then we can move this back to M12 as well.

Updated

18 years ago
Whiteboard: [PDT+] → [PDT+] 11/26 completion
Target Milestone: M13 → M12

Comment 12

18 years ago
David, buster sez that he'll have bug #12022 fixed no later than 11/19 so I
moved this one back to M12 and put a 11/26 completion date on it.  Yeah, I know
11/26 is Turkey Day, but jar wants the week ending dates for schedules.  This
means you'll need to get this completed by 11/24.  Can you do that?  If not,
let's move this out to 12/3/
(Reporter)

Updated

18 years ago
Summary: [DOGFOOD] Copy/Paste/Cut (etc) disabled in today's build → [DOGFOOD] Copy/Paste/Cut (etc) disabled within Browser

Comment 13

18 years ago
Added scalkins and amusil to cc as this affects IM.
Blocks: 8427

Updated

18 years ago
Blocks: 19423
*** Bug 16293 has been marked as a duplicate of this bug. ***

Updated

18 years ago
Whiteboard: [PDT+] 11/26 completion → [PDT+] 12/3 completion

Comment 15

18 years ago
Move the expected completion out to 12/3 since buster's current set to changes
to bug #12022 don't seem to solve the problem yet.

Updated

18 years ago
Blocks: 20203

Updated

18 years ago
Blocks: 20348

Updated

18 years ago
Whiteboard: [PDT+] 12/3 completion → [PDT+] 12/10 completion

Comment 16

18 years ago
Move completion target from 12/3 to 12/10 since buster has re-assigned us bug
#12022.  And bug #12022 is gated by bug #2253 which will not be completed until
12/3.  Ugh.

Comment 17

18 years ago
I checked in the code to enable cut/copy/and past in text areas. Single line edit
fields are waiting for a controller. There are also appear to be some bugs in the
text area controller. I'll wait a bit before marking this fixed although I think
my work is done.

Comment 18

18 years ago
Still have to do the hmtl content area. This requires some XPToolkit work to be
done. I have a controller which should work in my tree.

Updated

18 years ago
Blocks: 20870

Comment 19

18 years ago
*** Bug 20900 has been marked as a duplicate of this bug. ***
(Reporter)

Comment 20

18 years ago
*** Bug 21112 has been marked as a duplicate of this bug. ***

Updated

18 years ago
Status: NEW → ASSIGNED

Updated

18 years ago
Assignee: davidm → trudelle
Status: ASSIGNED → NEW

Updated

18 years ago
Assignee: trudelle → davidm

Comment 21

18 years ago
reassign back to me

Comment 22

18 years ago
Paste does not work in Win98 1999121308 mozilla build.  Appears to work fine for
Linux6 & MacOS86 of same build date.

Comment 23

18 years ago
I checked in a commented out version of the dom controller so that mjudge and
company can work on some related stuff. The enable and disable routines are not
being called for some reason. saari was going to look at that

Comment 24

18 years ago
copy now works in browser content areas.

Updated

18 years ago
Status: NEW → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Comment 25

18 years ago
copy now works in browser content areas.
(Reporter)

Comment 26

18 years ago
Really? Has this been checked into the main tree?

The "Copy" menu items (and keyboard shortcut) are both inactive on the 1999121408
Linux & Win32 builds. (Works fine on Mac OS.)

Comment 27

18 years ago
hmm it works on my linux build that I did last night and on mjudges windows box.
I don't know how to install nondebug builds on linux so I can't verify. Does the
edit field ( url bar ) or selection in a mail message work?

Comment 28

18 years ago
Saw the same thing on Don's linux machine. Will investigate a bunch more

Comment 29

18 years ago
It's working well for me in the browser window on my linux build today, except
that in the url bar (or presumably, other text fields if any exist), I have to
do the multiple-click sequence to get the focus there and get the menu enabled,
due to bug 21610.
(Reporter)

Comment 30

18 years ago
Using today's builds (all 3 platforms), I'm seeing the menu items enabled ---
however, copy doesn't work within the browser content region on many pages.

Not sure if 12022 needs to be re-opened, or if a new Dogfood bug should be
written for davidm, but will do one of the two in the next 60 minutes. ;)
(Reporter)

Comment 31

18 years ago
Broke off the problem of Browser content copy/paste not working on some pages
into a new bug --- 21832, "[DOGFOOD] Browser content frequently cannot be
copied".

Updated

18 years ago
Blocks: 22176

Comment 32

18 years ago
*** Bug 4147 has been marked as a duplicate of this bug. ***

Comment 33

18 years ago
*** Bug 4617 has been marked as a duplicate of this bug. ***

Updated

18 years ago
Status: RESOLVED → REOPENED

Comment 34

18 years ago
I'm seeing this again in the 1/3 and 1/4 linux builds.  When I first bring up
the browser window, everything works, but soon (going to Tinderbox from the
menubar bookmarks menu usually does it) I'll get into a mode where key events
are ignored, and hitting things like Page Down gives me the message: JavaScript
Error: TypeError: controller has no properties

Updated

18 years ago
Resolution: FIXED → ---

Comment 35

18 years ago
Clearing FIXED resolution due to reopen.

Updated

18 years ago
Assignee: davidm → hyatt
Status: REOPENED → NEW
OS: All → Linux

Comment 36

18 years ago
reassigning to hyatt ( feel free to pass it to who ever does unix events).
Marking Linux only since I can't reproduce on Mac or Windows builds.

Updated

18 years ago
OS: Linux → All

Comment 37

18 years ago
I see this on the Mac, as well, using a build from yesterday.  I don't see it on
Windows.  Changing platform back to All (since there's no setting for "All but
Windows" :-)

Updated

18 years ago
Assignee: hyatt → saari
Whiteboard: [PDT+] 12/10 completion → [PDT+]
Target Milestone: M12 → M13

Comment 38

18 years ago
I'm seeing this too. reassigning to saari, cc'ing hyatt, targetting M13,
clearing estimated fix date (please set again).

Comment 39

18 years ago
Incidentally, this isn't just a key binding problem.  The menu item Edit->Copy
doesn't do anything either.

Comment 40

18 years ago
On the mac once I give the html content area focus it seems to work for me. I
can't figure out where the focus goes after loading a new page. There is the
problem that page up and page down when the focus is in the editfield should
really apply to the content area but that is a separate bug.

Comment 41

18 years ago
On Linux, clicking in the content area doesn't help, either with copy or with
page up/down.
(Reporter)

Comment 42

18 years ago
Using the 2000010608 build (surfing around macweek.com and CNN.com):

Mac OS:
	Menu items are normally selectable, but ordinary content is irreproducibly
not copied.

Win32:
	Cut/Copy/Paste menu items are occasionally not enabled. Unable to
consistently reproduce.

Linux:
	(deferring to Akkana's comments from 2 hours ago)
(Reporter)

Comment 43

18 years ago
...I note that using the 2000010608 builds on Win32/Mac OS/Linux, if you:
	* launch the browser (exit it if it's already running, and then re-launch)
	* select text in the URL window
	* Open the Edit menu

...neither Cut nor Copy are enabled. (I think davidm mentioned this for Mac OS.)

Comment 44

18 years ago
Neither Hyatt nor myself changed anything that would have broke this again, so if anyone else knows what might have changed, please speak up.

Comment 45

18 years ago
I'm going to need help figuring this out. Ender seems to have its own ideas about
what should be enabled and when. I notice Paste is enabled when nothing is on the
clipboard for example.

Who wants to work on this?

Comment 46

18 years ago
I don't think this is an Ender issue -- it works fine in all the editor windows.
 I'm not sure how the browser window controller updating works, though.

Comment 47

18 years ago
Hrm, ok. In my current Mac build the Cut/Copy/Paste seem pretty reasonable until
I start messing with the URL bar, thus my ender comment.

Comment 48

18 years ago
On Linux, at least, Copy from the browser's content area hasn't worked since
last year.  I thought I'd heard the same from Mac users.

Comment 49

18 years ago
I just tried it, and it worked for me on MacOS. Running my linux and windows
builds right now, so I'll look at those ASAP.

Comment 50

18 years ago
Copying from text widgets and from the content area doesnt work at all in Linux.

Choosing "Copy Link Location" from the context menu over a link, then pasting

elsewhere copys the link, and a great deal of junk, that looks nastily like

random bits of memory:

http://bugzilla.mozilla.org/show_bug.cgi?id=22176^@^@^@1^@^@^@N,@N,@

^H^HO,@^@^@^@^@^@^@^@^@X^@^@^@^@^@^@N,@x^H0^@



(I have removed extended char's incase some of you have sensitive mail clients)

Comment 51

18 years ago
I can start looking at this too.

Comment 52

18 years ago
I just tried this out on my RedHat 6.0 system. Highlighting text and
choosing the "Copy" command from the "Edit" menu did copy text out
of the browser. The only problem with this is that it is totally
different from Netscape 4.0 and all other X based applications.
Just highlighting the text should be the same as a "Copy" operation
under Linux (and any other UNIX like system that uses X). If linux
folk are forced to select and choose "Copy" from a menu, there are
going to be really unhappy.

Comment 53

18 years ago
So this is working, weird X conventions withstanding?

I did the same thing (copy with menu command) on my linux box and it worked for
me too.

Comment 54

18 years ago
No, it's not working.  It works initially before you set the focus anywhere
else, but start messing with the focus and it'll stop working and, usually,
never come back.

Updated

18 years ago
Blocks: 22528

Comment 55

18 years ago
Start mozilla
go to www.mozilla.org
highlight text in content window
go to file menu, "copy" is not enabled.

however, if i play games with unhighlighting the text, and clicking
back and forth several times between the url bar and content window
then I can eventually get text copied to the clipboard. (This is with
today's nightly build from ftp.mozilla.org)

Updated

18 years ago
Status: NEW → ASSIGNED

Comment 56

18 years ago
So where is the controlller for these commands? I need to start littering debug
code to figure out what is happening.

Comment 57

18 years ago
nsGlobalWindow.cpp. nsDOMWindowController. The controller is created in
GlobalWindow::GetControllers.
(Reporter)

Updated

18 years ago
Blocks: 21832
(Assignee)

Comment 58

18 years ago
I'm doing related work in composer, and am getting somewhat stuck.

Comment 59

18 years ago
Simon, what are you working on? I just added a bunch of debug code on my Mac

build, just as a sanity test, and the command dispatcher is definitely calling

UpdateCommands at the appropriate times, and the commands are enabling/disabling.

Updated

18 years ago
Blocks: 4722

Updated

18 years ago
Target Milestone: M13 → M14

Comment 60

18 years ago
Moving to m14. If this is (still) blocking someone, please add a comment saying
who is blocked from doing what.

Comment 61

18 years ago
Blocker for 22528, 4722; general cut & paste.
If it's not m13-able then so be it.
Other dependencies are listed for this bug in
the dependency list.
(Assignee)

Comment 62

18 years ago
Oh we have a twisty maze of copy & paste bugs. All these are ultimately dependent
on what hyatt describes at the bottom of 20471, that saari has to move the
command dispatcher.

Comment 63

18 years ago
Is there a bug on moving the command dispatcher?  20471 only has one dependency
and it's marked fixed.

Updated

18 years ago
Whiteboard: [PDT+] → [PDT+] 1/28

Updated

18 years ago
Blocks: 24854

Updated

18 years ago
Keywords: beta1

Comment 64

18 years ago
*** Bug 22384 has been marked as a duplicate of this bug. ***

Comment 65

18 years ago
Give me a bug on moving the dispatcher.  I don't think there is one.

Updated

18 years ago
Blocks: 25824

Comment 66

18 years ago
*** Bug 21832 has been marked as a duplicate of this bug. ***

Updated

18 years ago
No longer blocks: 25824
(Reporter)

Comment 67

18 years ago
[Note to self: be sure to also verify 21832 upon verifying this bug.]
pardon the spam: adding me to cc'list as this touches on my area.

Comment 69

18 years ago
Putting dogfood in the keyword field.
Keywords: dogfood

Updated

18 years ago
Summary: [DOGFOOD] Copy/Paste/Cut (etc) disabled within Browser → Copy/Paste/Cut (etc) disabled within Browser

Updated

18 years ago
Blocks: 26981

Comment 70

18 years ago
All the cases I know of that cause this horkage seem to be fixed by the checkin 
Hyatt and myself just did. It fixed an interaction between the command dispatcher 
and focus.

Marking fixed until someone comes up with a new test case.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago18 years ago
Resolution: --- → FIXED
(Reporter)

Comment 71

18 years ago
Oy. 

For one thing, the focus-related problem blocking copying browser content 
described by trudelle in 21832 (1999-12-16 09:00) is still occuring on this 
morning's build. (only checked Mac OS)

Since 21832 is marked as a duplicate of this bug, I'm re-opening this bug, even 
though the problem has nothing to do with menu items being disabled. 

Of course, please feel free to re-resolve this one and re-open 21832. 

Thanks!
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Reporter)

Comment 72

18 years ago
...is the focus issue trudelle described covered by 12051, or is that a different 
focus problem?

If it's the same thing, then I think this bug can be marked as resolved. (Have 
went through the first half-dozen duplicates, and they're all fixed other than 
the focus problem.)
(Assignee)

Comment 73

18 years ago
There are a number of things still to be done here; focus is one issue, but there 
are also some underlying code pieces that still have to be fixed. I'll mark the 
bug fixed when I think it's done.

Comment 74

18 years ago
Let's confine this bug to the originally reported problem.  I think that focus
issues and underlying code should only keep this bug open if the original
problem still exists.  If it does not, then new bug reports should be filed for
these other problems.  I see at least one case where the original problem still
exists, at least on Win98 using comm opt bits, and one that should perhaps be a
new bug:
Copy some text from a web page. 
Select part of the URL in the URL bar.
Expected result: copy and paste enabled
Actual result: Paste enabled, copy greyed out but still enabled.
Select some text in the web page.
Click in the URL bar, leaving a caret but no selection.
Expected result: only paste enabled
Actual result: Cut, Copy and Paste enabled.
(Reporter)

Comment 75

18 years ago
Peter, also note bug #25714, which sounds like a variant of the first issue that 
you mention.

Comment 76

18 years ago
*** Bug 27068 has been marked as a duplicate of this bug. ***

Comment 77

18 years ago
For clarification, all the cut, copy and paste keybindings seem to be working 
properly, which means that the command dispatcher is working properly, which 
means that focus is working properly.

So, I think the menus enabling/disabling is the last issue

Comment 78

18 years ago
Be careful here.  Input fields and text areas don't use the command dispatcher. 
They use XBL.  Cut/copy/paste will work using key bindings in input fields and 
text areas without the command dispatcher necessarily working perfectly.

I happen to think that the command dispatcher is largely working well at this 
point anyway, but you can't verify that any more when using key bindings on 
input fields and textareas.

Comment 79

18 years ago
Okay, good point. So are 26882 and 27661 XBL as well?

Comment 80

18 years ago
Simon, you said you're working on this, here is the uber bug.
Assignee: saari → sfraser
Status: REOPENED → NEW
(Assignee)

Comment 81

18 years ago
This should work now, for both content and the url bar
Status: NEW → RESOLVED
Last Resolved: 18 years ago18 years ago
Resolution: --- → FIXED
(Reporter)

Comment 82

18 years ago
Per trudelle's comment that "Let's confine this bug to the originally reported 
problem.", I'm re-opening 21832, which I'm seeing after doing a quick check on 
the 2.17.00 Mac OS build.
(Reporter)

Comment 83

18 years ago
Outside of 21832, I don't see any problems in the current builds after about 20 
minutes of investigation.

Peter Trudelle & Company: This bug touched on a huge number of issues --- are any 
of you still seeing problems? If not, I'll go ahead and mark this as Verified on 
Monday. Thanks!

Comment 84

18 years ago
On NT, I'm unable to copy/paste from mail messages in the normal mail display. 
It works if I double click on them to bring them up into a browser window. 

Copy/paste from browser windows seems fine except that when you have something
highlighted in the browser window and then highlight text in the url bar, the
text in the browser window doesn't unhighlight. (this doesn't prevent
copy/paste though) (I tried both ctrl-C and menu)

Please direct me to the proper bugs if this isn't right.

Comment 85

18 years ago
I also am unable anything in mail windows.  It's only in mail, meaning it's not
this bug.  Has anyone filed a bug on the mail window problem yet? 

Comment 86

18 years ago
looks like I'll reopen http://bugzilla.mozilla.org/show_bug.cgi?id=21185 for 
mail after checking it out.

Comment 87

18 years ago
Answering my own question: 21832 was just reopened on an issue which I think is
the same one causing inability to copy from the mail window.
(Assignee)

Comment 88

18 years ago
Inability to copy from mail 3-pane window is bug 19428, which I am just about to 
check in a fix for.
(Reporter)

Comment 89

18 years ago
I believe that the second selection issue endico raises is covered by 7527, which 
is marked duplicate of 7465. Left note in 7465 to check that scenario when 
verifying.
(Reporter)

Comment 90

18 years ago
Since the originally reported problem is fixed, and no new issues were raised by 
the zillion people CC:'d on this bug, I'm marking this as Verified.
Status: RESOLVED → VERIFIED
(Reporter)

Comment 91

18 years ago
[Endico's second issue is known, but didn't have an active bug report. Wrote it 
up as 28999.]

Updated

18 years ago
No longer blocks: 20203

Updated

18 years ago
No longer blocks: 20870

Updated

18 years ago
No longer blocks: 22176
You need to log in before you can comment on or make changes to this bug.