Fix duplicate body event listeners by moving logic from render to lifecycle methods

[priyapathak39]

Problem
SidebarMobile calls _toggleBodyListener(isOpen) inside render(), causing event listeners to be added/removed on every render. This can lead to duplicate listeners and inconsistent sidebar behavior.

Solution
Move event listener handling to lifecycle methods:

• componentDidMount
• componentDidUpdate (when isOpen changes)
• componentWillUnmount

File
src/components/SidebarMobile/SidebarMobile.jsx

Steps to Reproduce

1. Run the project
2. Open sidebar in mobile view
3. Toggle multiple times
4. Click outside

Expected

• Sidebar closes reliably
• No duplicate event listeners

[priyapathak39]

I’d like to work on this issue. Please assign it to me.

[sujalgoel]

Took a look at the code. _handleBodyClick is a class field arrow function, so addEventListener always gets the same reference. The browser deduplicates by (event, callback, capture) tuple, so listeners don't actually pile up.

The real problem is side effects in render(). React StrictMode double-invokes renders in dev, which would cause the listener to get yanked and re-added on every render cycle. Still worth fixing, just not for the reason described. Working on a PR.