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

Already on GitHub? Sign in to your account

Nifty:layout application.html.haml fixed #103

Merged
merged 3 commits into from Jun 30, 2011

Conversation

Projects
None yet
2 participants
Contributor

ognevsky commented May 20, 2011

fixed yield(:title) behavior in nifty:layout. Now checking this with presence method.

Owner

ryanb commented Jun 29, 2011

It should actually do content_for?(:title) ? yield(:title) : "Untitled" like the ERB version. Thanks for reporting this. Feel free to update this pull request.

Contributor

ognevsky commented Jun 30, 2011

Oops, sorry for merge commit, my fault ;(
But I've updated, thanks for these awesome generators!

Owner

ryanb commented Jun 30, 2011

Perfect thanks!

ryanb added a commit that referenced this pull request Jun 30, 2011

Merge pull request #103 from ognevsky/master
Nifty:layout application.html.haml fixed

@ryanb ryanb merged commit 18dfe48 into ryanb:master Jun 30, 2011

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