Closed
Bug 349865
Opened 18 years ago
Closed 17 years ago
Drop "Copy Link Location" from mailto: link context menu
Categories
(Firefox :: Menus, enhancement)
Firefox
Menus
Tracking
()
VERIFIED
FIXED
Firefox 3 beta4
People
(Reporter: sgarrity, Assigned: ehsan.akhgari)
References
(Depends on 1 open bug, )
Details
Attachments
(1 file)
1.33 KB,
patch
|
Gavin
:
review+
beltzner
:
ui-review+
beltzner
:
approval1.9+
|
Details | Diff | Splinter Review |
If you right-click on an email link (with a "mailto:" in the href), there are a two variations of the copy command available: 1. Copy Email Address 2. Copy Link Location Copying the email address does what you would expect, putting "name@domain.com" in the clipboard. The Copy Link Location copies the contents of the "href" attribute, putting something like "mailto:name@domain.com" in the clipboard. I'm finding it a bit difficult to imagine a situation in which one would want to copy the link location for an email link. If there is something I'm missing, please let me know. The only argument I could think of for this was consistency with other links - though I'm not sure it's a strong point. Can we scrap the Copy Link Location in mailto: links? Bug #102418 covered this for the old Mozilla suite.
Comment 1•18 years ago
|
||
When I paste mailto:name@domain.com into the location bar it starts up my e-mail client I guess.
Comment 2•18 years ago
|
||
If I'm looking for a job and somewhere on monster says "click here to apply" where that hyperlink is a mailto: like the following, it makes sense (at least for me). href="mailto:jobs@example.com?subject=Senior Astrophysical Thermohydrologist&body=I would like to apply for this job. I have attached my resume for your review." Since I use gmail, when I click on that link, it doesn't go to my email client of choice (bug? extension request?). I need to make sure I spell "Senior Astrophysical Thermohydrologist" correctly so they even look at the email, and if they request that the body of my email look exactly like that, then I need to copy it just like it is. So it's useful to me, but I'm aware I might be the 1% that uses it and the menu item still might be ripe for removal. ~Mike
Assignee | ||
Comment 3•17 years ago
|
||
The use case in comment 2 can still be met by selecting Properties from the context menu, and copying all (or parts) of the Address field. I would argue that it's even better, since it makes sure you're only copying the parts you want (for example, only the subject of the email.) Taking over...
Assignee: nobody → ehsan.akhgari
Version: unspecified → Trunk
Assignee | ||
Updated•17 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → Firefox 3 M11
Assignee | ||
Comment 4•17 years ago
|
||
Simple fix.
Attachment #295548 -
Flags: ui-review?(johnath)
Attachment #295548 -
Flags: review?(gavin.sharp)
Comment 5•17 years ago
|
||
Comment on attachment 295548 [details] [diff] [review] Patch (v1) Cancelling ui-review request. I ui-r on security stuff only, I recommend asking mconnor or beltzner for ui reviews the rest of the time.
Attachment #295548 -
Flags: ui-review?(johnath)
Assignee | ||
Updated•17 years ago
|
Attachment #295548 -
Flags: ui-review?(mconnor)
Assignee | ||
Updated•17 years ago
|
Attachment #295548 -
Flags: ui-review?(mconnor) → ui-review?(beltzner)
Assignee | ||
Comment 6•17 years ago
|
||
Comment on attachment 295548 [details] [diff] [review] Patch (v1) Asking mano for review because this has been stuck in gavin's queue for a long time
Attachment #295548 -
Flags: review?(gavin.sharp) → review?(mano)
Comment 7•17 years ago
|
||
Comment on attachment 295548 [details] [diff] [review] Patch (v1) r=me, but ui-r is what's important here.
Attachment #295548 -
Flags: review?(mano) → review+
Assignee | ||
Comment 8•17 years ago
|
||
(In reply to comment #7) > (From update of attachment 295548 [details] [diff] [review]) > r=me, but ui-r is what's important here. Thanks for the review, gavin. Now, let's see what Beltzner thinks.
Assignee | ||
Comment 9•17 years ago
|
||
Beltzner: ping...
Comment 10•17 years ago
|
||
Comment on attachment 295548 [details] [diff] [review] Patch (v1) Yes, definitely.
Attachment #295548 -
Flags: ui-review?(beltzner) → ui-review+
Assignee | ||
Comment 11•17 years ago
|
||
Comment on attachment 295548 [details] [diff] [review] Patch (v1) This is a low-risk change which fixes a UI polish issue. Requesting approval to land this.
Attachment #295548 -
Flags: approval1.9?
Assignee | ||
Updated•17 years ago
|
Target Milestone: Firefox 3 beta3 → Firefox 3 beta4
Comment 12•17 years ago
|
||
Comment on attachment 295548 [details] [diff] [review] Patch (v1) a=beltzner for 1.9
Attachment #295548 -
Flags: approval1.9? → approval1.9+
Updated•17 years ago
|
Keywords: checkin-needed
Comment 13•17 years ago
|
||
Checking in browser/base/content/nsContextMenu.js; /cvsroot/mozilla/browser/base/content/nsContextMenu.js,v <-- nsContextMenu.js new revision: 1.36; previous revision: 1.35 done
Comment 14•17 years ago
|
||
Verified with: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.4; en-US; rv:1.9b4pre) Gecko/2008022504 Minefield/3.0b4pre ID:2008022504 Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b4pre) Gecko/2008022504 Minefield/3.0b4pre
Status: RESOLVED → VERIFIED
OS: Linux → All
Comment 15•13 years ago
|
||
My request bug 678927 is about "Copy link address", which is missing. Even if "Copy email address(es)" becomes perfect and copies 10 email addresses, that will not give me the link address. I want to copy the link address, like for any normal link. With "mailto:" and the email addresses and the subject and whatever. Web browsers have been providing that simple feature for 10 years. Please bring it back. Thanks. I particularly miss this feature because sometimes the link address is not fully shown although there is plenty of space : bug 658918. And here is another scenario where the user wants the whole link : The user wants to paste it in a place where it has to be a whole "mailto:" link, like an HTML page the user is writing. Regarding the "Properties" way (bug 349865, comment 3) : In Firefox 3.0.1, I have "Properties" in the contextual menu and then I have the link address in the properties. This is less handy than the "Copy link address" contextual menu item, but this is a good idea. But in more recent version of Firefox, I don't have any "Properties" contextual menu item.
Comment 16•13 years ago
|
||
Let's suppose I want to compose the email with the subject and everything, but not now, because my email program is not ready yet (I am downloading it, or installing it, or tweaking it…). I don't click on the "mailto:" link. Instead, I right-click : copy its address. And when ready, I "paste and go" in my navigator, and the email gets composed.
You need to log in
before you can comment on or make changes to this bug.
Description
•