scrolldir | 0 dependency JS plugin to leverage scroll direction | Frontend Framework library

First of all selectedMenu = "Clone" is not comparing anything. It is assigning the word Clone to the selectedMenu variable.

You should use == and since selectedMenu is a jQuery object you should check agains its text. so

useEffect hook takes second argument, an array. If some value in that array changes then useEffect hook will run again

If you leave that array empty useEfect will run only when component mount - (basicaly like ComponentDidMount lifecycle method)

And if you omit that array useEffect will run every time component rerenders

For example:

useEffect runs only once, when component mounts:

Yes! setState() is an async operation but after every setState call your component re-renders and eventually it can have performance issues when called very frequently like by a scroll handler.

Ideally, if changing a variable has nothing to do with updating the DOM (more specifically virtual DOM) then it shouldn't be kept as a state variable rather it can made class variable like

so basically the problem here is that the newly added rows in the table will not get the bindings from the initial load of the page. so the fix needed here is to bind the event to the main document by using this

To achieve this you could implement a new context menu item which clones the parent() element of one which triggered the event.

Also note that you can tidy up the logic by using a data attribute to specify the action to be performed instead of matching the text. Try this: