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

Client-side Support #39

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wintondeshong
Copy link

Here is my fork of haml-js to allow for client-side support (primarily internet explorer)

Outline of changes:
Refactored Array.forEach, Array.indexOf, Array.map, and Array.filter with equivilent underscore.js functions
Refactored String.trim to use internal trim function
Refactored primary Haml signature to exclude 'Haml' from '...function Haml(haml, config) {'

…with equivilent underscore.js functions; Refactored String.trim to use internal trim function; Refactored primary Haml signature to exclude 'Haml' from '...function Haml(haml, config) {'
@tribalvibes
Copy link

+1

1 similar comment
@heikki
Copy link

heikki commented Jul 3, 2011

+1

@wintondeshong
Copy link
Author

Note: You must add json2.js as a dependency when using this on the client-side (https://github.com/douglascrockford/JSON-js) as IE7 and lower don't natively support JSON.stringify.

@devth
Copy link

devth commented Jul 15, 2011

+1 🍺

@ajpiano
Copy link

ajpiano commented Aug 11, 2011

This is great. +1

@jstevenson
Copy link

+1

5 similar comments
@jkichline
Copy link

+1

@mwurzberger
Copy link

+1

@andCulture
Copy link

+1

@fnmunhoz
Copy link

+1

@joshrieken
Copy link

+1

@devth
Copy link

devth commented Nov 2, 2011

Anything happening here?

@rmw
Copy link

rmw commented Dec 15, 2011

+1

Looks like their have been a few updates to master since winton's changes.

@rmw
Copy link

rmw commented Dec 15, 2011

In the meantime, I've pulled the upstream changes into this fork on winton's branch: https://github.com/rmw/haml-js/blob/master/lib/haml.js

In case anyone wants it ...

@betamos
Copy link

betamos commented May 13, 2012

+1

1 similar comment
@du-song
Copy link

du-song commented Jan 5, 2013

+1

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

Successfully merging this pull request may close these issues.

None yet