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

Bug 457747

Summary:	The context object for the hover callback should include the content type
Product:	[ECD] Orion	Reporter:	Michael Rennie <Michael_Rennie>
Component:	Client	Assignee:	Project Inbox <orion.client-inbox>
Status:	RESOLVED WONTFIX	QA Contact:
Severity:	normal
Priority:	P3	CC:	curtis.windatt.public, emoffatt
Version:	8.0
Target Milestone:	---
Hardware:	PC
OS:	Windows 7
Whiteboard:
Bug Depends on:
Bug Blocks:	457749

Description Michael Rennie

2015-01-16 14:13:01 EST

The context passed in to the computeHoverInfo function should have the content type of the editor the hover will show in. This would allow clients to contribute one kind of hover to more than one content type. 

For example in the JS tools the hover code is identical for HTML (script blocks) vs JavaScript (compilation unit), but I need to switch on the content type to know to look for and process script nodes vs an entire file. We already do this for the validation callbacks.

Comment 1 Michael Rennie

2015-01-16 14:30:41 EST

I'm going to close this - I forgot we added API to EditorContext to get file metadata: EditorContext#getFileMetadata

This will give us the content type id.

Comment 2 Michael Rennie

2015-01-16 15:11:05 EST

resolving wontfix