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

fix #1055 option stayed selected in Select after removal from options list #1309

Closed
wants to merge 1 commit into from

Conversation

jakub-g
Copy link
Collaborator

@jakub-g jakub-g commented Sep 30, 2014

This is a follow-up of PR #1218

I've changed the test scenario from the first one to the second one reported in #1055 and added some more asserts (the first one was no longer reproducible since it was fixed in 1.6.5, moreover the second scenario was better showing the issue).


When @aria:Select options were bound to the data model,
and the currently selected option was removed from the option list
through the binding, it still remained as the data model value
of the widget (although no longer available in the UI).

Close #1218
Close #1309

… from options list

When `@aria:Select` options were bound to the data model,
and the currently selected option was removed from the option list
through the binding, it still remained as the data model value
of the widget (although no longer available in the UI).

Close ariatemplates#1218
Close ariatemplates#1309
@jakub-g jakub-g added this to the v1.6.6 milestone Sep 30, 2014
@jakub-g jakub-g changed the title fix #1055 invalid Select option widget data binding issue with options fix #1055 option stayed selected in Select after removal from options list Sep 30, 2014
@divdavem divdavem self-assigned this Sep 30, 2014
@divdavem
Copy link
Member

🆗 fd91fb2 seems fine for me

@smadapathi smadapathi closed this in d7d825c Oct 1, 2014
@jakub-g jakub-g deleted the pr1218again branch October 1, 2014 12:08
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