This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 239729 - add option to copy commenter's email address out of task editor
Summary: add option to copy commenter's email address out of task editor
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: dev   Edit
Hardware: All All
: P4 enhancement (vote)
Target Milestone: 3.1   Edit
Assignee: Steffen Pingel CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-07-06 13:56 EDT by Andreas Goetz CLA
Modified: 2009-01-29 15:53 EST (History)
4 users (show)

See Also:


Attachments
patch (5.37 KB, patch)
2008-07-09 16:24 EDT, Frank Becker CLA
no flags Details | Diff
mylyn/context/zip (5.99 KB, application/octet-stream)
2008-07-09 16:24 EDT, Frank Becker CLA
no flags Details
updated patch (7.40 KB, patch)
2008-12-18 14:44 EST, Frank Becker CLA
no flags Details | Diff
mylyn/context/zip (6.57 KB, application/octet-stream)
2008-12-18 14:44 EST, Frank Becker CLA
no flags Details
screenshot (5.98 KB, image/png)
2009-01-07 00:28 EST, Steffen Pingel CLA
no flags Details
updated patch (6.82 KB, patch)
2009-01-08 17:02 EST, Frank Becker CLA
no flags Details | Diff
mylyn/context/zip (7.26 KB, application/octet-stream)
2009-01-08 17:02 EST, Frank Becker CLA
no flags Details
chevron.gif (202 bytes, image/gif)
2009-01-19 15:34 EST, Frank Becker CLA
no flags Details
path with chevron (11.42 KB, patch)
2009-01-19 15:36 EST, Frank Becker CLA
no flags Details | Diff
mylyn/context/zip (12.96 KB, application/octet-stream)
2009-01-19 15:36 EST, Frank Becker CLA
no flags Details
updated patch with chevron (10.57 KB, patch)
2009-01-26 15:05 EST, Frank Becker CLA
no flags Details | Diff
mylyn/context/zip (12.88 KB, application/octet-stream)
2009-01-26 15:05 EST, Frank Becker CLA
no flags Details
implementation that uses toolbar instead of hyperlink (11.90 KB, patch)
2009-01-27 00:37 EST, Steffen Pingel CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andreas Goetz CLA 2008-07-06 13:56:26 EDT
Use case: copy email address of a commenter to add him on CC to another bug. Have not found any way to do this. Was looking for right-click and context menu e.g. in the outline view or the CC box.
Comment 1 Frank Becker CLA 2008-07-09 16:24:36 EDT
Created attachment 107005 [details]
patch

this patch is not finished.

ToDo is to generate an copy EmailAdress gif and change
Comment 2 Frank Becker CLA 2008-07-09 16:24:39 EDT
Created attachment 107006 [details]
mylyn/context/zip
Comment 3 Frank Becker CLA 2008-12-18 14:44:35 EST
Created attachment 120880 [details]
updated patch

This is the updated version of patch from comment#2
Comment 4 Frank Becker CLA 2008-12-18 14:44:39 EST
Created attachment 120881 [details]
mylyn/context/zip
Comment 5 Steffen Pingel CLA 2009-01-07 00:28:37 EST
Created attachment 121744 [details]
screenshot
Comment 6 Steffen Pingel CLA 2009-01-07 00:31:18 EST
Thanks for the patch Frank. I find that the additional icon looks a bit noisy for an operation that I would expect to get used infrequently. Maybe a context menu on the comment header if that is possible would work better in this case?
Comment 7 Frank Becker CLA 2009-01-08 17:02:29 EST
Created attachment 122021 [details]
updated patch

I implement the using Command Click to copy the mal adress.

