Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 317866 - Hovers don't look right when there are multiple occurrences
Summary: Hovers don't look right when there are multiple occurrences
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: PDT (show other bugs)
Version: unspecified   Edit
Hardware: Macintosh Mac OS X
: P2 minor (vote)
Target Milestone: ---   Edit
Assignee: xu jiaxi CLA
QA Contact: Ilina Stefanova CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-24 13:52 EDT by Jacob Weber CLA
Modified: 2020-05-14 11:09 EDT (History)
3 users (show)

See Also:

Attachments
Hovering over the Debug class (5.10 KB, image/gif)
2010-06-24 13:52 EDT, Jacob Weber CLA 		no flags Details
patch (4.88 KB, patch)
2010-12-06 02:38 EST, xu jiaxi CLA 		no flags Details | Diff
View All 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 Jacob Weber CLA 2010-06-24 13:52:20 EDT 
When I hover over a class name, and there are multiple classes with that name, the size of the hover isn't wide enough to fit its contents, even though its contents are very narrow.

Also, the hover shows the class name twice, but doesn't provide any information about either one. It would be more useful to either show the description/PHPDoc from both versions, show just the first version, or at least show links to both versions.

See attached screen shot.



-- Configuration Details --
Product: Eclipse 1.3.0.20100617-0521 (org.eclipse.epp.package.jee.product)
Installed Features:
 org.eclipse.platform 3.6.0.v20100602-9gF78GpqFt6trOGhL60z0oEx3fz-JKNwxPY
Comment 1 Jacob Weber CLA 2010-06-24 13:52:53 EDT 
Created attachment 172655 [details]
Hovering over the Debug class
Comment 2 xu jiaxi CLA 2010-12-06 02:38:49 EST 
Created attachment 184576 [details]
patch

We will display every version of the classes and split them by "<hr>" tag
Comment 3 Zhongwei Zhao CLA 2011-01-13 00:39:20 EST 
when we have following code:
$a = new Test();

we hover at Test,the hover will show:
Test
Test::Test
()
Test::test
()
the width is still not wide enough,make small workaround to the patch.
Comment 4 Kalin CLA 2011-07-29 08:14:12 EDT 
Closing...