The current _bootstrap_glyphicons_supported() implementation will only work if bootstrap or a sub-theme was the the default theme, and not the active theme that is currently running which should be the case.

// Get the active theme.
$drupal_static_fast['theme'] = variable_get('theme_default', $GLOBALS['theme']);

I think adding a setting in the info file beside the CDN option, for the location of bootstrap file may be the best way to go, what do you think?

CommentFileSizeAuthor
#1 check-current-theme-2260907-1.patch1.01 KBmkhamash

Comments

mkhamash’s picture

mkhamash commented
StatusFileSize
new check-current-theme-2260907-1.patch1.01 KB
mkhamash’s picture

mkhamash commented
Related issues: +#2224819: Iterate through all active (base) themes in _bootstrap_glyphicons_supported(), +#2206725: Create _bootstrap_glyphicons_supported() helper function
markhalliwell’s picture

markhalliwell
Primary language English
commented
Assigned: mkhamash » Unassigned
Priority: Minor » Normal
Status: Active » Postponed (maintainer needs more info)

I'm not entirely sure if this is a bug. It's probably works as designed. What is the use case here exactly? Shouldn't the default theme be the active theme? The reason this uses the default theme is because of bootstrap_icon_bundles() for Icon API support. If the admin theme is not Bootstrap based (ie: seven), then the icons wouldn't be picked up.

markhalliwell’s picture

markhalliwell
Primary language English
commented
Status: Postponed (maintainer needs more info) » Closed (works as designed)

No answer to the above questions.