Bug 120806 - [Help] Closing the Book by using the left arrow causes that the focus jumps to the frame (top) of the navigation tree frame
Summary: [Help] Closing the Book by using the left arrow causes that the focus jumps ...
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: User Assistance (show other bugs)
Version: 3.1.1   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.3 M1   Edit
Assignee: platform-ua-inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2005-12-14 01:48 EST by Jiang Lin Quan CLA
Modified: 2007-06-05 19:30 EDT (History)
0 users

See Also:


Attachments
Patch for this bug (6.27 KB, patch)
2005-12-28 22:49 EST, Jiang Lin Quan CLA
no flags Details | Diff
Patch for this bug (7.27 KB, patch)
2005-12-29 02:28 EST, Jiang Lin Quan CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jiang Lin Quan CLA 2005-12-14 01:48:13 EST
Reporter: Jiang Lin Quan
JRE Version:  SUN JRE 1.4.2_07
Browser: IE 6
OS: windows XP professional
Eclipse version:3.1.0

<<Steps To Reproduce>>     
     1, Launch the Help.
     2, Use the keyboard is able to expand the top-level items in the navigation tree(the Book icons) using the right and left arrows.
        
<<Actual Result>>
   When the user clicks the Right arrow to expand it, it expands and focus is kept on the heading with the Book icon. 
   But if the user then closes the Book by using the left arrow, the focus jumps to the frame (top) of the navigation
   tree frame instead of remaining on the heading for the Book icon.

<<Expected Result>>
   When the user clicks the Right arrow to expand it, it expands and focus is kept on the heading with the Book icon. 
   When the user then closes the Book by using the left arrow, the focus remains on the heading for the Book icon.

<<Cause>>
  There are no code to deal with book collapse.

<<Solution>>
  Add the code to deal with book collapse in toc.js and tocview.jsp
Comment 1 Jiang Lin Quan CLA 2005-12-28 22:49:47 EST
Created attachment 32308 [details]
Patch for this bug
Comment 2 Jiang Lin Quan CLA 2005-12-29 02:28:13 EST
Created attachment 32317 [details]
Patch for this bug

For book level operation, This patch add:
1, use left key collapse the book function.
2, single click the book icon expand/collapse function. 
3, double click the book link expand/collapse function.
Comment 3 Curtis d'Entremont CLA 2006-06-02 14:39:24 EDT
I like the functionality you added with the patch, particularly the single click on the book icon to expand. However sometimes this part doesn't work for me. For example, if I click on the link to expand, I can click on the book icon to collapse, but if I click on another book icon to expand, it won't work. Any ideas?
Comment 4 Jiang Lin Quan CLA 2006-06-28 22:27:45 EDT
Hi, you could add following patch to fix what you said.

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
@@ -202,7 +202,7 @@
 		} else {
 %>
 		<li>
-		<img src="<%=prefs.getImagesDirectory()%>/toc_closed.gif" alt="<%=ServletResources.getString("bookClosed", request)%>" title="<%=ServletResources.getString("bookClosed", request)%>"><a id="b<%=toc%>" name="<%=data.getTocHref(toc)%>" style="font-weight: bold;" href="<%=data.getTocDescriptionTopic(toc)%>" onclick='loadTOC("<%=data.getTocHref(toc)%>")'><%=data.getTocLabel(toc)%></a>
+		<img src="<%=prefs.getImagesDirectory()%>/toc_closed.gif" onclick='loadTOC("<%=data.getTocHref(toc)%>")' alt="<%=ServletResources.getString("bookClosed", request)%>" title="<%=ServletResources.getString("bookClosed", request)%>"><a id="b<%=toc%>" name="<%=data.getTocHref(toc)%>" style="font-weight: bold;" href="<%=data.getTocDescriptionTopic(toc)%>" onclick='loadTOC("<%=data.getTocHref(toc)%>")'><%=data.getTocLabel(toc)%></a>
 <%
 		}
 %>
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Comment 5 Curtis d'Entremont CLA 2006-06-29 11:49:32 EDT
Thanks Jiang, that did the trick.
Patch applied.