On Thu, 6 Aug 2020 09:39:35 GMT, Jeanette Winzenburg <faste...@openjdk.org> 
wrote:

> okay, I still favor the not-adding-if-not-needed approach

I have updated the PR to use this approach. I did not think that all of
`FocusTraversalInputMap.getFocusTraversalMappings()`  can be removed. But looks 
like they can be removed without any
side effect. This approach looks good. Additionally, changed name of the 
property as
`excludeKeyMappingsForComboBoxEditor` to reflect the action.


> whether or not the mappings are not/added to the inputMaps.

Added this test for both ComboBox and ListView.



> Anyway, could be left open for the scope of this issue, IMO.

I also think the same that it can be kept out of scope for now.

-------------

PR: https://git.openjdk.java.net/jfx/pull/172

Reply via email to