Needs work
Project:
Lost & found issues
Component:
Twig templates conversion (front-end branch)
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
2 Nov 2012 at 14:59 UTC
Updated:
18 Feb 2013 at 18:48 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
jenlamptonno no no.
Having a template that prints a single variable is going to drive our entire front-end community nuts. We need to actually give them something in this template file to override, or at least a hint of where to go to get at what they wanted.
I'm thinking our twig template should look something like this to start:
Progress on this template might be blocked by #1778968: Convert theme_table to Twig
Comment #2
artofeclipse commentedthe attached patch have re-write to taxonomy-overview-terms.html.twig and remove the include inside twig template.
Comment #3
artofeclipse commentedComment #4
jpamental commentedI'm not entirely sure this is still valid. Some bits seem to be in the sandbox already, some don't seem present at all. Patch fails in both sections.
Setting back to 'needs work' but it may simply no longer be valid.
@artofeclipse can you comment?
Comment #5
artofeclipse commented@jpamental the template already there, I think you can close this issue.
Comment #6
jpamental commented@artofeclipse it's in the sandbox, but now needs to be converted/rerolled against core. I can put this on my list to do!
Comment #7
artofeclipse commentedI can help in this, I'm trying to do something to this initiative but there is some many documents to read, I just want to contribute to drupal 8 core, just guide me and I will do it.