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

Updating tag & content_tag for better HTML 5 support #760

Merged
merged 1 commit into from Jan 19, 2012

Conversation

Projects
None yet
3 participants
@Cirex

Cirex commented Jan 19, 2012

I dropped support for :content in tag but this change had almost no impact on the test coverage and I expect it to have just as little on your average project.

@DAddYE

This comment has been minimized.

Show comment
Hide comment
@DAddYE

DAddYE Jan 19, 2012

Member

@nesquena seems great, what do you think?

Member

DAddYE commented Jan 19, 2012

@nesquena seems great, what do you think?

@nesquena

This comment has been minimized.

Show comment
Hide comment
@nesquena

nesquena Jan 19, 2012

Member

Agreed, thanks @Cirex we should merge in soon.

Member

nesquena commented Jan 19, 2012

Agreed, thanks @Cirex we should merge in soon.

nesquena added a commit that referenced this pull request Jan 19, 2012

Merge pull request #760 from Cirex/master
Updating tag & content_tag for better HTML 5 support

@nesquena nesquena merged commit 1f7293f into padrino:master Jan 19, 2012

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