Javascript error when pages containing TOC are resized

[deepfriedbrain]

Environment

• Minimal Mistakes version: 4.16.2
• Ruby gem or remote theme version: NA
• Jekyll version: 3.8.5
• Git repository URL: NA
• GitHub Pages hosted (if yes provide URL to site): NA
• Operating system: MacOS 10.14.1

Expected behavior

If any page that contains a TOC is resized, the following Javascript error is thrown in the console:

main.min.js:9 Uncaught TypeError: Cannot read property 'sort' of undefined
    at i (main.min.js:9)
    at main.min.js:9

I'm using the default version of the theme (no fork), and have not modified main.min.js. I'm using Chrome 74.x (latest version).

[mmistakes]

Not seeing this on the demo site (for example this page doesn't have a TOC and I see no JavaScript errors in the console on resize).

Do you have a reproducible test case repository?

[deepfriedbrain]

I think you misread my post. It appears on pages that have TOC. I can reproduce it on the demo site.

Refer to this page: https://mmistakes.github.io/minimal-mistakes/layout-table-of-contents-post/

[mmistakes]

OK. I'm able to reproduce. When I rolled back jQuery the other day and uglified the scripts there were a few warnings related to the Gumshoe and scroll scripts. Perhaps something is getting mangled there when everything is concatenated.

Will have to play around.

[mmistakes]

It's Gumshoe related. When I use the unconcatenated versions of the scripts the error has more context.

TypeError: contents is undefined[Learn More] gumshoe.js:111:3
    sortContents http://127.0.0.1:4000/test/assets/js/plugins/gumshoe.js:111
    timeout http://127.0.0.1:4000/test/assets/js/plugins/gumshoe.js:412

[mmistakes]

Updating Gumshoe to v5.1.1 seems to have fixed it.

[deepfriedbrain]

Thank you. Was the jquery change from minified version to the full version intentional?

[mmistakes]

Yes. The uglify script I run minifies all the JavaScript when it concatenates. It does a better job of that if you run it on unminified code first... shaved about 2k off the file size.

[deepfriedbrain]

Yes, makes sense. I just wanted to confirm. The error has been resolved. Thanks!

BTW, I did notice that the jquery.min.js has been left in eventhough it's not used.