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

Bug 313943

Summary: [hovering] Java source hovers not readable using Ubuntu 10.04 (white-on-black tooltips)
Product: [Eclipse Project] JDT Reporter: Markus Keller <markus.kell.r>
Component: TextAssignee: Markus Keller <markus.kell.r>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: bokowski, daniel_megert, grant_gayed, markus.kell.r, thatnitind
Version: 3.6Flags: daniel_megert: review+
Target Milestone: 3.6.1   
Hardware: PC   
OS: Linux-GTK   
Whiteboard:
Attachments:
Description Flags
Fix
none
Fix for 3.6.1
none
Screenshot none

Description Markus Keller CLA 2010-05-21 13:01:03 EDT
Created attachment 169526 [details]
Fix

I20100520-1744, follow-up to bug 312966

Java source hovers are not readable using Ubuntu 10.04 (which uses white-on-black tooltips by default), see e.g. attachment 168593 [details].

The fix is to use a better "System Default" if the brightness of the Java editor default text color is too close to the hover background color (resorting to the editor background color in that case).
Comment 1 Markus Keller CLA 2010-06-23 09:55:05 EDT
Fixed in HEAD.
Comment 2 Dani Megert CLA 2010-08-12 01:59:54 EDT
*** Bug 309334 has been marked as a duplicate of this bug. ***
Comment 3 Markus Keller CLA 2010-08-17 10:54:06 EDT
Created attachment 176793 [details]
Fix for 3.6.1
Comment 4 Markus Keller CLA 2010-08-17 10:59:57 EDT
Reopening for 3.6.1.
Comment 5 Dani Megert CLA 2010-08-19 02:40:37 EDT
+1 for 3.6.1 RC2 with a minor change: SourceViewerInformationControl.getVisibleBackgroundColor needs 3.6.1 @since tag and not 3.6. Fixed that in HEAD.
Comment 6 Markus Keller CLA 2010-08-23 07:06:48 EDT
> SourceViewerInformationControl.getVisibleBackgroundColor needs 3.6.1 @since tag
> and not 3.6.

Fixed for 3.6.1 including the @since tag fix.
Comment 7 Markus Keller CLA 2010-08-26 05:43:43 EDT
Verified all hovers on M20100825-0800 with Ubuntu 10.04.
Comment 8 Rajesh CLA 2010-08-26 06:14:53 EDT
Created attachment 177511 [details]
Screenshot


Verified that its no longer white on black but the new colors aren't great either :) See attached image.
Comment 9 Markus Keller CLA 2010-08-26 06:59:02 EDT
(In reply to comment #8)
> Created an attachment (id=177511)

This bug is about source hovers (hold Shift while hovering in the Java Editor) and change ruler hovers. Bug 313530 was about updating hover colors when they change in the OS.

If the native tooltips on your OS look different from our Javadoc hovers, please file a separate bug and attach screenshots of native hovers (e.g. from gEdit buttons) and of an SWT tooltip (e.g. on a toolbar button) for comparison.