Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 452384 - [Hover] Provide HTML hover information for JS
Summary: [Hover] Provide HTML hover information for JS
Status: RESOLVED WONTFIX
Alias: None
Product: Orion
Classification: ECD
Component: JS Tools (show other bugs)
Version: 8.0   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Curtis Windatt CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 449240
Blocks:
  Show dependency tree
 
Reported: 2014-11-19 16:15 EST by Curtis Windatt CLA
Modified: 2015-05-08 10:53 EDT (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Curtis Windatt CLA 2014-11-19 16:15:23 EST 
Now that we have support for HTML in hovers, we should use HTML in the JS hovers over markdown so we get a lot more control over what it look like.  Fixed width fonts, better spacing, etc.
Comment 1 Curtis Windatt CLA 2014-11-19 16:23:32 EST 
http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=50a46704b642558daad61be9ef26f94e2d2e741b
Fixed in master

I'll leave this bug open for now as I should confirm how it looks on multiple browsers and there may be other tweaks we make.  I'm avoiding spending too much time fixing this up as we will likely move to a delegatedUI/UIPlugin style hover that is rendered by the JS plug-in.
Comment 2 Michael Rennie CLA 2014-11-20 11:20:13 EST 
For now I reverted this back to using markdown until we can figure out the resize / scrolling story for tooltips - as it was the HTML tooltip wrapped function signatures making it impossible to read.
Comment 3 Michael Rennie CLA 2015-05-08 10:53:31 EDT 
Now that Tern is handling our hovers we are only supporting markdown in our hovers. I'm going to close this one as wontfix.