Code review comment for lp:~larsu/ido/lp1242550

Revision history for this message
Charles Kerr (charlesk) wrote :

Works as advertised and the code looks good.

One suggestion, it looks like we could pick up a little more spec compliance by adding '+' / '-' key support by adding GDK_KEY_KP_Add, GDK_KEY_KP_Subtract, GDK_KEY_plus, GDK_KEY_minus in the switch statement in ido_scale_menu_item_parent_key_press_event()

« Back to merge proposal