Bug 13346

Summary: Accessibility: cannot navigate with keyboard in Preview view
Product: [Eclipse Project] Platform Reporter: Christophe Elek <celek>
Component: Update (deprecated - use Eclipse>Equinox>p2)Assignee: Dejan Glozic <dejan>
Status: RESOLVED FIXED QA Contact:
Severity: critical    
Priority: P2 CC: klicnik, n.a.edgar, Tod_Creasey
Version: 2.0Keywords: accessibility
Target Milestone: 2.0 F3   
Hardware: PC   
OS: Windows 2000   
Whiteboard:

Description Christophe Elek CLA 2002-04-10 11:18:50 EDT
M5

Open Install/Update Perspective
Click CTRL+F7 to show the views
Select Preview
Any ketstroke will occur in another view
Comment 1 Christophe Elek CLA 2002-04-16 22:03:15 EDT
[accessibility] 
Comment 2 Nick Edgar CLA 2002-06-03 10:48:26 EDT
This is important for accessibility.  Please bump up priority to P1 and 
consider for F3.
The Preview view should implement setFocus().

Tod, since this is using an embedded web browser, please comment on how they 
should give it focus.
Comment 3 Nick Edgar CLA 2002-06-03 10:53:17 EDT
We are currently referring to this PR from our checklist of accessibility 
failures.
Comment 4 Vlad Klicnik CLA 2002-06-04 12:36:03 EDT
F3 candidate
Comment 5 Dejan Glozic CLA 2002-06-10 10:11:04 EDT
Preview page is NOT using a web browser (thanks for not noticing it is our own 
in-house brower layout engine) :-).
Comment 6 Dejan Glozic CLA 2002-06-10 13:43:45 EDT
Fixed. Two problems were related:

1) Some pages have not honoured 'setFocus' method. This was because an attempt 
to fix the problem in the past resulted in MultiPageView grabbing focus on 
selection change.

2) The reason MultiPageView grabbed focus was because it was calling 'setFocus' 
on page switch. 

Fixed by removing 'setFocus' on page switch and by honouring 'setFocus' in all 
pages that show up in the 'Preview' view.