Okay
  Public Ticket #3762750
Non-Standard Script Type in WPBakery Implementation
Open

Comments

  • blickgerecht started the conversation

    Dear Support Team,

    I am encountering an issue related to the theme's integration with the WPBakery plugin, which appears to include a custom script element in the frontend HTML source code as follows:

    <script type="text/html" id="wpb-modifications"> window.wpbCustomElement = 1; </script>

    While this non-standard type="text/html" setting does not typically cause issues, it poses challenges when used in conjunction with caching or static site generation plugins, such as Simply Static. These tools can misinterpret or alter this script, resulting in errors on the site.

    To avoid these issues and to ensure compatibility with a broader range of caching and static site plugins, could you consider modifying this script element to comply with standard HTML practices in future theme updates? This would save me from having to manually adjust the code after each update.

    Additionally, if possible, could you clarify the specific function of this line in the theme's frontend output? Understanding its purpose may help me address or troubleshoot any related issues more effectively.

    Thank you very much for your assistance and for considering this update in future releases.

    Best,
    Roland

  •  1,923
    Judith replied

    Hi Roland,

    Thanks for writing to us.

    Salient uses the Salient WPBakery plugin which is bundled with the theme. Please let us know whether you are referring to the Salient WPBakery plugin or the native WPBakery plugin: https://wpbakery.com/

    Please confirm so that we can provide a more helpful response,

    I look forward to your response.

  • blickgerecht replied

    Hi Judith,

    we only use the "Salient WPBakery plugin" that comes with the theme. That's why I was opening that ticket here.

    Best,
    Roland

  •  1,923
    Judith replied

    Hello Roland,

    Thanks for writing back.

    We appreciate your sharing your feedback and findings with us as well as recommendations. We take seriously our user feedback to help us improve our products.

    I am forwarding this to the developer to be able to assist and as well further respond to your query.

    I appreciate your patience and understanding.

    Best Regards.

  • blickgerecht replied

    I already found the file where the line is echoed:
    /js_composer_salient/include/classes/core/class-vc-modification.php

    Probably that will help …

  •  1,923
    Judith replied

    Hi there,

    Thanks for writing back.

    We will compile this as well while sharing it with the developer.

    Thanks for pointing it out.

    Best Regards,

  •  1,086
    ThemeNectar replied

    Hey Roland,

    The output of that script attribute is a core feature of WPBakery Page Builder and not something we’ve modified in Salient. I’ll forward this issue to the plugin’s developers, but you could also leave a comment on their page to add additional weight to your concern. https://codecanyon.net/item/wpbakery-page-builder-for-wordpress/242431/comments

    Kind regards

  • blickgerecht replied

    Hi,

    I left a comment there, but as we didn't purchase that plugin, I don't know if they will read it or work on it.

    Can't you change that line manually in your updates? As that really is causing trouble. I didn't find any proof that it's even needed in the front-end either. Deleted it, nothing changed and I couldn't find the variable in any other file within Salient or the plugin.

    Best,
    Roland

  • blickgerecht replied

    So that's the answer. We actually purchased WPBakery some time ago, but we don't have active support at the moment.

    Could you offer the information to them if you not already have? We really would love to see a fix for that.

    Attached files:  Bildschirmfoto 2024-11-14 um 11.55.55.png

  •  1,086
    ThemeNectar replied

    Hey Roland,

    I've also sent them a direct message regarding your concerns.

    Kind regards

  • blickgerecht replied

    Hi! Any news on that?

  •  1,086
    ThemeNectar replied

    Hey Blickgerecht,

    Unfortunately, they haven’t commented on this yet. Because of that, I’ve added it as an item for us to look into more deeply and potentially modify from our side.

    Kind regards

  • blickgerecht replied

    Hi!

    So did you already implement a fix in 17.0.5?

    Best,
    Roland

  •  1,086
    ThemeNectar replied

    Hey Roland,

    It will be included in the next release of the Salient WPBakery Page Builder plugin, which is due out next week :)

    Kind regards