We have a lot of frighteningly similar theme functions that output administration pages. These should be consolidated into one template file, and all modules should use the same one for displaying administration pages.
- theme_admin_page()
- theme_system_admin_index
- theme_user_admin_permissions
- theme_views_ui_style_plugin_table()
Remaining tasks
Related tasks
#1595614: [meta] Remove all the theme functions and templates in core that simply output a link. Replace with #type 'link' render arrays
#1812684: [meta] Consolidate all table templates and add theme_hook_suggestions
#1819284: [meta] Consolidate all form element container templates, and add theme_hook_suggestions
#1813426: [meta] Consolidate all item list templates and add theme_hook_suggestions
#1812724: Consolidate all form element templates and add theme_hook_suggestons
#1804614: [meta] Consolidate similar twig templates and properly use theme suggestions in core
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | Remove-similar-admin-page-theme-functions-1930840-3.patch | 7.06 KB | FreekyMage |
Comments
Comment #0.0
jenlamptonone more
Comment #0.1
jenlamptontheme_views_ui_style_plugin_table/8
Comment #0.2
jenlamptonreorder
Comment #1
FreekyMage commentedLooking into this. Can't find theme_user_admin_permissions and theme_views_ui_style_plugin_table() though, not in the code nor on api.drupal.org.
Comment #2
FreekyMage commentedJust a little status update to show I'm still working on this.
Comment #3
FreekyMage commentedComment #3.0
FreekyMage commentedupdate
Comment #4
pfrenssenI did a functional test of the patch. It still needs work, on /admin/index the human readable block titles are replaced by machine names (eg. "Database logging" is changed to "dblog").
Also, as this is a meta issue, this should not have any patches, instead the work should be done in the child issues which are listed in the issue summary.
Comment #5
star-szrComment #19
smustgrave commentedThank you for creating this issue to improve Drupal.
We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.
Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.
Thanks!
Comment #20
smustgrave commentedwanted to bump this meta 1 more time.