Bug 20887

Summary: Topic tree collapse behavior
Product: [Eclipse Project] Platform Reporter: Knut Radloff <knut_radloff>
Component: User AssistanceAssignee: Dorian Birsan <birsan>
Status: RESOLVED FIXED QA Contact:
Severity: major    
Priority: P3 Keywords: readme
Version: 2.0   
Target Milestone: 2.0.1   
Hardware: PC   
OS: Windows 2000   
Whiteboard:
Attachments:
Description Flags
toc.js to replace existing one in org.eclipse.help.webapp none

Description Knut Radloff CLA 2002-06-24 11:32:29 EDT
build 20020620

The topics tree view in the help browser has erratic behavior when collapsing 
items with the mouse.
To reproduce:
-expand enough tree items so that some bottom items are no longer visible
-select a topic item at the bottom of the tree
-scroll the tree so that the selected item is no longer visible
-click on the - icon to collapse an item
->the collapse will not work and the tree will instead scroll to show the 
selected item.

This is highly annoying and makes the topic browser hard to use. To work around 
I have to select the item that I want to collapse and then click the - icon.
Comment 1 Dorian Birsan CLA 2002-06-24 11:42:20 EDT
Thanks for finding this one, we have never seen it, but I can certainly 
reproduce it.

BTW, a simpler workaround is to double click on the node you want to collapse.
Comment 2 Dorian Birsan CLA 2002-06-24 14:59:05 EDT
 Clicking on the minus icon (which is an <IMG> html element) triggers a click 
event in IE. The source of the event is the <IMG> node when the selected child 
topic is visible, but it is the containing <LI> node when the selected child 
topic is not visible (as described in the bug description).
It looks like this is a bug in IE and we need to code a workaround for it.
BTW, things work fine on Linux with Mozilla.
Comment 3 Dorian Birsan CLA 2002-06-25 15:45:37 EDT
Created attachment 1587 [details]
toc.js to replace existing one in org.eclipse.help.webapp
Comment 4 Dorian Birsan CLA 2002-06-25 15:45:53 EDT
After some more investigation, it turns out the problem is caused by the focus 
event being sent before the click. I have a fix for the problem, but the defect 
is not approved for 2.0

Download the attached toc.js file, and copy it to 
plugins\org.eclipse.help.webapp_2.0.0
Comment 5 Dorian Birsan CLA 2002-06-25 16:13:39 EDT
This bug will go into README.
Comment 6 Knut Radloff CLA 2002-06-25 16:22:56 EDT
Thanks, the fix works for me.
Comment 7 Dorian Birsan CLA 2002-07-08 10:04:54 EDT
fixed.
Code is checked in HEAD
Comment 8 Jim des Rivieres CLA 2002-08-21 11:31:21 EDT
Is this fixed in 2.0.1 stream?
Comment 9 Konrad Kolosowski CLA 2002-08-21 11:46:40 EDT
Yes, it is fixed in 2.0.1.
The fix was relesed in head, and we split streams to create 2.0.1 branch later.