215993 - "Copy Email Address" doesn't unescape or strip leading space

Status: RESOLVED FIXED

(Firefox :: Menus, defect)

Description

[Jesse Ruderman]

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5b) Gecko/20030811 Mozilla
Firebird/0.6.1+

Steps to reproduce:
1. Go to http://www.claremont.edu/org/shades/index2.html
2. Right-click on one of the mailto links and select "Copy Email Address"

Result: "[e-mail]" on clipboard
Expected: "[e-mail]" or " [e-mail]" on clipboard

Source:
<a href="mailto:[e-mail]">[e-mail]</a>

If I click the link and it opens in Outlook Express, it turns into a space.
If I click the link and it opens in Mozilla Mail, the space disappears.

Comment 1

[Jesse Ruderman]

Was fixed for seamonkey in bug 160055.

Comment 2

[Alexey Chernyak]

Attachment: testcase

Comment 3

[Mike Connor [:mconnor]]

easy.  taking.

Comment 4

[Mike Connor [:mconnor]]

Attachment: port of fix from bug 160055

Comment 5

[Mike Connor [:mconnor]]

fixed branch and trunk.

Comment 6

[David Balažic]

Is this really fixed ?

Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:1.7.5) Gecko/20041107 Firefox/1.0

Visit http://lists.debian.org/debian-devel/2004/12/msg02200.html
right click on "To: [e-mail]" , select "Copy Email Address"
Then paste, result : debian-devel%40lists.debian.org
Should be : [e-mail]

HTML source :

<em>To</em>: <a
href="mailto:debian-devel%40lists.debian.org">[e-mail]</a>

Comment 7

[Jesse Ruderman]

I still see this bug in 1.0 and today's trunk, using the testcase in comment 2.

Comment 8

[Mike Connor [:mconnor]]

hmm, the code is still there, wonder what's going on now?  When I fixed this, it
worked...

Comment 9

[Phil Ringnalda (:philor)]

Regressed between 20041010 and 20041017 (I should have saved more builds): bug
263462 maybe? It's also regressed in Seamonkey.

Comment 10

[Mike Connor [:mconnor]]

Fixed again.  The original patch for Seamonkey relied on a broken bit of the
NS_EscapeURL to work, which was fixed 10-12 by jshin.  mscott brought the fix
onto the aviary branch, which regressed this, but it was missed by testing until
after Firefox 1.0, but it was backed out for tbird 1.0.

Using unEscapeURIForUI instead fixes this again.

Comment 11

[:Gavin Sharp [email: [e-mail]]]

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