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

Latte: implemented new macro {snippetRunner} #1011

Closed
wants to merge 1 commit into from

Conversation

hrach
Copy link
Contributor

@hrach hrach commented Mar 31, 2013

Adds ability to run some part of template without catching the output. Really useful with dynamic snippets.

  • adds ability to run dynamic snippets
  • adds ability to run include with snippets

Discussion http://forum.nette.org/cs/5713-snippety-a-inkludovana-sablona#p73830 - see some requests for rebasing etc, it's really needed behaviour.

Also, see how it's possible to hack this without snipperRunner: https://github.com/nextras/datagrid/blob/3f1370e1b4b86c668d2f6cbcb48dca643337597f/Nextras/Datagrid/Datagrid.latte#L187-L189

@rostenkowski
Copy link
Contributor

👍

@hrach
Copy link
Contributor Author

hrach commented Dec 9, 2013

ping @dg

@fprochazka
Copy link
Contributor

👍 I like the idea

@hrach hrach closed this in 82721e2 Dec 10, 2013
@hrach hrach deleted the f-snippet-runner branch December 11, 2013 00:57
@hrach
Copy link
Contributor Author

hrach commented Dec 11, 2013

thanks!

@Majkl578
Copy link
Contributor

Majkl578 commented Jan 8, 2014

Would you mind documenting it? There is not a single word about it in documentation.

@hrach
Copy link
Contributor Author

hrach commented Jan 8, 2014

Personally I don't wanna document this. If I find time to make doc, I will do database updates.

@Majkl578
Copy link
Contributor

Majkl578 commented Jan 8, 2014

Ok, hpfly someone will, I just came around it today when I was working with dynamic snippets and documentation was quiet about it. :)

hrach added a commit to nette/latte that referenced this pull request Mar 18, 2014
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

4 participants