Using subtheme. When trying to save the theme settings, I get this error,
Standard layout max-width is empty - you forgot to enter a value for the max width!
There is no max-width setting.
Am I missing something?
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | setings.png | 61.46 KB | alan mccoll |
| #7 | cap.png | 51.01 KB | DrupalUser293867 |
| #3 | standardScreenshot.jpg | 59.74 KB | alan mccoll |
Comments
Comment #1
Jeff Burnz commentedWhat version of Drupal core are you running, its possible there is an issue here with the form #states (so its there alright, but hiding in the form).
Can you take a screenshot please, of the standard layout form. Please include the entire page.
Comment #2
Jeff Burnz commentedI do not believe this is a bug, I need more info like the screenshot.
Comment #3
alan mccoll commentedDrurpal core v 7.14
Attached a sccreenshot.
I've dowloaded and installed a pristine copy of Drupal 7.14 on my laptop, installed Adaptive Themes and enabled AT Subtheme 7.x-2.2 as the default theme. After going to settings, and saving, I get the same complaint,
Standard layout max-width is empty - you forgot to enter a value for the max width!Comment #4
Jeff Burnz commentedYeah, I know, its not really a bug, 7.14 changed form states and how they work so the setting is there but hidden. Not sure why its not populated though - that is a bit weird. Pretty sure I fixed this in DEV, will check and probably should make a new release - I've just been waiting until the level of 7.14 installs started to build up.
Comment #5
webmaster-eddie commentedI have the same problem - drupal 7.14 core, using corolla ver2 and AT ver 2.
Comment #6
Jeff Burnz commentedOK, sounds like its time to release a new version to account for the form state changes.
Comment #7
DrupalUser293867 commentedConfirmed, see attachment.
Using:
Corolla 7.x-2.2
AT Admin 7.x-2.2
AT Core 7.x-2.2
Drupal 7.14
Comment #8
Jeff Burnz commentedIts fixed in dev, new release coming in a day or two.
http://drupal.org/commitlog/commit/10872/af8e917f70b3c4f6718c6705b525c5f...
Comment #9
alan mccoll commentedThanks for you prompt attention, Jeff.
Downloaded the latest devels of both corolla and adaptive themes. In settings, I now see the SET A MAX WIDTH checkbox and widget. If I leave it set at 1260 px or change it, I get the
Standard layout max-width is empty - you forgot to enter a value for the max width!dialog box and the pixel input box is bordered red, when trying to save. If I untick the SET A MAX WIDTH checkbox and save, I get the green checkmark andThe configuration options have been saved.dialog box.I've attached the screenshot.
Am I doing something wrong?
Comment #10
Jeff Burnz commentedYou are not doing anything wrong - from what I can tell the theme is not really using the right check in the validation function, which means its not quite checking the right variable and can be empty in certain circumstances.
I will commit a fix shortly, I'm pretty sure I know what the issue is (although for various reasons this is really hard for me to reproduce, but I can see how this could happen now you mentioned it).
Comment #11
Jeff Burnz commentedPushed to dev.
Comment #12
alan mccoll commentedJeff,
Downloaded the new 7.x-2.2+11-dev. Installed, copied subtheme, renamed, edited the .info, template.php, and theme-settings.php. Still no joy. Although the SET A MAX WIDTH checkbox appears, if a max width is indicated, I get the same error as the previous post screenshot. Unticking the checkbox allows a successful save.
Curiously, when I first did the above using the same custom subtheme name as I have been using, the SET A MAX WIDTH checkbox was not visible. It was however visible using the default subtheme named adaptivetheme_subtheme. I changed the custom subtheme name to something different, changed all the other subsequent entries in the various files, and the checkbox setting showed up. By the way, if I try to use the adaptivetheme_subtheme as the default theme, it behaves the same as the custom subtheme. Is it possible that my cache has not been properly cleared? I've done this by both the Admin > Performance > Clear Cache menu entry through Drupal and also
$drush cc all. I'm doing this work on my local server. Is there anything else I'm missing or should try?Al
Comment #13
Jeff Burnz commentedThe one you want is 7.x-2.2+13-dev, its there now. They can take a long time to update.
What behavior are you referring to? I am not sure I follow.