Skip Navigation

[Resolved] Custom id’s broken after Layouts update

This thread is resolved. Here is a description of the problem and solution.

Problem: After updating Layouts I am finding invalid HTML markup around the "id" attribute for layouts rows. The custom IDs I have applied to each row are not being applied correctly, and I see markup like:

<div id="id="our-body"" class="row content " >

Solution: This bug will be addressed in a future release, but for now you can work around the issue by inserting a 1x1 grid of cells and applying your custom ID to the cell itself.

This support ticket is created 7 years, 4 months ago. There's a good chance that you are reading advice that it now obsolete.

This is the technical support forum for Toolset - a suite of plugins for developing WordPress sites without writing PHP.

Everyone can read this forum, but only Toolset clients can post in it. Toolset support works 6 days per week, 19 hours per day.

Sun Mon Tue Wed Thu Fri Sat
8:00 – 12:00 8:00 – 12:00 8:00 – 12:00 8:00 – 12:00 8:00 – 12:00 - -
13:00 – 17:00 13:00 – 17:00 13:00 – 17:00 13:00 – 17:00 13:00 – 17:00 - -

Supporter timezone: America/New_York (GMT-04:00)

This topic contains 9 replies, has 3 voices.

Last updated by davidW-12 7 years, 3 months ago.

Assisted by: Christian Cox.

Author
Posts
#527158

After updating a client site from Layouts 1.8.10 to the newest version, I am getting errors in html code. Source code no longer outputs proper div id's which have been set in Layouts. This appears to only affect layouts that use a Genesis row type (i.e. Genesis site header row, Genesis content row, Genesis sidebar row). All other id names in other row types appear to output properly. Below is an example of the html being generated by the plugin (this is a Genesis content row type):

<div id="id="our-body"" class="row content " >

This is extremely frustrating as it has broken all styling associated with these div id's. I do not want to use an old version of Layouts, but I can't update if this breaks my client's site.

I thought the issue might be related to the fact that the client hadn't updated the layout since 1.8.10, so I downloaded 1.9 and tried updating to that version. Had exactly the same problem. Restored site from backup, disabled all plugins and updated Layouts again. Same result ... broken id tags.

Please help!

Thank you,

David

#527216

Shane
Supporter

Languages: English (English )

Timezone: America/Jamaica (GMT-05:00)

Hi David,

Thank you for contacting our support forum.

Would you mind providing me with a copy of the genesis theme so that I can verify the issue and submit to our development team.

Thanks,
Shane

#527256

Shane
Supporter

Languages: English (English )

Timezone: America/Jamaica (GMT-05:00)

Hi David,

It seems you sent only the child theme.

Could you provide the parent one as well as i currently don't have a copy of the genesis theme.

Thanks,
Shane

#527269

Hi Shane,

I updated my private message with the link to the parent theme. Let me know if you need anything else.

David

#527643

Hi, Shane is off today so I'm going to take over this ticket. I hope that's okay with you.

I've downloaded your child theme and run some tests. I installed Layouts 1.8.10 and created some Layouts using Genesis Content Rows. Then I updated to Layouts 1.9.3 and edited a Layout. I added a custom ID to one of the Genesis Content Rows and saved, then viewed a post where the Layout was applied. I can confirm the same results - the "id" attribute is malformed. I also ran the test using only Genesis without a custom child theme to rule out any child theme compatibility issues, and the problem remained. So it appears this is a compatibility issue with the Genesis theme and Layouts. I have escalated this concern to our 2nd tier support team for a bit more investigation. Hopefully they will be able to recommend a workaround so we can get your ID's functioning correctly again very soon. I will update you here as soon as I hear something from them.

#527894
Screen Shot 2017-05-24 at 7.45.28 AM.png

Hi, as a workaround can you please try the following steps:
- Create a new row
- Add a Grid of Cells, 1x1, to this row at full-width (see attached screenshot)
- Apply your ID to this grid cell instead of the row
- Place your content in this new grid cell
- Remove your old row

Let me know if this works for you as a temporary fix while we work to resolve the row issue.

#528016

Christian,

Thanks for confirming that the issue was Toolset and Genesis compatibility, not a problem with my child theme. I'm glad I was able to bring this to your attention, as Genesis is so popular among developers.

I was able to create my own workaround for this client site by just using classes instead of div ID's. I will try your grid cell workaround for ID's in a project I am working on now.

I assume that this bug was found too late in the development cycle of Layouts 2.0 to get in. Please let me know when an update with a patch comes out.

Thanks,

David

#528019

That's correct, so I will keep an eye on the progress of a permanent solution to this issue and update you here when I have more information. Thanks!

#542459

Hi, just a quick update to let you know that the latest Layouts release should have resolved this issue without the need for a workaround.

#542496

Thanks for staying in touch Christian.

I'll make sure all of my installs are updated and check it out.

This ticket is now closed. If you're a Toolset client and need related help, please open a new support ticket.