Drupal 7.16

Going to Pixture Reloaded -> Settings -> Extensions -> Markup Overrides and Enabling: Use page--full-width-wrappers.tpl.php by default generates this error: Notice: Undefined index: leaderboard in include() (line 104 of /install-location/sites/all/themes/adaptivetheme/at_core/templates/page--full-width-wrappers.tpl.php).

Happy to give any more details requested.

Comments

Jeff Burnz’s picture

Version: 7.x-3.0-rc1 » 7.x-3.x-dev
katannshaw’s picture

Title: Undefined index: leaderboard in include() (line 104 » Undefined index: leaderboard in include() (line 104 of ...adaptivetheme\at_core\templates\page--full-width-wrappers.tpl.php).

I was able to reproduce this same error by following the steps from the AT documentation on templates. This is what it says:

page--full-width-wrappers.tpl.php
This page template that has 100% width wrappers, which effectively divide the page up into sections to you can style with full width backgrounds. To use copy this to your subtheme and rename it to page.tpl.php, or enable this in theme settings under Extensions > Markup Overrides > Design.

So like smhanes on the original post, I did the following:

  • Checked the checkbox titled "Markup overrides" at Appearance > Settings > Pixture Reloaded > Extensions > Enable Additional Features
  • Checked the checkbox titled "Use page--full-width-wrappers.tpl.php by default" at Appearance > Settings > Pixture Reloaded > Markup Overrides > Design

Which resulted in this error:

Notice: Undefined index: leaderboard in include() (line 104 of \mysite\sites\all\themes\adaptivetheme\at_core\templates\page--full-width-wrappers.tpl.php).

I've also found these similar issue reports but there doesn't appear to be a solution with the UI as of yet:

katannshaw’s picture

@Jeff Burnz: Has there been any update on this issue? It's still happening for me, so I cannot use the 100% width option.

Jeff Burnz’s picture

Issue summary: View changes

At this stage I have to "not fix" this bug, instead in the latest dev the option to use full width wrappers is now disabled (completely removed for this theme).

The main reason is this - many themes are not designed to work will full width heades, footers etc, PR is one of those themes, the main header image is a fixed size image, stretching it etc looks bad, also it's a lot of additional work to make that option work in many themes (images are just one of the issues).

The setting was envisioned for use on new themes, for example if you're building one from AT Subtheme, it was short sighted of me to not think that people would expect it to work with existing themes like PR, Sky, Corolla, however it will never work with those themes.

Therefor I have to set this to won't fix, and the real fix is to disable this option in themes that really cannot support it, or where it's just too much work.

I really do apologies for taking a year a to answer this, I have been away for the best part of two years due to illness and have only been able to give sporadic support to these projects.

Jeff Burnz’s picture

Status: Active » Closed (won't fix)

Sorry, setting status.

katannshaw’s picture

I really appreciate the quick and thorough response to my #3 post, and it makes a lot of sense. It's a great theme regardless so it will work for me. Good luck to you with the illness stuff, and let me know if you'd ever like for me to test things out against my install in the future.

Jeff Burnz’s picture

Project: Pixture Reloaded » AdaptiveTheme
Component: Code » CSS/HTML
Status: Closed (won't fix) » Active

I'm going to fix this after all, too many issues and conceptually a bad idea to start with:

We need to use this in theme settings to remove the option, note to self to do this for ALL sub themes that cannot support this feature:

    // Remove options not usable or applicable to this design.
    $form['at']['modify-output']['design']['page_full_width_wrappers'] = array(
      '#access' => FALSE,
    );
Jeff Burnz’s picture

Status: Active » Fixed

OK, this should be fixed in all sub theme dev versions.

katannshaw’s picture

I've made this change on my install. Thanks Jeff.

Status: Fixed » Closed (fixed)

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