Closed Bug 19340 Opened 25 years ago Closed 24 years ago

Copy image location not null terminating string?

Categories

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

Product:
Core
Component:
DOM: Editor
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: jpranevich, Assigned: law)

Details

If you go to a page, http://50.lycos.com

Right click on the big "Lycos 50" image and select "Copy Image Location"


Select the URL bar at the top and type Ctrl-V to paste the URL of the image and
I get:

http://lygo.com/ly/c/td/50_type.gif¨?`1Qtext/plain


The first part is obviously correct. You can see however that it is not properly
terminated and other data can be seen.
I can verify that this also happens for "Copy Link Location" menu items. This is
on today's (11/19/99) daily build.
Component: Browser-General → XP Toolkit/Widgets
beppe, would this cut and paste action be yours?
Component: XP Toolkit/Widgets → Editor
QA Contact: leger → sujay
OS: Windows 98 → All
Hardware: PC → All
This is also present in my Solaris 2.6 build of M11.
Looks like leger didn't hit "reassign".  Doing so, since that would allow
beppe@netscape.com to read the comment addressed to her.
Assignee: beppe → buster
yes it is ours, assigning to buster
Assignee: buster → law
cc'ing some folks who own related code.  do any of you think this bug is yours?
I won't have time to investigate for a couple of days.

However, first things first:  I can't reproduce the problem because I get a
javascript error from nsContentMenu.js when I right-click on an image and select
"Copy Image Location"

JavaScript Error: TypeError: this.imageURL is not a function
URL: chrome://navigator/content/nsContextMenu.js
LineNo: 293

It looks like imageURL is used both as an attribute and a function.  I don't
know javascript very well, but this seems pretty odd.  Throwing over to bill to
fix this problem first.
Status: NEW → ASSIGNED
Target Milestone: M15
Move to M15.
This is also present in M12-1 from Debian.




Move to M16 for now ...
Target Milestone: M15 → M16
Target Milestone: M16 → M18
Move to M21 target milestone.
Target Milestone: M18 → M21
context menu over an image in mailnews is completely non-functional, apart from
"select all" which selects text, not image.

If i click the image, the "copy" item is later enabled in context menu.
If i then click "copy" - the alt-text is copied.

"View Image", "Save image as" and "Copy Image location" all do nothing, but
spawn this in console on linux 2000122306:

... entering nsContextMenu constructor in nsContextMenu.js
... leaving nsContextMenu constructor in nsContextMenu.js
JavaScript error:
 line 0: contextMenu has no properties

regarding my comment above: that was fixed in bug 62964
Fixed a while back.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
works now..the whole URL is pasted in the field...

verified in 1/24 build.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.