Bug 197745 - [IRC] Hyperlink support for IRC channels within the IRC UI
Summary: [IRC] Hyperlink support for IRC channels within the IRC UI
Status: CLOSED FIXED
Alias: None
Product: ECF
Classification: RT
Component: ecf.ui (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: ecf.core-inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: bugday, contributed, helpwanted
Depends on:
Blocks:
 
Reported: 2007-07-25 00:35 EDT by Chris Aniszczyk CLA
Modified: 2008-05-18 19:35 EDT (History)
3 users (show)

See Also:


Attachments
Patch to add chat room hyperlink (10.37 KB, patch)
2007-08-02 12:15 EDT, Abner Ballardo CLA
no flags Details | Diff
Patch to add chat room hyperlink (update) (9.61 KB, patch)
2007-08-02 16:22 EDT, Abner Ballardo CLA
no flags Details | Diff
Patch (10.16 KB, patch)
2008-03-30 20:59 EDT, Abner Ballardo CLA
no flags Details | Diff
mylyn/context/zip (3.04 KB, application/octet-stream)
2008-03-30 21:00 EDT, Abner Ballardo CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Aniszczyk CLA 2007-07-25 00:35:04 EDT
For example, say you see this in the IRC UI:

(23:31:10) irc.freenode.net: whois pombreda
(23:31:10) irc.freenode.net: pombreda@c-71-204-153-28.hsd1.ca.comcast.net
(23:31:10) irc.freenode.net: channels: #fedora-java #gentoo-java #trac #seagull #udig #weblogic-eclipse #jfire #rssowl #jbosstools #eclipse-ecf #phpeclipse #eclipse-dev #easyeclipse #eclipse #eclipse-commits 

It would be nice to ctrl+click those and join a channel within your existing connection.

I'm marking this as bugday but this one may be a bit tricky.
Comment 1 Abner Ballardo CLA 2007-08-02 12:15:32 EDT
Created attachment 75243 [details]
Patch to add chat room hyperlink

Eclipse Bug Day is over but I want to continue to contribute to ECF. BTW, this patch is for plugins: org.eclipse.ecf.presence.ui, org.eclipse.ecf.provider.irc and
org.eclipse.ecf.provider.irc.ui

The hard part was to find a way to get a reference to ChatRoomManagerView and IRCRootContainer. I've used extension org.eclipse.ui.workbench.texteditor.hyperlinkDetectorTargets and IRCChannelContainer.getAdapter(), but I'm not sure this is the best solution.

If there is a better way, please, tell me and I'll fix it.
Comment 2 Abner Ballardo CLA 2007-08-02 12:28:36 EDT
I've just notice that chat room hyperlink only works in channel tabs, I'll update the patch to support hyperlink in server tab.
Comment 3 Abner Ballardo CLA 2007-08-02 16:22:32 EDT
Created attachment 75267 [details]
Patch to add chat room hyperlink (update)

With this patch the hyperlink support works in channel tabs y server tab, it's a little different that the previous patch.
Comment 4 Remy Suen CLA 2008-03-25 23:08:24 EDT
This hyperlink would theoretically show up for all chatrooms (which would include non-IRC ones). Is there a way to get around this?
Comment 5 Abner Ballardo CLA 2008-03-26 01:10:21 EDT
(In reply to comment #4)
> This hyperlink would theoretically show up for all chatrooms (which would
> include non-IRC ones). Is there a way to get around this?

I think there should be a way. I'll work on it.
Comment 6 Abner Ballardo CLA 2008-03-30 20:59:59 EDT
Created attachment 94158 [details]
Patch

Patch updated.

The targetId is based on provider, so now the hyperlink will only show up in IRC Chatrooms.
Comment 7 Abner Ballardo CLA 2008-03-30 21:00:03 EDT
Created attachment 94159 [details]
mylyn/context/zip
Comment 8 Scott Lewis CLA 2008-03-31 13:20:51 EDT
Testing patch 94158.
Comment 9 Scott Lewis CLA 2008-03-31 13:27:46 EDT
Applied patch 94158, tested, updated IP log.  Thanks very much Abner!  It's highly appreciated!


Comment 10 Abner Ballardo CLA 2008-04-01 00:47:44 EDT
 (In reply to comment #9)
> Applied patch 94158, tested, updated IP log.  Thanks very much Abner!  It's
> highly appreciated!

thanks for your words :D

I've just notice that I forgot to add the copyright & license comment at the beginning of IRCChannelHyperlinkDetector and IRCChannelHyperlink classes.
Comment 11 Scott Lewis CLA 2008-04-01 10:40:55 EDT
(In reply to comment #10)
>  (In reply to comment #9)
> > Applied patch 94158, tested, updated IP log.  Thanks very much Abner!  It's
> > highly appreciated!
> 
> thanks for your words :D
> 
> I've just notice that I forgot to add the copyright & license comment at the
> beginning of IRCChannelHyperlinkDetector and IRCChannelHyperlink classes.
> 

If you put the desired copyright on this bug as a patch I will apply it to these two classes.
Comment 12 Scott Lewis CLA 2008-05-18 19:35:35 EDT
closing.