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.

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

Change to less-browser for old android #2727

Closed
wants to merge 1 commit into from

Conversation

namingFailed
Copy link

Change to check if tag.dataset.hasOwnProperty is a function to prevent
errors in error reporting in apps that use web view in older versions of
android (pre-4.4).

Change to check if tag.dataset.hasOwnProperty is a function to prevent
errors in error reporting in apps that us web view in older versions of
android (pre-4.4).
@seven-phases-max
Copy link
Member

I'm not sure but isn't it a duty of the recommended polyfills?

@SomMeri
Copy link
Member

SomMeri commented Dec 1, 2015

@namingFailed Did you tested it with es5-shim? It should add also hasOwnProperty if it is not available. Is there a reason not to use it? If shim does not work we should raise an issue there or/and merge this.

If the shim does work, then we recommend to use the shim. Less.js assumes ECMAScript 5 compatibility and next version may use something else from it. You risk that something else will break again if you do not use the shim in old browsers.

@seven-phases-max
Copy link
Member

Closing due to lack of info on the purpose of this PR (and it looks like it's just missed http://lesscss.org/usage/#browser-support).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants