Summary: | [Cell Editors] DialogCellEditor in the PropertySheet bases its width on its text rather than its cell's width | ||
---|---|---|---|
Product: | [Eclipse Project] Platform | Reporter: | Joe Winchester <Winchest> |
Component: | UI | Assignee: | Randy Giffen <Randy_Giffen> |
Status: | VERIFIED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | P3 | CC: | n.a.edgar |
Version: | 2.0 | ||
Target Milestone: | 2.0.1 | ||
Hardware: | PC | ||
OS: | Windows 2000 | ||
Whiteboard: | |||
Bug Depends on: | 21829 | ||
Bug Blocks: |
Description
Joe Winchester
2002-06-17 06:42:09 EDT
There are two competing interests here: 1) The combo cell editor should use the width of its column so its button is always visible. 2) The editor must be wide enough for the items in its list. Currently the editor is sized to the widest item in its list (minimum 30). Given that 1) is the desired behavior we could: a) change the horizontal alignment of the cell editor to SWT.RIGHT. This will cause the button to always be aligned with the right hand column and the text to expand to the left as required for the longest item in the list. This looks a little weird as the text for the editor does not cleanly overlap the text for the property sheet value. b) Ask for api on CCombo to cause the list to always use its preferred width. It's a bit weird to see a combo with its list wider than its button and label, c) Make the changes for 1) only. The user can resize the column if the combo list is too narrow. My prefered solution is b) Filed 21829 to get new CCombo api from SWT Rereading the PR you state that the problem is with both DialogCellEditor and ComboCellEditor. I have not been able to recreate it with DialogCellEditor. The button is always visible regardless of the text label with. Can you verify that the problem in is only with ComboCellEditor in 2.0 Changed ComboBoxCellEditor The editor opens with the width of the column to a min of 50 pixels. Steve is releasing a change to CCombo to ensure that the list will appear with the required width. Released to R2_0_1 and HEAD SWT changes also released |