React | HTML Components: Adding component dependency to useImperativeHandle

[rokotyan]

Fixes #472

Updated dependencies for the useImperativeHandle hook to include the component in its dependency array. This ensures that references to the component are correctly updated whenever it changes, leading to improved synchronization and reliability in the component's behavior.

[lee00678]

@rokotyan maybe I'm not testing this currently, but the mapRef in the leaflet-map example https://github.com/f5/unovis/blob/main/packages/shared/examples/advanced-leaflet-map/advanced-leaflet-map.tsx#L23-L32 are still always null for both events.

[rokotyan]

@lee00678 How are you testing it? I quickly tried it on my end with the dev app, and it worked.

[lee00678]

@lee00678 How are you testing it? I quickly tried it on my end with the dev app, and it worked.

I tried it with the leaflet-map gallery example. I just added console.log in those two click functions here. Sees to me myRef.current is always null.

[rokotyan]

@lee00678 Worked on my end. Have you built @unovis/react before trying?

[lee00678]

@lee00678 Worked on my end. Have you built @unovis/react before trying?

That's odd. I did npm run install:clean. I also just tried to build react separately by doing npm run build:react, still no luck. The code changes are there.

[lee00678]

@rokotyan I can't seem to get this fix working.

I added a console.log inside react/src/html-component/leaflet-map/index.tsx to ensure react was built.

My myRef.current is always null.

[rokotyan]

@lee00678 And what happens when you click on current? It doesn't seem to be null

[lee00678]

@lee00678 And what happens when you click on current? It doesn't seem to be null

You are right. I thought it's returning an empty object. The component is inside. Sorry.