There are several list items that could use some theme hook suggestions to identify the render array element in themes.

CommentFileSizeAuthor
#2 add_theme_hook-2863400-2.patch2.44 KBmarkhalliwell

Comments

markcarver created an issue. See original summary.

markhalliwell’s picture

Status: Active » Needs review
StatusFileSize
new2.44 KB
eugene.ilyin’s picture

Status: Needs review » Postponed (maintainer needs more info)

hmm, seems your theme callbacks are not defined in the module, maybe I forget something about theming, but looks like it will not work.

markhalliwell’s picture

Status: Postponed (maintainer needs more info) » Needs review

seems your theme callbacks are not defined in the module, maybe I forget something about theming

The portion of the theme hook after the first __ separator is a theme hook suggestion. The base theme hook, item_list is already provided by core. If suggestion(s) don't exist, then it falls back to the subsequent shorter theme hook suggestion and ultimately the base theme hook, e.g.:

item_list__facetapi__current_search_item_active   // <- looks for this fist
item_list__facetapi                               // <- then looks for this
item_list                                         // <- falls back to this
but looks like it will not work

It does work, theme hook suggestions have been around since 6.x. This isn't anything new.

eugene.ilyin’s picture

Thank you for an explanation. Committed.

eugene.ilyin’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.