Set Max width on standard layout setting disappeared after core update to 7.22
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | page-max-width.png | 255.12 KB | Jeff Burnz |
Set Max width on standard layout setting disappeared after core update to 7.22
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | page-max-width.png | 255.12 KB | Jeff Burnz |
Comments
Comment #1
Jeff Burnz commentedNo it didn't. Maybe you have an issue, in which case a link, login, more information?
Comment #2
psampaz commentedSorry, after drupal core update to 7.14 I mean.
Same thing happened to 3 different websites.
Comment #3
nkleopas commentedSame problem happened to me with "Picture Reloaded" and "Sky" theme. After core drupal update (7.14) the "Set a max width" option in the layout settings is no longer visible.
Comment #4
Jeff Burnz commentedHow odd, I will try running 7.14, normally I run core dev, I am pretty sure I know the issue and updated the title.
Comment #5
Jeff Burnz commentedCertinaly some of the form #states have been broken, probably most likely due to:
- Form API #states: Fix conditionals to allow OR and XOR constructions.However, I cannot reproduce this exact instance of the state being broken, meaning, the max width does show when % width is selected for the page width - can you try clearing your sites cache or turning OFF JavaScript aggregation then turning it back on again. See what happens after that, and save your theme settings also.
Comment #6
Jeff Burnz commentedI've committed an update to DEV that fixes and changes some of the form #states for sidebars (the ones for smartphone were broken), and made the others better. I still can't reproduce the issue in the OP, not saying its not happening, but its probably an artifact of upgrading on your system - possibly a combination of upgrading Drupal core and the themes, or something along those lines, I tend to think it may well be something with JS aggregation, but I am not sure.
BTW: if you are wondering what all this #states business is about - notice in the form how certain things hide or show depending on what you select? These things are done with something called form states, its part of Drupal core form API, which changed in 7.14.
Comment #7
nkleopas commentedHello Jeff. Tried everything you said. Unfortunately the problem remains.
Comment #8
Jeff Burnz commentedLike I said, I can't reproduce it, its on your system, without being able to see your site I can't do anything, or there is something you have omitted to tell me, maybe some vital piece of information without which I cannot emulate the conditions you are running under.
Comment #9
Jeff Burnz commentedSee the screenshot, this is what I see.
Comment #10
Jeff Burnz commentedSorry, I have really really tried to reproduce this and without more information or even access to your site I cant really do much more.
Comment #11
aung_m commentedIt happens with Drupal 7.14. When I revert back to Drupal 7.12 Set a max width re-appears again!
Comment #12
Jeff Burnz commentedThe screen shot in #9 is Drupal 7.14, I suspect you need to update your dev version of Adaptivetheme.
Comment #13
Jeff Burnz commentedBah, ignore #12, you are using 7.x-2.2, yes this is a problem due to the changes in 7.14 this will break, I am updating the form states to work in 7.14, the new 7.x-2.x version will be out in a few days.
Not really a bug, just need to chase the changes in Drupal core.
Comment #14
Jeff Burnz commentedRecent commits updated form states in 7.x-3.x, I found a few inconsistencies in how I was doing this in the layout forms, I need to check my commits if I updated 7.x-2.x yet or not, memory fails me, pretty sure I still need to do this.
Comment #15
progone commentedAny updates, I've been having this issue since the 7.14 update was released. I don't see any new updates either.
Comment #16
Jeff Burnz commented7.x-2.x was updated several days ago. Use the dev or wait a bit for a new release, will be a few days or sooner, depends how things go with the final round of clean install tests.
Comment #17
Jeff Burnz commentedThis is definitively fixed now. Just watch out for this issue: #1618288: Standard layout max-width is empty - you forgot to enter a value for the max width!
Comment #18
progone commentedThanks Man, I'm going to try it now.