Bug 205130 - Expanding/Collapsing tree nodes in Quick Windows closes window
Summary: Expanding/Collapsing tree nodes in Quick Windows closes window
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 3.3   Edit
Hardware: Macintosh Mac OS X - Carbon (unsup.)
: P3 major (vote)
Target Milestone: 3.4 M3   Edit
Assignee: Steve Northover CLA
QA Contact:
URL:
Whiteboard:
Keywords: needinfo
Depends on:
Blocks:
 
Reported: 2007-10-01 18:04 EDT by Andre Weinand CLA
Modified: 2007-10-04 03:18 EDT (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andre Weinand CLA 2007-10-01 18:04:42 EDT
- on Mac open the Quick Outline on some Java CU
- expand/collapse any node wioth children

Observe: a single click expands/collapses the node *and* closes the window.

In addition single clicking on any tree item closes the window too (I'm not sure whether this is the correct behavior).
Comment 1 Dani Megert CLA 2007-10-02 07:02:52 EDT
Which build?
Did this work  before (e.g. 3.2)?

Do you click exactly on the +/-? We had a similar bug on Linux but that was long ago.

>In addition single clicking on any tree item closes the window too (I'm not
>sure whether this is the correct behavior).
This is the intended behavior and like that for years now.
Comment 2 Dani Megert CLA 2007-10-02 07:28:37 EDT
The Linux bug was bug 54499. Changes in TreeItem.getItem(Point) might cause this. Moving to SWT as we did not change that code for years.
Comment 3 Dani Megert CLA 2007-10-03 03:18:28 EDT
Verified that it works correctly on Linux GTK and WindowsXP, so it's definitely something that just changed on the Mac.
Comment 4 Andre Weinand CLA 2007-10-03 12:46:44 EDT
I'm seeing the problem in Eclipse 3.3, but it is a long-standing issue and not a recent regression.

[I always wanted to file a bug for it, but the effort to write the bug report was always bigger than living with  the bug :-) The severity "major" reflects the sum of 50 "minors". I've corrected it to "normal"]
Comment 5 Steve Northover CLA 2007-10-03 14:53:27 EDT
Fixed > 20071003

You should really be using a selection listener rather than doing your own selection in mouse up (although I'm sure there is a good reason you are not doing this).
Comment 6 Dani Megert CLA 2007-10-04 03:18:36 EDT
>(although I'm sure there is a good reason you are not doing this).
So it is ;-)
We mimic the workbench's single-click open strategy which opens the element on mouse up and not on mouse down (to be technically correct it's done on post selection).