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

--attr-wrapper option for command line and HAMLPY_ATTR_WRAPPER setting for Django #106

Merged
merged 4 commits into from Nov 12, 2012

Conversation

avsd
Copy link
Contributor

@avsd avsd commented Nov 12, 2012

Hi, Jesse!

Thanks for such a great tool!

Please, review this pull request - it's about specifying attribute wrapper for generated html:

<a href='/'>

vs

<a href="/">

I think, other options can be specified the same way (on demand, if somebody needs them). For more, see: http://haml.info/docs/yardoc/Haml/Options.html

P.s. sorry for so many whitespace changes - it's done by my IDE.

David

bcoughlan pushed a commit that referenced this pull request Nov 12, 2012
--attr-wrapper option for command line and HAMLPY_ATTR_WRAPPER setting for Django
@bcoughlan bcoughlan merged commit 451beb1 into jessemiller:master Nov 12, 2012
@bcoughlan bcoughlan mentioned this pull request Nov 12, 2012
Naddiseo added a commit to AsymmetricVentures/HamlPy that referenced this pull request Dec 20, 2012
jessemiller added a commit that referenced this pull request Dec 21, 2012
Fix for watcher options that broke in pull request #106
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.

2 participants