552843 - Keyboard code triggers "charCode is meaningless" warnings in Firebug

Status: RESOLVED INVALID

(Skywriter Graveyard :: General, defect, P3)

Description

[Patrick Walton (:pcwalton)]

In Bespin, SproutCore triggers the following Firebug warnings repeatedly:

    The 'charCode' property of a keyup event should not be used. The value is meaningless.

    The 'charCode' property of a keydown event should not be used. The value is meaningless.

Since there are many such warnings that tend to scroll the debugging console, this should probably be fixed upstream.

Comment 1

[Kevin Dangoor]

We're no longer using SC.

Comment 2

[Kevin Dangoor]

Reopening with a new description. Julian reports that this problem is still present, but it is now in code under our management, not in SC.

Comment 3

[Patrick Walton (:pcwalton)]

Having trouble reproducing this. Julian, could you weigh in on how to reproduce?

Comment 4

[Julian Viereck]

Reproduction:

1. Load Bespin in Firefox
2. Open the Firefox Console
3. Make sure you have the option 'Show JavaScript Warning' activated
4. Select the editor and start typing

Result:

The message "The 'charCode' property of a keydown event should not be used. The value is meaningless." appears on the command line.

Expected Result:

There shouldn't be a warning.

If I can give you more assistent, let me know!

Comment 5

[Gordon P. Hemsley [:GPHemsley]]

I believe this is related to jQuery (or JavaScript) more than Bespin itself. In particular, I've found that calling keydown() or keyup() rather than keypress() will trigger this message in Firebug.

However, from my limited experience, there are cases where using keypress() doesn't produce the desired result, but using keydown() or keyup() does.

Comment 6

[satyr]

http://github.com/jquery/jquery/blob/master/src/event.js#L467

Comment 7

[Kevin Dangoor]

ACETRANSITION

The Skywriter project has merged with Ajax.org's Ace project (the full server part of which is their Cloud9 IDE project). Background on the change is here:

http://mozillalabs.com/skywriter/2011/01/18/mozilla-skywriter-has-been-merged-into-ace/

The bugs in the Skywriter product are not necessarily relevant for Ace and quite a bit of code has changed. For that reason, I'm closing all of these bugs. Problems that you have with Ace should be filed in the Ace issue tracker at GitHub:

https://github.com/ajaxorg/ace/issues