Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
On the Theme System Overview topic:
https://api.drupal.org/api/drupal/core!modules!system!theme.api.php/grou...
We need a new section added that explains the basics of Twig. It needs to:
- Tell what Twig is
- Link to the docs on how to do D8 theming: https://www.drupal.org/theme-guide/8
- Link to http://twig.sensiolabs.org/doc/templates.html
Comment | File | Size | Author |
---|---|---|---|
#10 | Drupal-twig_documentation_added-2326093-10.patch | 1.43 KB | astrocling |
#7 | Drupal-twig_documentation_added-2326093-7.patch | 1.46 KB | astrocling |
#2 | Drupal-twig_documentation_added-2326093-2.patch | 1.46 KB | astrocling |
Comments
Comment #1
astrocling CreditAttribution: astrocling commentedI think I would like to take a stab at this. I am going to assign it to myself and get working on it this week.
Comment #2
astrocling CreditAttribution: astrocling commentedSo here is an attempt on this. I wanted to be concise and just provide a brief overview of Twig as well as providing the links.
I am not sure that more is necessary, but I am can rework this if necessary.
Comment #3
star-szrThank you @astrocling, at a glance the only thing that jumped out at me so far was:
s/TWIG/Twig/
Comment #4
jhodgdonAgreed with #3. Also:
Since these are both one-item lists... they shouldn't really be lists. Just leave out the : and - and we'll be good there.
The rest looks fine to me -- quick repatch?
Comment #5
astrocling CreditAttribution: astrocling commentedOk good notes, I missed that capitalization thing. I hadn't considered the list issue jhodgdon. I will repatch it after lunch and post.
Question though, should I leave out the - prior to the links as well because it is not technically a list, or are those appropriate? I tried to follow as much as I could standards wise, just want to make sure I am not forming bad habits.
Comment #6
jhodgdonYes, leave out the - so it is not a list.
Comment #7
astrocling CreditAttribution: astrocling commentedAlright lets try this one with the edits as we discussed.
Comment #8
jhodgdonGreat! So, sorry, I missed this the first go-around:
If you're including a URL without link text, you should leave out the @link / @endlink tags. They are only needed if you want to make a link with specific link text.
Comment #9
astrocling CreditAttribution: astrocling commentedOk no problem, I can do that. Thanks for the heads up, I didn't know I was misusing the @link tag.
Comment #10
astrocling CreditAttribution: astrocling commentedHere it is with the @links removed. Let me know if there is anything else. Thanks for looking at it.
Comment #11
jhodgdonLooks great, thanks!
Comment #12
astrocling CreditAttribution: astrocling commentedYou're welcome, thanks for reviewing it.
Comment #13
alexpottCommitted ea83349 and pushed to 8.0.x. Thanks!
Comment #16
batigolixComment #17
batigolixoops :( sorry for that