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

8129123: ComboBox popup list view does not scrollTo when ComboBox value/selection is set #166

Closed
wants to merge 2 commits into from
Closed
Changes from 1 commit
Commits
File filter...
Filter file types
Jump to…
Jump to file
Failed to load files.

Always

Just for now

@@ -2095,6 +2095,15 @@ public void test_rt_40012_selectedItemNotificationOnDisjointRemovesAbove() {
assertTrue(" visible range [" + first + ", " + last + "] must include " + index,
first <= index && index <= last);

This comment has been minimized.

@aghaisas

aghaisas Apr 17, 2020
Collaborator

You have tested first and last index selection.
It will be nice to have a case where we select index = LIST_SIZE/2

This comment has been minimized.

@ccavanaugh

ccavanaugh Apr 18, 2020
Author

Agreed, mid point test added

index = LIST_SIZE / 2;
comboBox.getSelectionModel().select(index);
Toolkit.getToolkit().firePulse();

first = virtualFlow.getFirstVisibleCell().getIndex();
last = virtualFlow.getLastVisibleCell().getIndex();
assertTrue(" visible range [" + first + ", " + last + "] must include " + index,
first <= index && index <= last);

index = LIST_SIZE - 1;
comboBox.getSelectionModel().select(index);
Toolkit.getToolkit().firePulse();
ProTip! Use n and p to navigate between commits in a pull request.