Remove theme_checkboxes() and call theme('container') instead

Created on 14 July 2013, about 11 years ago
Updated 21 February 2023, over 1 year ago

Problem/Motivation

There are way too many theme functions / template files in core. Some are similar (if not identical) and should be consolidated.

Proposed resolution

We have two options:

  1. Remove theme_checkboxes() and call theme('container') instead (build a render array with '#theme' => 'container')
  2. If we really do need a separate preprocess function for this, let's use theme('container__checkboxes') instead of theme('container') and add all our preprocess magic into template_preprocess_container__checkboxes()

Remaining tasks

Decide which of the 2 options above to pursue, and do it.

User interface changes

None

API changes

Removal of theme_checkboxes

Related Issues

#2041825: Remove theme_radios() and call theme('container') instead
#1819284: [meta] Consolidate all form element container templates, and add theme_hook_suggestions

📌 Task
Status

Needs work

Version

10.1

Component
Theme 

Last updated about 5 hours ago

Created by

🇺🇸United States jenlampton

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024