Fix issue #37 - KeyEvent on Mac OS X

[pberkes]

Issues with KeyEvent on Mac OS X issue #37

[corranwebster]

This seems reasonable, although I'm not entirely sure about the addition of the 'Meta' key to Enable. If you are going to add 'Meta' you need to ensure that it is supported in all the backends (in particular, we do use the VTK back end in production code).

Also, have you validated that these changes don't lead to unexpected behaviour with the standard Chaco tools?

[pberkes]

IMHO, being able to detect "meta" is important so that Mac users may use key combinations that are common on Mac. An alternative would be to map Meta to Ctrl always.

I can look into supporting 'meta' in other backends. Which of them are actually used? E.g. 'savage' seems to extinct.

I did not see see any unexpected change in standard chaco tools so far. Do you have any one in particular in mind?

[pberkes]

vtkRenderWindowInteractor does not define a GetMetaKey method. The natural thing to do seems to always set meta_down to False in that case. It cannot break any code, since it could not be used before.
(Ref: http://www.vtk.org/doc/release/5.0/html/a01972.html)

[itziakos]

@pberkes and @corranwebster, is there an update on this PR? Should it be merged?

[pberkes]

@itziakos From what I remembered, the consensus was that these changes should be done for all the enable backends. I still think that we should be able to distinguish between Meta and Ctrl, especially on Mac, but this specific PR can be closed as far as I'm concerned.

[jwiggins]

Closing