for the keydownhandler a modifier is passed as parameter, but for the keyuphandler modifier.None is passed.
That difference would need a comment as explanation, surprises me.