Closed Bug 331745 Opened 18 years ago Closed 18 years ago

"Email This Link…" doesn't appear for links whose "content" is an image

Categories

(Camino Graveyard :: Toolbars & Menus, defect)

PowerPC
macOS
defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED
Camino1.5

People

(Reporter: alqahira, Assigned: froodian)

References

()

Details

(Keywords: fixed1.8.1.1, polish)

Attachments

(1 file)

25.15 KB, application/zip
alqahira
: review+
stuart.morgan+bugzilla
: superreview+
Details
Bug 227803 added what is now known as "Email This Link…" to the context menu.  However, if the content between the <a></a> tag is an image, the "Email This Link…" item does not appear.

STR:
1. Visit the link above
2. Hover over the Mozilla banner; note the cursor turns into the hand to indicate a link
3. Ctrl-click the banner; note the absence of "Email This Link…" in the CM

I'm aware this is borderline bloat, but I think pink's comments in bug 227803 comment 5 are valid, and what's more, it's confusing that the item appears in the CM for some links not others.  (I tried to use that CM option today to mail a link to someone's forum profile, but the link was a little "button" image, so the CM item wasn't there....)
Taking this bug.
Assignee: mikepinkerton → mozilla
Status: NEW → ASSIGNED
Depends on: 342780
Bruce, do you think you will have time to get to this in the next month or so?

(The menu is "already" ported out of the nib into code, just waiting for review in bug 342780....)
BruceD, if you don't mind, taking.  I'll put up a fix for this once the nib in bug 343767 lands.
Assignee: mozilla → stridey
Status: ASSIGNED → NEW
Attached file New BrowserWindow.nib
This adds the menu item for the image/link context menu, and changes ... to … in all the context menus in that nib
Attachment #243857 - Flags: review?(alqahira)
Comment on attachment 243857 [details]
New BrowserWindow.nib

This isn't the right fix, since it ends up adding the "Email this Link" to Image links whose link is a mailto: url.
Attachment #243857 - Flags: review?(alqahira) → review-
Comment on attachment 243857 [details]
New BrowserWindow.nib

Well, it would help if my testcase didn't have a typo in it :P

This does work; sorry :(
Attachment #243857 - Flags: superreview?(stuart.morgan)
Attachment #243857 - Flags: review-
Attachment #243857 - Flags: review+
Comment on attachment 243857 [details]
New BrowserWindow.nib

sr=me. w00t!
Attachment #243857 - Flags: superreview?(stuart.morgan) → superreview+
Checked in on 1.8branch and trunk.
Status: NEW → RESOLVED
Closed: 18 years ago
Keywords: fixed1.8.1.1
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: