I prepared a fix that moves the socket.on('update') listener into a useEffect with cleanup to prevent duplicate registrations.
However, the Gerrit repository labs/tools/VideoCutTool appears to be read-only (project state does not permit write), so I’m unable to submit the patch.
Please advise on the preferred contribution path (GitHub PR or alternative repo).

"Hi @Jdlrobson-WMF , T415809 seems closely tied to the layout bug in T415709. Since I'm already working in the Vector code, I'm happy to tackle this feedback issue as well. Should I take it on, or do you have a specific fix in mind?"

"Hi @Aklapper, I did use an AI tool to help format my response and improve the clarity of my English, as I wanted to make sure my suggestions for the team were professional and easy to understand. However, the options I proposed (reordering the menu, scoped CSS, or shortening text) are based on my own analysis of the bug and the feedback I received from Jon in Gerrit. I'm fully committed to the task and want to make sure the solution aligns with the team's preferences!"

"Hi @Jdlrobson-WMF, thanks for the feedback. I see your point about avoiding long-term changes to the core menu height for a temporary feature. Since the issue is primarily visible in the pinned sidebar state where the default dropdown scroll logic might not apply, what behavior would the team prefer?

@matmarex t thank you very much for the review and for approving the patch
You’re right — my description didn’t accurately reflect what the change actually does. I should have tested and described the behavior more carefully before submitting.

Review I changed

Review I changed

Sorry for the accidental move.

Review I changed

Review I changed

Review I changed

I have completed the work for this task and submitted the patch. I’ve tested the changes on my end and they seem to be working as expected. Please take a look and let me know if any improvements or modifications are needed. view patch on gerrit

I have completed the work for this task and submitted the patch. I’ve tested the changes on my end and they seem to be working as expected. Please take a look and let me know if any improvements or modifications are needed. view patch on gerrit

@Aklapper Thanks for the clarification. That was by mistake on my side and not intentional. I’ll make sure to be more careful with project tags in the future.

I have completed the work for this task and submitted the patch. I’ve tested the changes on my end and they seem to be working as expected. Please take a look and let me know if any improvements or modifications are needed. Thank you for your time and guidance.

I have completed the work for this task and submitted the patch. I’ve tested the changes on my end and they seem to be working as expected. Please take a look and let me know if any improvements or modifications are needed. Thank you for your time and guidance.

1. Set the Callback URL