Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 447586

Summary:	Polish the Hover for Orion
Product:	[ECD] Orion	Reporter:	Eric Moffatt <emoffatt>
Component:	Client	Assignee:	Eric Moffatt <emoffatt>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3	CC:	aboyko, curtis.windatt.public, Michael_Rennie
Version:	unspecified
Target Milestone:	8.0
Hardware:	PC
OS:	Windows 7
Whiteboard:
Bug Depends on:
Bug Blocks:	450152

Description Eric Moffatt

2014-10-16 11:04:32 EDT

We need to clean up the hover implementation (and docs) to bring it up to the state it needs to be in for the Orion 7 release.

This includes at least:

- normalizing the parameters (i.e. 'title' and 'content' should be the returned fields regardless of the 'type')

- removing the 'tipTitle'

- Fixing any errors (for example the current JSDoc hover fails if the JSDoc is in a state where it fails to parse...)

- Removing the 'proposal' type from the docs (but we can keep the code for now)

- anything else we find...

Comment 1 Eric Moffatt

2014-10-16 15:52:55 EDT

Here's the Gerrit link for the first pass...

https://git.eclipse.org/r/#/c/35005/

Comment 2 Eric Moffatt

2014-10-24 12:39:19 EDT

The code for this is in...

Comment 3 Alex Boyko

2014-10-29 13:04:25 EDT

Previously I could pass straight HTML to tooltip via title property of the defferedInfo, but now this has been removed, which is OK - it looked like a bug anyway. However, we'd still like some support to pass HTML content to a tooltip (Javadoc for example). Would you be able to add support for HTML as well in tooltip.js?

Comment 4 Curtis Windatt

2014-10-29 15:16:19 EDT

(In reply to Alex Boyko from comment #3)
> Previously I could pass straight HTML to tooltip via title property of the
> defferedInfo, but now this has been removed, which is OK - it looked like a
> bug anyway. However, we'd still like some support to pass HTML content to a
> tooltip (Javadoc for example). Would you be able to add support for HTML as
> well in tooltip.js?

I opened this as Bug 449240.