Fix Cmd-m key misbehaviour on MacOS

This fixes meta key handling on MacOS where keydown event for it is intercepted by browser
(unlike Windows).

Closes rietveld issue #228 arised as a followup of patchset #2055042

[Andi Albrecht, 2010-09-18 16:46:24]

http://codereview.appspot.com/2200044/diff/1/static/script.js
File static/script.js (right):

http://codereview.appspot.com/2200044/diff/1/static/script.js#newcode2001
static/script.js:2001: // case ??: Meta ?
What's the purpose of this line? Is it a placeholder?

[techtonik, 2010-09-18 17:20:25]

http://codereview.appspot.com/2200044/diff/1/static/script.js
File static/script.js (right):

http://codereview.appspot.com/2200044/diff/1/static/script.js#newcode2001
static/script.js:2001: // case ??: Meta ?
On 2010/09/18 16:46:25, Andi Albrecht wrote:
> What's the purpose of this line? Is it a placeholder?

Yes. But frankly it can be safely ignored, because I really can't see how would
"Meta" key alone could be useful for Rietveld. Just for completeness I wanted to
place a key code here, but it seems impossible. At least on Windows I couldn't
intercept the state when event.metaKey = True

[Andi Albrecht, 2010-09-18 17:44:19]

Ok, thanks for the feedback, Anatoly!

I'm fine with this to go in as it seems to have no side-effects.

Andi

2010/9/18, techtonik@gmail.com <techtonik@gmail.com>:
>
> http://codereview.appspot.com/2200044/diff/1/static/script.js
> File static/script.js (right):
>
> http://codereview.appspot.com/2200044/diff/1/static/script.js#newcode2001
> static/script.js:2001: // case ??: Meta ?
> On 2010/09/18 16:46:25, Andi Albrecht wrote:
>> What's the purpose of this line? Is it a placeholder?
>
> Yes. But frankly it can be safely ignored, because I really can't see
> how would "Meta" key alone could be useful for Rietveld. Just for
> completeness I wanted to place a key code here, but it seems impossible.
> At least on Windows I couldn't intercept the state when event.metaKey =
> True
>
> http://codereview.appspot.com/2200044/
>

[Andi Albrecht, 2010-09-19 07:36:54]

Committed r574 and live.

Andi Albrecht <albrecht.andi@googlemail.com> writes:

> Ok, thanks for the feedback, Anatoly!
>
> I'm fine with this to go in as it seems to have no side-effects.
>
> Andi
>
> 2010/9/18, techtonik@gmail.com <techtonik@gmail.com>:
>>
>> http://codereview.appspot.com/2200044/diff/1/static/script.js
>> File static/script.js (right):
>>
>> http://codereview.appspot.com/2200044/diff/1/static/script.js#newcode2001
>> static/script.js:2001: // case ??: Meta ?
>> On 2010/09/18 16:46:25, Andi Albrecht wrote:
>>> What's the purpose of this line? Is it a placeholder?
>>
>> Yes. But frankly it can be safely ignored, because I really can't see
>> how would "Meta" key alone could be useful for Rietveld. Just for
>> completeness I wanted to place a key code here, but it seems impossible.
>> At least on Windows I couldn't intercept the state when event.metaKey =
>> True
>>
>> http://codereview.appspot.com/2200044/
>>