After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 310539 - Copy Email address instead of Copy Link address when match is an email
Copy Email address instead of Copy Link address when match is an email
Status: RESOLVED FIXED
Product: gnome-terminal
Classification: Core
Component: general
2.11.x
Other Linux
: Normal enhancement
: ---
Assigned To: GNOME Terminal Maintainers
GNOME Terminal Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-07-15 20:39 UTC by Jorge Daza
Modified: 2006-01-05 21:19 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Distinguish between mailto:<address> and just <address> (1.88 KB, patch)
2005-07-15 20:41 UTC, Jorge Daza
none Details | Review
Previous was not valid, this is the valid. (2.23 KB, patch)
2005-07-15 20:45 UTC, Jorge Daza
none Details | Review
New patch trying to be in accordance with coding style (2.32 KB, patch)
2005-07-17 00:22 UTC, Jorge Daza
rejected Details | Review

Description Jorge Daza 2005-07-15 20:39:55 UTC
Version details: 2.11.1

This is a forward of a wish entered in Ubuntu's malon bugtracking system:

https://launchpad.ubuntu.com/malone/bugs/1219

Attached you'll find a proposed patch.
Comment 1 Jorge Daza 2005-07-15 20:41:16 UTC
Created attachment 49265 [details] [review]
Distinguish between mailto:<address> and just <address>
Comment 2 Jorge Daza 2005-07-15 20:45:24 UTC
Created attachment 49266 [details] [review]
Previous was not valid, this is the valid.
Comment 3 Michele Baldessari 2005-07-16 21:12:35 UTC
Hi Jorge,

please format your patch to be consistent with gnome-terminal's coding style.
(i.e. tabs != spaces)

After that I think it's a sensible change, hence moving to NEW
Comment 4 Jorge Daza 2005-07-17 00:22:57 UTC
Created attachment 49306 [details] [review]
New patch trying to be in accordance with coding style

Hello Michele,

actually, I don't know if it's a debian thing but tabs are spaces on my
original sources. I substituted the spaces by tabs as you said. Actually that's
how I code usually.

I hope this patch is the way it should.

Thanks,

Jorge
Comment 5 Guilherme de Siqueira Pastore 2006-01-05 21:19:35 UTC
Fixed in CVS HEAD with a different patch. Thanks!