Javascript Error in tagdiv_theme.js

Posted in: Newspaper
Post count: 17

Hi,

we’re getting a javascript error on our custom built homepage (we created our own page and designed it with Visual Composer). The error is following: “Uncaught TypeError: Cannot read property ‘top’ of undefined at wp-content-themes-Newspaper-js-tagdiv_theme.min-7.8.1.js:278”. This error is visible through a browser console like the one in the Chrome Dev Tools.

To find the source of the error I disabled all plugins except Visual Composer, WooCommerce and WooCommerce Memberships. Unfortunally I cannot disable the Woocommerce Plugins since that makes our frontend crash and not load properly.

The error can easily come from one of our customisation. I just wanted to ask, if you have any idea where this error could come from.

I also found a quick fix for this problem, you just need to check, if the object contains any elements with:

if (cur_item_ref.length) {
cur_item_ref.content_top = cur_item_ref.content_jquery_obj.offset().top;
}

and also for the second one:

if (cur_item_ref.length) {
cur_item_ref.sidebar_top = cur_item_ref.sidebar_jquery_obj.offset().top;
}

Maybe this helps someone with the same issue or you could even consider to add this to the script for compatibility reasons?

Thanks in advance

Post count: 22421

Hello,
There are no errors in our js code as we cannot release broken code to our customers. Without modifications or plugins, our theme js code runs fine and does not provide errors.
Thank you for posting the solution for this issue, as it will help other who face the same issue.
Thank you!

Post count: 17

Hello,

I just found that my fix breakes the smart sidebar. I fixed it by checking the offset directly:

					
if (cur_item_ref.content_jquery_obj.offset() != undefined){
  cur_item_ref.content_top = cur_item_ref.content_jquery_obj.offset().top;
}

if (cur_item_ref.sidebar_jquery_obj.offset() != undefined){
  cur_item_ref.sidebar_top = cur_item_ref.sidebar_jquery_obj.offset().top;
}

(around line 6148 ff)

  • This reply was modified 9 years by euw.
Viewing 3 posts - 1 through 3 (of 3 total)
You must be logged in to reply to this topic.