I think we have the original jquery.js included in core. If there is a need for a fix, we likely should just update to the lastest release instead of applying fixes ourself.

Yes, we want to avoid modifying any external project's code if possible.

Best,

Matt Thomas
@betweenbrain
http://matt-thomas.me/
http://betweenbrain.com/
https://github.com/betweenbrain

Sent from mobile. Please pardon any typos or brevity.
On Dec 13, 2013 3:32 PM, "Thomas Hunziker" notifications@github.com wrote:

I think we have the original jquery.js included in core. If there is a
need for a fix, we likely should just update to the lastest release instead
of applying fixes ourself.

—
Reply to this email directly or view it on GitHub#2675 (comment)
.

As it is only a deprecation warning, I wouldn't be too concerned with it.

Best,

Matt Thomas
@betweenbrain
http://matt-thomas.me/
http://betweenbrain.com/
https://github.com/betweenbrain

Sent from mobile. Please pardon any typos or brevity.
On Dec 13, 2013 5:01 AM, "IberoMedia" notifications@github.com wrote:

file cmsroot/meida/jui/js/jquery.js triggers warning "event.returnValue is
deprecated. Please use the standard event.preventDefault() instead."

changing line 5374 - 5375

FROM:

this.isDefaultPrevented = ( src.defaultPrevented || src.returnValue ===
false ||
src.getPreventDefault && src.getPreventDefault() ) ? returnTrue :
returnFalse;

TO:

this.isDefaultPrevented = ( src.defaultPrevented || src.preventDefault()=== false ||
src.getPreventDefault && src.getPreventDefault() ) ? returnTrue :
returnFalse;

fixes the warning

—
Reply to this email directly or view it on GitHub#2675
.

Closing based on the comments above