nodegarden icon indicating copy to clipboard operation
nodegarden copied to clipboard

Help with my site pls

Open Chillstice opened this issue 7 years ago • 8 comments

I know you probably hate the proposition of end-users asking for help with their websites, but there are some strange issues I've been running into that others aren't which means it's localized to what I'm doing.

I've been modifying the nodegarden script to match my needs and I've got it functional, but some functions in the js file aren't running.

  1. When the container div is behind a wrapper (div is ordered before wrapper in the html) the click function works, but it doesn't create any nodes. Removing the wrapper or setting the z-index higher than the wrapper returns functionality.

  2. For whatever reason the resize function isn't working for my site. It's not a problem with the script per-say because it's working on the demo and on other sites that utilize nodegarden. I have no leads. When refreshing the site with a tiny browser window and resizing it after the document loads, it's obvious that the canvas isn't resizing.

  3. The animation speed seems to be tied to framerate. The framerate of my main monitor is 144hz (standard is 60). When tweaking the animation parameters to alter the perceived speed, other devices show it at less than half that speed. It's inconsistent between different framerate devices.

I didn't want to spam independent issues, but rather to put them all in one post. I know you're probably busy working on new and exciting projects, but I would appreciate some help with this.

The site is https://chillstice.com

Chillstice avatar Feb 10 '18 05:02 Chillstice

I'm just going to close this, These issues are not that severe.

Chillstice avatar Mar 11 '18 20:03 Chillstice

Oh, sorry I have totally missed this!!

pakastin avatar Mar 11 '18 20:03 pakastin

  • I will fix 3. ASAP – that’s easy 🙂

  • I’ll check out 1. and 2. tomorrow morning (GMT+3), I’m sure they’re not a big issue

pakastin avatar Mar 11 '18 20:03 pakastin

I made a big update, hope everything works now!

pakastin avatar Mar 12 '18 22:03 pakastin

The animation speed is fixed, but I think making the container div position: fixed in combination with the z-index being behind the wrapper (as I've done) causes it to fail to create new nodes on click. Also the div/ canvas doesn't seem to resize either. I have converted my version of the code over to the latest version and it's live at https://chillstice.com

I will experiment later, but I know that changing the z-index of the container div/ canvas to above everything else solves the problem of nodes not being created on click.

Chillstice avatar Mar 18 '18 20:03 Chillstice

Also I've just noticed that text is unselectable (some text is marked as unselectable), but all text and text areas i.e. in the form are not selectable anymore. I think that might have been caused by changes trying to solve the no-node-on-click problem.

EDIT: with some more extensive testing, I have concluded that it's something to do with nodegarden.

Chillstice avatar Mar 19 '18 04:03 Chillstice

I disabled e.preventDefault around line 100 and the click functionality returned. Still don't know why clicking won't create any new nodes and the resize issue is a bit annoying. Other than that, I'm pretty happy with the work you've put into this.

I've decided that the issue is when there's a div of any type with a higher z-index in front of the canvas. This appears to cause nodes to fail to be created.

Chillstice avatar Mar 31 '18 17:03 Chillstice

That’s weird, will look into it

pakastin avatar Mar 31 '18 17:03 pakastin