fix: optimize event listener cleanup and scope wheel event to sidebar

- Make event listener cleanup conditional based on platform
- Add container.contains check to wheel event to prevent affecting whole app
- Fixes code review feedback from Greptile
Co-authored-by: Anthony <AnthonyRonning@users.noreply.github.com>

Author: github-actions[bot]

frontend/src/components/ChatHistoryList.tsx

@@ -290,6 +290,9 @@ export function ChatHistoryList({
 consthandleWheel=(e: WheelEvent)=>{
 if(!isDesktopPlatform||isPullRefreshing)return;
 
+// Only handle if the event target is within our container
+if(!container.contains(e.targetasNode))return;
+
 // Check if we're at the top and trying to scroll up
 if(container.scrollTop===0&&e.deltaY<0){
 // Prevent default to avoid browser overscroll bounce
@@ -318,10 +321,14 @@ export function ChatHistoryList({
 container.removeEventListener("touchstart",handleTouchStart);
 container.removeEventListener("touchmove",handleTouchMove);
 container.removeEventListener("touchend",handleTouchEnd);
-container.removeEventListener("wheel",handleWheel);
-container.removeEventListener("mousedown",handleMouseDown);
-window.removeEventListener("mousemove",handleMouseMove);
-window.removeEventListener("mouseup",handleMouseUp);
+
+if(isDesktopPlatform){
+container.removeEventListener("wheel",handleWheel);
+}else{
+container.removeEventListener("mousedown",handleMouseDown);
+window.removeEventListener("mousemove",handleMouseMove);
+window.removeEventListener("mouseup",handleMouseUp);
+}
 };
 },[containerRef,isPullRefreshing,handleRefresh]);