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?

Comments

Jeff Burnz’s picture

What 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.

Jeff Burnz’s picture

Category: bug » support

I do not believe this is a bug, I need more info like the screenshot.

alan mccoll’s picture

StatusFileSize
new59.74 KB

Drurpal 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!

Jeff Burnz’s picture

Yeah, 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.

webmaster-eddie’s picture

I have the same problem - drupal 7.14 core, using corolla ver2 and AT ver 2.

Jeff Burnz’s picture

OK, sounds like its time to release a new version to account for the form state changes.

DrupalUser293867’s picture

StatusFileSize
new51.01 KB

Confirmed, see attachment.

Using:
Corolla 7.x-2.2
AT Admin 7.x-2.2
AT Core 7.x-2.2
Drupal 7.14

Jeff Burnz’s picture

Version: 7.x-2.2 » 7.x-2.x-dev
Status: Active » Fixed

Its fixed in dev, new release coming in a day or two.

http://drupal.org/commitlog/commit/10872/af8e917f70b3c4f6718c6705b525c5f...

alan mccoll’s picture

StatusFileSize
new61.46 KB

Thanks 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 and The configuration options have been saved. dialog box.
I've attached the screenshot.
Am I doing something wrong?

Jeff Burnz’s picture

Status: Fixed » Needs work

You 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).

Jeff Burnz’s picture

Status: Needs work » Fixed

Pushed to dev.

alan mccoll’s picture

Jeff,
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

Jeff Burnz’s picture

The one you want is 7.x-2.2+13-dev, its there now. They can take a long time to update.

if I try to use the adaptivetheme_subtheme as the default theme, it behaves the same as the custom subtheme

What behavior are you referring to? I am not sure I follow.

Status: Fixed » Closed (fixed)

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