Re: Accelerators - odd behavior

[Stephen F Northover]

Two on Windows, one on Mac.  See 
https://javafx-jira.kenai.com/browse/RT-38830

Steve
On 2014-09-26, 1:25 PM, Stephen F Northover wrote:
This is on Mac.  Will try Windows.

Steve

On 2014-09-26, 1:24 PM, Kevin Rushforth wrote:
Is this on a Mac or on Windows?  I just ran your test case and I get 
two runnables, which is what I would expect.

-- Kevin


Stephen F Northover wrote:
I am only seeing the runnable fired once in FX 8u40.

Steve

Steps:

1) Run TestKeyCombination
2) Press Control+-

Here is the test code:

import javafx.application.Application;
import javafx.scene.Group;
import javafx.scene.Scene;
import javafx.scene.control.Button;
import javafx.scene.input.KeyCharacterCombination;
import javafx.scene.input.KeyCode;
import javafx.scene.input.KeyCodeCombination;
import javafx.scene.input.KeyCombination;
import javafx.stage.Stage;

public class TestKeyCombination extends Application {
    public static void main(String[] args) {
        Application.launch(args);
    }

    @Override public void start(Stage stage) {
        stage.setTitle("Test KeyCombination");
        Scene scene = new Scene(new Group(), 600, 450);
        Button button1 = new Button();
        button1.setText("Click Me");
        stage.setScene(scene);
        stage.show();

        KeyCombination cmdMinus = new 
KeyCodeCombination(KeyCode.MINUS, KeyCombination.CONTROL_DOWN);
        KeyCombination cmdMinusFromCharacter = new 
KeyCharacterCombination("-", KeyCombination.CONTROL_DOWN);
        Runnable runnable = () -> System.out.println("HI");
        scene.getAccelerators().put(cmdMinus, runnable);
        scene.getAccelerators().put(cmdMinusFromCharacter, runnable);
    }
}


On 2014-09-26, 1:01 PM, Scott Palmer wrote:
KeyCombination cmdMinus = new Key*Code*Combination(KeyCode.MINUS,
KeyCombination.CONTROL_DOWN);
KeyCombination cmdMinusFromCharacter = new 
Key*Character*Combination("-",
KeyCombination.CONTROL_DOWN);

Using the above like this:
scene.getAccelerators().put(cmdMinus, runnable);
scene.getAccelerators().put(cmdMinusFromCharacter, runnable);

Will result in the runnable being fired twice from the same keypress.

I propose changing the accelerator processing logic so that only one
runnable gets called as the intention appears to be that a 
KeyCombination
can only have one runnable associated with it, but the logic in Map 
doesn't
see the above two KeyCombinations as the same key in the Map.

Note: With the second combination above I really wanted something that
worked for both MINUS and SUBTRACT simultaneously - since they both 
type
the same Character and only one accelerator can be set on a MenuItem.

Scott