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

Bug 463516

Summary:	[Hover] Style MarkeD rendered markdown better
Product:	[ECD] Orion	Reporter:	Curtis Windatt <curtis.windatt.public>
Component:	Client	Assignee:	Curtis Windatt <curtis.windatt.public>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3	CC:	Michael_Rennie
Version:	8.0
Target Milestone:	9.0
Hardware:	PC
OS:	Windows 7
Whiteboard:

Description Curtis Windatt

2015-03-30 16:05:59 EDT

Many tooltips contain rendered markdown content.  The markdown renderer, MarkeD uses <p> and <h3> tags which do not have the same margins/padding as other tooltip content.  We should use the tooltip.css to style the content more efficiently.

Comment 1 Curtis Windatt

2015-04-01 17:28:46 EDT

http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=59ba93b1f10f85fe0a9d3cb848dd519dbbd5c1a4

Reorganized the css file based on the type of tooltip the styles apply to.  Removed the leading whitespace on h3 which are used in the titles.

Comment 2 Curtis Windatt

2015-05-26 12:15:31 EDT

Other problematic cases:
1) Navigation links have long paths after the file name.  When the path is longer than width allows, the path is moved to the next line, but the tooltip has already been sized.
2) We have more hovers without <h3> titles now, so the extra margin above looks wrong.

Comment 3 Curtis Windatt

2015-05-26 16:34:57 EDT

http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=9c57013a313b73ca3d6a585c6374f8cc473c57d3
Fixed in master.

If you have a single line hover that should wrap (width of div is larger than the default tooltip width), we will set a minimum height of 50px.

The first <p> tag in the div will have its leading margin removed.

Fixed margins for firefox (MarkD was using a webkit specific style).

All <p> tags will now wrap within words rather than letting the long single lines extend past the tooltip border.