Closed (duplicate)
Project:
Drupal core
Version:
8.0.x-dev
Component:
theme system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
5 Jul 2013 at 22:34 UTC
Updated:
29 Jul 2014 at 22:37 UTC
There are way too many theme functions / template files in core. Some are similar (if not identical) and should be consolidated.
We have three options:
theme('table') instead.theme('table') and add all our preprocess magic into template_preprocess_table__image_styles()theme('table__image_styles__list') instead of theme('table__image_styles') and do all our preprocess magic into template_preprocess_table__image_styles__list()Decide which of the 3 options above to pursue, and do it.
None.
Removal of theme_image_style_list
#2035903: Remove theme_image_style_effects() and call theme('table__image_styles__effects') instead.
#1812684: [meta] Consolidate all table templates and add theme_hook_suggestions
#1898420: image.module - Convert theme_ functions to Twig
#1938910: Convert image theme tables to table #type
Comments
Comment #1
andypostAlready fixed #1788542: Use EntityFormController and EntityListController for image styles
Comment #1.0
andypostmotivation, related