Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 332436 - [Combo] Refactor combo dropdown-list popup behavior
Summary: [Combo] Refactor combo dropdown-list popup behavior
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: 1.4   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.4 M5   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-12-13 08:42 EST by Tim Buschtoens CLA
Modified: 2010-12-15 08:52 EST (History)
1 user (show)

See Also:

Attachments
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 Tim Buschtoens CLA 2010-12-13 08:42:24 EST 
The combo-list is currently a basic-list with a hacky-solution to provide the popup-behavior.The peviously used method to use setCapture wasn't usable anymore due to the new scrollbar.  (See Bug 332413.) 

There arealso very few client-tests for the combo.
Comment 1 Tim Buschtoens CLA 2010-12-13 08:56:25 EST 
The issue of Bug 329907 should also be considered.
Comment 2 Tim Buschtoens CLA 2010-12-13 09:11:42 EST 
Bug 332413 also created a new issue: Scrolling the list, then clicking on the dropdown-button does not hide the list. (It disappears and immediately reappears.)
Comment 3 Tim Buschtoens CLA 2010-12-15 08:52:09 EST 
Fixed in CVS HEAD.

The new solution is based on re-dispatching events after capturing them in combo, so the scrollbar still works. Its a bit hacky in some aspects, but much less so than the previous solution.