| Summary: | Wikitext link to task opens in browser | ||||||
|---|---|---|---|---|---|---|---|
| Product: | z_Archived | Reporter: | Sam Davis <sam.davis> | ||||
| Component: | Mylyn | Assignee: | Steffen Pingel <steffen.pingel> | ||||
| Status: | RESOLVED FIXED | QA Contact: | |||||
| Severity: | minor | ||||||
| Priority: | P3 | CC: | greensopinion, steffen.pingel | ||||
| Version: | unspecified | ||||||
| Target Milestone: | 1.6.0 | ||||||
| Hardware: | All | ||||||
| OS: | All | ||||||
| Whiteboard: | |||||||
| Attachments: |
|
||||||
|
Description
Sam Davis
David, how does WikiText open URLs? With the changes for bug 346046 we could easily open URLs in the corresponding rich editors if BrowserUtil.openUrl() is used. Wow, that makes things much easier. I've pushed a change to Gerrit for review: http://review.mylyn.org/#change,240 Created attachment 210468 [details]
mylyn/context/zip
The change looks good to me. Only consideration could be the dependency on Mylyn 3.7 that we removed some time ago. If it's okay to add that back we should also consider bug 365512. I have opened bug 370504 to make the URL handler API work for tasks, currently it only works for links to Hudson builds and certain links to Gerrit reviews. On second thought, this should already work. Since WikiText is using the TasksUi.openUrl() method we should recognize those as task URLs already so this shouldn't require any changes in WikiText. I'll take a look why this isn't working. David, the easiest fix would be to change the call from TasksUiUtil.openUrl() to TasksUiUtil.openTask(). The latter will fallback to BrowserUtil.openUrl() if it's not a task URL and the API is also available on 3.6. with bug 365512 implemented, this bug is fixed. Thanks for your contribution Steffen. |