Is this OK?
Comment 8 Frank Becker CLA 2009-01-08 17:02:32 EST
Created attachment 122022 [details]
mylyn/context/zip
Comment 9 Steffen Pingel CLA 2009-01-10 16:10:47 EST
That's an interesting approach. I am afraid though the copy action would be very hard to discover. I have put this on the agenda for the next conference call to gather some ideas how to do the user interaction. 
Comment 10 Frank Becker CLA 2009-01-11 08:20:20 EST
(In reply to comment #9)
> That's an interesting approach. I am afraid though the copy action would be
> very hard to discover. I have put this on the agenda for the next conference
> call to gather some ideas how to do the user interaction. 
> 

What's about to change the tooltip text from "emailadress" to "use command + click to copy \"emailadress\" to the clipbord"
Comment 11 Steffen Pingel CLA 2009-01-13 22:22:10 EST
Mik suggested to add a chevron to the reply button, similar to the task list working set chooser or the Run toolbar button in Eclipse. That way the editor would still only show a single button per comment which would default to Reply but other actions would be easily discoverable. What do you think about that Frank?
Comment 12 Eugene Kuleshov CLA 2009-01-15 13:07:23 EST
With bug 203798 appear to be fixed, why don't you use object contribution for this? There are other useful actions could be added comment text and/or commenter email.
Comment 13 Frank Becker CLA 2009-01-19 15:34:17 EST
Created attachment 122989 [details]
chevron.gif

put this file in org.eclipse.mylyn.tasks.ui.icons.etool16
Comment 14 Frank Becker CLA 2009-01-19 15:36:33 EST
Created attachment 122991 [details]
path with chevron

see comment#11
Comment 15 Frank Becker CLA 2009-01-19 15:36:37 EST
Created attachment 122992 [details]
mylyn/context/zip
Comment 16 Steffen Pingel CLA 2009-01-19 16:21:56 EST
That's great stuff! I think we should keep the reply action like it is now and add the chevron that you implemented next to it. That way, the frequently used reply action is accessible through a single click and additional actions can be contributed to the popup menu.
Comment 17 Frank Becker CLA 2009-01-26 15:05:47 EST
Created attachment 123801 [details]
updated patch with chevron

reply hyperlink is now between title and chevron
Comment 18 Frank Becker CLA 2009-01-26 15:05:50 EST
Created attachment 123802 [details]
mylyn/context/zip
Comment 19 Steffen Pingel CLA 2009-01-27 00:37:30 EST
Created attachment 123852 [details]
implementation that uses toolbar instead of hyperlink

Great work Frank. I have modified your patch slightly to use a toolbar instead of hyperlink. It looks reasonable on GTK/Linux and I find that it has slighlty better user interaction.

Leo, could you apply this patch on Carbon and Cocoa to check if it also works on Mac?
Comment 20 Frank Becker CLA 2009-01-27 16:59:05 EST
(In reply to comment #19)
> Created an attachment (id=123852)
> implementation that uses toolbar instead of hyperlink
> 
> Great work Frank. I have modified your patch slightly to use a toolbar instead
> of hyperlink. It looks reasonable on GTK/Linux and I find that it has slighlty
> better user interaction.
> 
> Leo, could you apply this patch on Carbon and Cocoa to check if it also works on
> Mac?

I dit the test on Carbon and Cocoa both wit 3.5M4 and find that this looks much better.
Comment 21 Steffen Pingel CLA 2009-01-27 17:47:51 EST
Thanks a lot Frank! I'll do some clean up of the patch and apply it.
Comment 22 Steffen Pingel CLA 2009-01-28 01:13:03 EST
I have committed a new implementation that uses a toolbar and registers an extensible popup menu with the workbench. I still need to externalize the strings once we have agreed on labels and actions that should be contributed to the menu.
Comment 23 Frank Becker CLA 2009-01-28 17:12:39 EST
(In reply to comment #22)
> I have committed a new implementation that uses a toolbar and registers an
> extensible popup menu with the workbench. I still need to externalize the
> strings once we have agreed on labels and actions that should be contributed to
> the menu.

COMMENT_CHEVRON and COMMENT_CHEVRON_TOOLTIP is not used and can be removed.
Comment 24 Steffen Pingel CLA 2009-01-29 15:53:14 EST
Code cleaned up, reply action added to menu and copy action renamed to "Copy User ID" as per disussion on today's call.