I was seeing a confusing behavior. When I set the theme to display default logo, the image would appear broken (404) with a path like http://example.com/profiles/openoutreach/themes/contrib/granada/logo.png when the image is in themes/contrib/granada/img/logo.png. When I unchecked default logo but would leave the input field blank, then the logo would appear since template.php sets it to themes/contrib/granada/img/logo.png where it's found.

I believe the correct way to do this is to move the logo.png file and change its reference in template.php. Included patch does exactly this.

As a sidenote, the Favicon toggle wouldn't stick. I'd save the settings, it would remain unchecked. After fixing the logo problem, this problem also went away...

CommentFileSizeAuthor
granada-fix-logo.patch499 bytesRobin Millette
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Robin Millette’s picture

Robin Millette CreditAttribution: Robin Millette commented
Status: Active » Needs review

Hmm, I was expecting the patch to include the mv instructions somehow. The correct patch should also move granada/img/logo.png to granada/logo.png

nedjo’s picture

nedjo
he/him/his
Primary language English
CreditAttribution: nedjo commented
Status: Needs review » Fixed

Thanks, applied! see http://drupalcode.org/project/granada.git/commit/351c0a9.

Robin Millette’s picture

Robin Millette CreditAttribution: Robin Millette commented
Status: Fixed » Active

The logo.png file still exists in the img directory which could be confusing (it's now in the root directory where it belongs).

nedjo’s picture

nedjo
he/him/his
Primary language English
CreditAttribution: nedjo commented
Status: Active » Fixed

Thx, removed now.

Status: Fixed » Closed (fixed)

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