Summary: | Do not set layout data for search page in SearchDialog.createPageControl | ||
---|---|---|---|
Product: | [Eclipse Project] Platform | Reporter: | Benno Baumgartner <benno.baumgartner> |
Component: | Search | Assignee: | Platform-Search-Inbox <platform-search-inbox> |
Status: | NEW --- | QA Contact: | |
Severity: | normal | ||
Priority: | P3 | ||
Version: | 3.3 | ||
Target Milestone: | --- | ||
Hardware: | PC | ||
OS: | Windows XP | ||
Whiteboard: |
Description
Benno Baumgartner
2007-06-22 06:25:09 EDT
The search page can't know what layout the container uses. We would have to spec this. Of course we could add this as a hidden feature, that wen the control already has a grid layout set, we keep this. But I'm not sure if this wouldn't break existing implementations that happen to have added a buggy layout data. (In reply to comment #1) > The search page can't know what layout the container uses. That's why it should not make assumptions about what the container needs. > We would have to > spec this. Of course we could add this as a hidden feature, that wen the > control already has a grid layout set, we keep this. But I'm not sure if this > wouldn't break existing implementations that happen to have added a buggy > layout data. > We could also add an optional attribute to org.eclipse.search.searchPages page extension, like grapExcessiveSpace true|false, with false as default Of course changing the line to page.getControl().setLayoutData(new GridData(SWT.FILL, SWT.FILL, true, true)); would work too, at least for me;-) This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant. |