Community
Participate
Working Groups
Build ID: M20060118-1600 Steps To Reproduce: 1. Create a section and assign a shortcut: final Section addressSection = toolkit.createSection(this, SWT.NONE); addressSection.setText("&Adresse"); 2. Add a text field to the section final Composite nameComposite = toolkit.createComposite(addressComposite, SWT.NONE); toolkit.createLabel(nameComposite, "Name", SWT.NONE); final Text nameText = toolkit.createText(nameComposite, null, SWT.NONE); 3. Run this in a Eclipse 3.0 dialog. Pressing the shortcut key ALT-A moves the focus to the text input file nameText. 4. Run this in a Eclipse 3.1 dialog. Pressing the shortcut key ALT-A hides the section! More information: In the ExpandableComposite which is the base class of Section, there is always a TraverseListener added to the object. This TraverseListener always toggles the state of the section, even if isFixedStyle() returns true. To restore the old behaviour, simply call programmaticToggleState() only if isFixedSytle() returns false.
Created attachment 78409 [details] Proposed fix for Eclipse 3.3 A small patch implementing the solution suggested in the bug description.
Adam, can you review this patch?
The patch is fine. Setting a mnemonic on a non-expandable section is a slightly odd use case, but it does let you focus on it with a keyboard shortcut. Regardless, we should certainly prevent the section from being collapsed in this case. +1
Patch committd to HEAD.