Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fixes #2109. #2167

Merged
merged 2 commits into from
Jan 8, 2012
Merged

Fixes #2109. #2167

merged 2 commits into from
Jan 8, 2012

Commits on Dec 12, 2011

  1. Fixes mootools#2109. 

    IE < 9, getAttribute returns properties set via `el.attribute = value`.
Other browsers return null for such properties that are not part of the
specs (e.g. `href`, `width`, or `title`).

This tries to normalize the behavior and prevents unwanted "expando"
properties from being returned (e.g. `inject` and other Element
methods).

TL;DR: whitelist any setProperty, and getProperty checks in the
whitelist or if the attribute is an expando (not part of the known
specs).

Fixed a spec with maxlenght property, when it needed to be maxlength.

PASSED: IE6-9; FFx 3-5, 8, 10, Opera 11, Chrome latest, Safari 5
    @ibolmo
    ibolmo committed Dec 12, 2011
    Configuration menu
    Copy the full SHA
    8715836 View commit details
    Browse the repository at this point in the history

  2. Removed extra alert.

    @ibolmo
    ibolmo committed Dec 12, 2011
    Configuration menu
    Copy the full SHA
    8fe6021 View commit details
    Browse the repository at this point in the history