Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JBIDE-22316 Filter template + arrow down should focus on template list and navigate through templates #1154

Merged
merged 1 commit into from
May 12, 2016

Conversation

scabanovich
Copy link
Contributor

  1. Children widgets of tab items are included into the list
    of controls traversed by tab key.
  2. List is navigated with arrows while filter remains focused.

@fbricon
Copy link
Member

fbricon commented May 11, 2016

This is pretty cool @scabanovich! Only gripe I have is when you go up, the cursor is put to the 1st position. It shouldn't. See http://screencast.com/t/Fb6w5wFU

Also TabFolderTraverseListener is really a not trivial piece of code, I'd feel better if you added some unit tests for it

@scabanovich scabanovich force-pushed the jbide-22316 branch 2 times, most recently from 684c5fb to e2f9203 Compare May 12, 2016 00:59
@scabanovich
Copy link
Contributor Author

scabanovich commented May 12, 2016

I removed the default behaviour on arrows up/down which on MacOS apparently moved cursor, and added tests for TabFolderTraverseListener which is now a public class.

@fbricon
Copy link
Member

fbricon commented May 12, 2016

well now down arrow doesn't do anything when in the filter box

@@ -0,0 +1,85 @@
package org.jboss.tools.openshift.internal.ui.wizard.newapp;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing copyright header

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

…t and navigate through templates

1. Children widgets of tab items are included into the list
of controls traversed by tab key.
2. List is navigated with arrows while filter remains focused.
@fbricon
Copy link
Member

fbricon commented May 12, 2016

Ok, everything now works, except... the test fails at:
Assert.assertTrue(item.controls[0].isFocusControl());

The test pass when run from CLI. Must be something in my eclipse stealing the focus during tests. I suspect usage reporting.

@fbricon
Copy link
Member

fbricon commented May 12, 2016

Yup usage reporting was messing the test. It's all good now.

@fbricon fbricon merged commit 9377286 into jbosstools:master May 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants