| Summary: | connector discovery wizard keyboard navigation and focus management | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | z_Archived | Reporter: | David Green <greensopinion> | ||||||||||
| Component: | Mylyn | Assignee: | David Green <greensopinion> | ||||||||||
| Status: | RESOLVED FIXED | QA Contact: | |||||||||||
| Severity: | normal | ||||||||||||
| Priority: | P2 | CC: | steffen.pingel | ||||||||||
| Version: | unspecified | ||||||||||||
| Target Milestone: | 3.2 | ||||||||||||
| Hardware: | All | ||||||||||||
| OS: | All | ||||||||||||
| Whiteboard: | |||||||||||||
| Bug Depends on: | |||||||||||||
| Bug Blocks: | 276232 | ||||||||||||
| Attachments: |
|
||||||||||||
|
Description
David Green
The TraverseListener in ControlListViewer and the revealControl() method may have some of the code that you need to implement this. Turning on 'Full Keyboard Access' on a mac enables tabbing onto a descriptor's checkbox. I don't think that we need to programatically control which items get focus, however ensuring that a focus change causes the newly focused item to be visible in the scrolled pane is still relevant. Keyboard navigation does not work on Linux. Once the list gets focus I am stuck and can not focus on other controls or leave the dialog. There is a flag on the ScrolledComposite that makes it auto-scroll if a child gets focus. Steffen I've made a fix for this, committed to CVS. There was a problem with a listener sucking up all the events -- It may be fixed for Linux now. Can you take a look and see if it's working for you? Created attachment 138085 [details]
mylyn/context/zip
Works for me with the latest from head. Thanks David! Only nit is that I am not able to navigate to the Info hyperlinks. (In reply to comment #7) > Only nit is that I am not able to navigate to the Info hyperlinks. Good point. I'll see what I can do about that. Created attachment 138299 [details]
patch that manages traversal
The attached patch attempts to manage focus programatically. It works as it should, with the exception that on Mac/Cocoa the SWT Label widget refuses to accept focus. The Label widget is used for the information image link. We could use a Button instead of a Label, however I'm not aware of a way to style the button so that it looks as it does now (SWT.FLAT is not enough). We could probably use a Button and paint it ourselves. Open to feedback here.
Created attachment 138300 [details]
mylyn/context/zip
Fixed. Info link is now a real button, which can accept focus. Custom painting makes it look flat. Created attachment 138303 [details]
mylyn/context/zip
|