Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 323827 - Wrong colour on parent composite in the error log view
Summary: Wrong colour on parent composite in the error log view
Status: RESOLVED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.6   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 3.7 M2   Edit
Assignee: Ankur Sharma CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-27 10:13 EDT by Michael Rennie CLA
Modified: 2010-09-07 15:56 EDT (History)
4 users (show)

See Also:

Attachments
screen shot (24.76 KB, image/png)
2010-08-27 10:13 EDT, Michael Rennie CLA 		no flags Details
Patch (902 bytes, patch)
2010-08-30 10:35 EDT, Ankur Sharma 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 Michael Rennie CLA 2010-08-27 10:13:10 EDT 
Created attachment 177617 [details]
screen shot

I20100824-1210 and 3.6.1

In the error log view if you turn on "show text filter" you will notice that the background of the composite that holds the filter widget is also white. I would expect it to be the standard tab colour.

The attached screen shot shows the problem.
Comment 1 Ankur Sharma CLA 2010-08-30 02:46:19 EDT 
That's some eye for details :)

Forwarding to Platform UI.
Comment 2 Prakash Rangaraj CLA 2010-08-30 03:19:08 EDT 
Michael,
     Do you see this in only in Error Log view or in all the filtered trees (like the workbench New Wizards/Show View dialog, etc?
Comment 3 Michael Rennie CLA 2010-08-30 10:03:18 EDT 
(In reply to comment #2)
> Michael,
>      Do you see this in only in Error Log view or in all the filtered trees
> (like the workbench New Wizards/Show View dialog, etc?

I have only noticed it in the Error Log view

> Forwarding to Platform UI.

Isn't the Error Log view owned by PDE UI?
Comment 4 Ankur Sharma CLA 2010-08-30 10:10:35 EDT 
Yes. I was horribly mistaken. I have the fix too now.
Comment 5 Ankur Sharma CLA 2010-08-30 10:35:47 EDT 
Created attachment 177731 [details]
Patch
Comment 6 Ankur Sharma CLA 2010-08-30 10:40:19 EDT 
Fixed in HEAD
Comment 7 Dani Megert CLA 2010-08-31 02:45:18 EDT 
Patch misses update of MANIFEST.MF.
Comment 8 Curtis Windatt CLA 2010-09-07 15:56:59 EDT 
Bundle version updated to 1.0.200.qualifier in the manifest.  I assume that was what Dani was referring to.