Skip to content
This repository was archived by the owner on Oct 8, 2021. It is now read-only.
This repository was archived by the owner on Oct 8, 2021. It is now read-only.

JQM 1.4.1: using 'default' as property name causes syntax error on some platforms #7170

@rgherardi

Description

@rgherardi

I have found a potential problem in the new code of release 1.4.1
At line 3195, inside $.fn.animationComplete function, the righ side expression contains an identifier named 'default'.
In the same function, al line 3222, the same property is defined but the assignment fails.

I think it happens since 'default' is a reserved keyword of the javascript language, and in some old browsers (IE 8 or Android 2.3.x) it causes a conflict or syntax error, making an unrecoverable crash of JQM.

I found a very very simple solution: it's enough to change the name of the variable from 'default' to '_default' or defaultTime or something else different from 'default' and everything works ok!

I hope this will be done in the next release since it guarantess more backward compatibility, even in older platforms.

Bye

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions