Skip to main content

Brizy and Themes' Default Template Bug

John Garvin

When using Brizy Builder on any of various themes' default templates that already load header via the get_header() hook, Brizy appears to load get_header() again, which seems to replace the_content() with this additional call before instantiating the loop. 

Example markup when using Brizy on any theme file other than Brizy Blank Template

<html>
<head>
<meta>...</meta>
<style>...</style>
</head>
<body>

<div> <!-- default theme opening tag -->

(Brizy should be using the_content() here but beforehand Brizy also adds the following markup which has already been added above via get_header hook in the theme template)

<html>
<meta>...</meta>
<style>...</style>
<body>... no closing tag added

(brizy content rendered correctly here)

</div> <!-- default theme closing tag -->

</body> <!-- default theme closing tag -->
</html> <!-- default theme closing tag -->

This is latest versions of Brizy and Brizy Pro, latest version of WordPress, multiple themes.

Hope you can get a resolution to this ASAP, I often prefer to use default template headers and footers and Brizy for content alone rather than relying solely on the Brizy Blank Template.

0

Comments

10 comments

Date Votes
  • Sandra Prunici
    • Edited

    Hi,

    No, it isn't an issue. It is the right way of working the default template. When using this template, you use the theme template and it creates 3 parts for the page. In the first is added the theme header, in the second is added a container for the content and in the third - the theme footer. In the second part which is a container, it is added the content you created in Brizy.

    Thanks!

    Best regards,
    Sandra

    0
  • John Garvin
    • Edited

    I'm struggling to understand how this is correct. Brizy should not be outputting a second <html><head><body> within the main <body> after the main template already outputs this. A DOM tree should not contain two <html><head><body> tags. This is clearly an issue and is not correct.

    0
  • Heiko Gaß
    • Edited

    I'll take notes here. I also only use the standard template in the brizy builder, header and footer from the theme (blocksy or kadence). There are then two body and two head html tags on one page. I always thought for correct html the body tag or the head tag should only exist once.
    And I have <html lang = "de-DE"> normally on the page and once <html lang = "en"> with head, body etc. in the <div class = "ct-builder-content">

    Can you clarify the situation, please?

    Thank you

    0
  • John Garvin

    Yes, you are correct. There should not be duplicate DOM object tags for root <html><head><body>...

    I've done further testing this morning.

    Brizy Pro 2.0.4 does not have this issue. Brizy Pro 2.0.7 does have this issue. So not sure in which version it began.

    I have tested with multiple themes, both local and live environments, with no other plugins installed.

    0
  • Heiko Gaß

    Look here:
    https://support.brizy.io/hc/en-us/community/posts/360074163231-duplicate-DOM-tree-once-again

    0
  • Sandra Prunici
    • Edited

    Hi there,

    Thank you for your messages and reports. Our team investigated this problem and already fixed it. We added the fix in our last update. Could you please update the Brizy Free 2.016 and Brizy Pro 2.0.8?
    Thank you for your patience!

    Best regards,
    Sandra

     

    0
  • Heiko Gaß

    Yes, it works. 
    Thank you very much. You are the greatest...

    0
  • Sandra Prunici

    Thank you too! Have a great day!

    Best regards,
    Sandra

    0
  • John Garvin

    Thank you so much for looking into this issue and patching it so quickly. Very much appreciate it. :)

     

    0
  • Sandra Prunici

    It's our pleasure to help you! Great to know that all is fine now!
    Have a great day!

    Best regards,
    Sandra

    0

Please sign in to leave a comment.

Didn't find what you were looking for?

Ask a Question