fix: showStipes not working (#266)

[yotamberk]

closes #266

[Thomaash]

👍

[vis-bot]

🎉 This PR is included in version 7.0.2 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[Async0x42]

Unfortunately this fix breaks clustering when showStipes is false. Using the example page, if you disable showStipes, it throws an exception because the dot/line don't exist.

https://codepen.io/Async0x42/pen/RwPQYvY?editable=true

ClusterItem.js:542 Uncaught TypeError: Cannot read property 'offsetHeight' of undefined
    at e.value (ClusterItem.js:542)
    at e.o (ClusterItem.js:109)
    at Group.js:1126
    at Object.forEach (vis-util.js:38)
    at t.value (Group.js:1125)
    at p (Group.js:378)
    at t.value (Group.js:422)
    at Array.r (Group.js:533)
    at ItemSet.js:837
    at Object.forEach (vis-util.js:38)

[Thomaash]

Hi @Async0x42,

if a PR broke something it's generally much better idea to open a new issue and link the offending PR from there. Comments on closed PRs are very prone to getting lost and forgotten rather than addressed.

[Async0x42]

Hi @Async0x42,

if a PR broke something it's generally much better idea to open a new issue and link the offending PR from there. Comments on closed PRs are very prone to getting lost and forgotten rather than addressed.

Good to know, I'll post a new issue, thanks!