Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 335496 - [Help] In Help View too much vertical white space between Scope and Alternate Search Expression
Summary: [Help] In Help View too much vertical white space between Scope and Alternate...
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: User Assistance (show other bugs)
Version: 3.7   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.7 M7   Edit
Assignee: Chris Austin CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-26 14:08 EST by Chris Goldthorpe CLA
Modified: 2011-03-29 10:03 EDT (History)
1 user (show)

See Also:

Attachments
Screenshot with suggested change (22.46 KB, image/png)
2011-01-26 14:09 EST, Chris Goldthorpe CLA 		no flags Details
Patch to fix white space (1.72 KB, patch)
2011-03-29 10:01 EDT, Chris Austin 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 Chris Goldthorpe CLA 2011-01-26 14:08:59 EST 
I20110118-0800

In the search tab of the help view there is too much vertical space between Scope and Alternate Search Expression, this should be trimmed down.
Comment 1 Chris Goldthorpe CLA 2011-01-26 14:09:51 EST 
Created attachment 187670 [details]
Screenshot with suggested change
Comment 2 Chris Austin CLA 2011-01-26 14:15:01 EST 
This bug is actually a result of the Bug 280459 - the Remote IC error message link fits in that space.  I noticed this during work on the search processor, but I don't have a good fix for it.  The space seems to be allocated even if the link method setVisible(false) has been called.
Comment 3 Chris Austin CLA 2011-03-29 10:01:20 EDT 
Created attachment 192093 [details]
Patch to fix white space

This patch disposes the object when not in use, thus re-allocating its ui-space.
Comment 4 Chris Austin CLA 2011-03-29 10:03:16 EDT 
Patch committed to HEAD.