Agreed on leaving on_key_press empty. 
Still, might as well make this a regular method..   perhaps "default_key_handler" or something? Connecting to hidden functions just seems sneaky.