Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 337288 - Push result dialog should render hyperlinks as links
Summary: Push result dialog should render hyperlinks as links
Status: RESOLVED FIXED
Alias: None
Product: EGit
Classification: Technology
Component: UI (show other bugs)
Version: 0.12   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 4.1   Edit
Assignee: Benjamin Muskalla CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-16 04:45 EST by Matthias Sohn CLA
Modified: 2015-09-07 02:52 EDT (History)
2 users (show)

See Also:


Attachments
Screenshot from version 1.2.0 with text instead of link (74.23 KB, image/jpeg)
2012-01-12 08:53 EST, Mathias Kinzler CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Matthias Sohn CLA 2011-02-16 04:45:40 EST
This would be nice for use with Gerrit which responds on new changes with the hyperlink where the change can be reviewed. Would be nice if this would be rendered as a link I can click to go there.
Comment 1 Benjamin Muskalla CLA 2011-08-10 08:22:08 EDT
Pushed review: http://egit.eclipse.org/r/#change,3909
Comment 2 Matthias Sohn CLA 2011-08-10 17:22:17 EDT
merged as b5310e23b9f52acf875dde5e733718e8f21982b7 

thanks Benny
Comment 3 Mathias Kinzler CLA 2012-01-12 08:52:52 EST
I believe this got lost with release 1.2.0. I'm attaching a screenshot from the push result dialog where the link under "New Changes" is rendered as text.
Comment 4 Mathias Kinzler CLA 2012-01-12 08:53:48 EST
Created attachment 209376 [details]
Screenshot from version 1.2.0 with text instead of link
Comment 5 Mykola Nikishov CLA 2012-03-03 11:12:33 EST
[Batch change] Remove passed Target Milestones

If anyone on CC list is going to fix/implement this, feel free to assign a new, post-1.3/2.0, target milestone.
Comment 6 Eclipse Genie CLA 2015-09-06 12:42:11 EDT
New Gerrit change created: https://git.eclipse.org/r/55368
Comment 7 Eclipse Genie CLA 2015-09-07 02:52:00 EDT
Gerrit change https://git.eclipse.org/r/55368 was merged to [master].
Commit: http://git.eclipse.org/c/egit/egit.git/commit/?id=c715f4fcedca66cde53553c78528fc828d8ab194
Comment 8 Matthias Sohn CLA 2015-09-07 02:52:24 EDT
submitted patch