On Fri, 28 Aug 2020 10:21:50 GMT, Jeanette Winzenburg <faste...@openjdk.org> wrote:
> If the ComboBox is not editable, it will have the effect of making the HOME > and END keys no-ops, which is a (possibly > unwanted) change in behavior. I have updated PR with changes for non editable ComboBox. I could think of adding another property to propagate the editable property of ComboBox to ListViewBehavior. So now this fix adds another property `editableComboBoxEditor`, not sure if there is other way to handle it. The change adds HOME and END KeyMappings when ComboBox is non editable and removes them when ComboBox is editable. If the change sounds Ok, I shall include test in next commit. Also, there is one change in the if condition that was suggested by Jeanette before, `if (!Boolean.TRUE.equals(control.getProperties().containsKey("excludeKeyMappingsForComboBoxEditor")))` is changed to, `if (Boolean.FALSE.equals(control.getProperties().containsKey("excludeKeyMappingsForComboBoxEditor")))` It seems safe as `control.getProperties().containsKey()` returns either true or false. ------------- PR: https://git.openjdk.java.net/jfx/pull/172