222204 - A mailto: link for the reporter would be very useful.

Status: RESOLVED FIXED

(Bugzilla :: User Interface, enhancement)

Description

[Mark Lassau]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5) Gecko/20030925
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5) Gecko/20030925

Additional comments include a mailto: link for the user who entered the comments.
The same functionality would be even more helpful for the initial reporter.

This could occur at the top of the page:
add a link to "Reporter:   	[e-mail] (Joe Bloggs)"
AND/OR above the desciption:
eg have "Description: (From Joe Bloggs)" similar to the "Additional Comment #1
From Joe Bloggs"


Reproducible: Always

Steps to Reproduce:

Comment 1

[Vlad Dascalu]

Attachment: V1

Comment 2

[Vlad Dascalu]

Confirming.

Comment 3

[Christian Reis]

Comment on attachment 133365 [details] [diff] [review]
V1

Looks simple enough. 

I considered spambots, but I don't think a mailto: makes any difference when we
have the email address right there anyway :-)

Comment 4

[Dave Miller [:justdave]]

Comment on attachment 133365 [details] [diff] [review]
V1

Seems like the comments just show the real name for the link text.  This would
make it inconsistent with those, wouldn't it?

Comment 5

[Christian Reis]

I don't think that's a very useful consistency to enforce <wink> -- the comments
are miles away from the reporter name, and if we change the interface (as I
would very much like to do) to have them closer, we can make it consistent then.

Comment 6

[Myk Melez [:myk] [@mykmelez]]

Comment on attachment 133365 [details] [diff] [review]
V1

>-        [% bug.reporter.identity FILTER html %]
>+        <a href="mailto:[% bug.reporter.email FILTER html %]">
>+          [% bug.reporter.email FILTER html %]</a>
>+        ([% bug.reporter.name FILTER html %])

You should linkify the whole identity rather than just the email address. 
"identity" does more than just munge email and name together; it checks to see
if the user has a name first.  The code in this patch can result in something
like the following, which won't look good:

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

Comment 7

[Vlad Dascalu]

Attachment: V2

Comment #3 - regarding spambots, "FILTER html" eliminates the @ sign from the
html source so the "protection" againest bots is identical to the protection
that we currently have on "Additional Comments".

I agree with comment #5 (inconsistency is already present and it can be useful
in some situations) and with comment #6.

Comment 8

[Vlad Dascalu]

Attachment: The correct V2

(I've uploaded the wrong diff)

Comment 9

[timeless]

not that i care, but this will make it *harder* for people to email me.
this might be a good thing.

Comment 10

[Christian Reis]

Comment on attachment 133407 [details] [diff] [review]
The correct V2

Err, what about the parenthesis?

Comment 11

[Vlad Dascalu]

In reply to comment #9: When viewing the results of a bug-search, we already
have a link called "Send Mail to Bug Owners" that represents a href mailto link,
which already is in the CVS and breaks your approach of using shadows accounts
(watchers) in order to receive mail.

In reply to comment #10: the implementation of the patch version 2 is "above"
the layer of code that adds the "(" and ")" thing. If those are set in the
.identity property, then we display them, otherwise we don't. The CVS tip
currently uses a different format (see for example the reporter in
http://landfill.bugzilla.org/bugzilla-tip/show_bug.cgi?id=333 ).

Comment 12

[Christian Reis]

Comment on attachment 133407 [details] [diff] [review]
The correct V2

Right.

Comment 13

[Mark Lassau]

Maybe I'm too late, but I just realised that a mailto: for the "Assigned To"
field would also be useful.

Please let me know if I should log this as a separate enhancement request.

I realise that Bugzilla is primarily designed for mozilla.org and other Open
Source projects, and therefore encouraging private emails may not be desirable
for all installations.
This could be addressed by a config option turning mailto: on or off.

Please also see bug 223507 for another suggested mailto: enhancement.

Comment 14

[Dave Miller [:justdave]]

eh, why not?  :)
We can always revisit this when we redo the UI if people don't like it.

Comment 15

[Vlad Dascalu]

Checking in template/en/default/bug/edit.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/edit.html.tmpl,v  <--
 edit.html.tmpl
new revision: 1.35; previous revision: 1.34
done