Summary: | The Load target button in the PDE preferences page for Target Platform is not enabled correctly | ||||||
---|---|---|---|---|---|---|---|
Product: | [Eclipse Project] PDE | Reporter: | Félix Cuadrado <fcuadrado> | ||||
Component: | UI | Assignee: | Brian Bauman <baumanbr> | ||||
Status: | RESOLVED FIXED | QA Contact: | |||||
Severity: | normal | ||||||
Priority: | P3 | CC: | grant_gayed, wassim.melhem | ||||
Version: | 3.3 | Keywords: | contributed | ||||
Target Milestone: | 3.3 M5 | ||||||
Hardware: | PC | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Bug Depends on: | 170215 | ||||||
Bug Blocks: | |||||||
Attachments: |
|
Description
Félix Cuadrado
2006-12-20 07:53:58 EST
Created attachment 55960 [details]
Patch for TargetPlatformPreferencesPage fixing the error
Grant, is this a bug in SWT? ie. a bug in the handling of ModifyEvent on Linux. or should we proceed with the patch that adds yet another listener on the combo box? This is an swt bug, so I've logged bug 170215. However you may wish to apply the attached patch in the meantime, as I'm not sure when this bug will be addressed, and setting the enabled state on the Load Profile button twice should not be a problem. I can take this bug. I looked into this quickly when it was first opened. I believe we can remove the selection listener completely if we use a modify listener. If so, I would rather have one listener instead of two. brian, we already have a modify listener. the patch adds a selection listener. ok, then reverse what I said :) I get confused first thing in the morning Felix, thank you very much for the patch! I released a slightly modified version (removed the ModifyListener and extended SelectionAdapter instead of implementing ISelectionListener). Should be available in 3.3M5. |