It doesn't appear that advanced help is removing the theme CSS from the help popup, Looking at the code in the .module file:

  // Remove theme css.
  foreach ($css as $media => $types) {
    if (isset($css[$media]['theme'])) {
      $css[$media]['theme'] = array();
    }
  }

Does not remove the theme CSS, printing out the $css array, it looks like the CSS is being stored in the array differently in in drupal7 than it was in drupal6, I am creating a patch for this to properly remove the theme CSS.

CommentFileSizeAuthor
#1 advanced_help-removing_help_popup_theme_css-1803488-1.patch681 bytesundrgrnd
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

undrgrnd’s picture

undrgrnd CreditAttribution: undrgrnd commented
Status: Active » Needs review
FileSize
advanced_help-removing_help_popup_theme_css-1803488-1.patch681 bytes

this patch checks the $css array properly for any css with the group = CSS_THEME (100), this now properly removes CSS themes from the help popup window.

  • gisle committed 2814a0f on 7.x-1.x authored by undrgrnd 
    Issue #1803488 by undrgrnd: Ficed popup not removing theme CSS
gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
CreditAttribution: gisle commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community
gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
CreditAttribution: gisle commented
Status: Reviewed & tested by the community » Fixed
gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
CreditAttribution: gisle commented
Version: 7.x-1.x-dev » 7.x-1.2

Status: Fixed » Closed (fixed)

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