resolve comment

Author: xinyi-gong

com.microsoft.copilot.eclipse.ui/src/com/microsoft/copilot/eclipse/ui/chat/SourceViewerComposite.java

@@ -128,7 +128,7 @@ public void focusGained(FocusEvent e) {
 
       @Override
       public void focusLost(FocusEvent e) {
-        hideActionButtons(null);
+        hideActionButtons();
       }
     });
 
@@ -144,7 +144,7 @@ public void mouseEnter(MouseEvent e) {
 
       @Override
       public void mouseExit(MouseEvent e) {
-        hideActionButtons(new Point(e.x, e.y));
+        hideActionButtons();
       }
     });
 
@@ -229,23 +229,27 @@ private void showActionButtons() {
     actionsComposite.setVisible(true);
   }
 
-  private void hideActionButtons(Point cursorLocation) {
+  private void hideActionButtons() {
     if (this.actionsComposite == null || this.actionsComposite.isDisposed()) {
       return;
     }
     SwtUtils.invokeOnDisplayThreadAsync(() -> {
       if (actionsComposite == null || actionsComposite.isDisposed()) {
         return;
       }
-      if (!isCursorWithinCodeBlockActions(cursorLocation) && !isFocusWithinCodeBlockActions()) {
+      if (!isCursorWithinCodeBlockActions() && !isFocusWithinCodeBlockActions()) {
         actionsComposite.moveBelow(sourceViewer.getTextWidget());
         actionsComposite.setVisible(false);
       }
     }, this);
   }
 
-  private boolean isCursorWithinCodeBlockActions(Point cursorLocation) {
-    return cursorLocation != null && actionsComposite.getBounds().contains(cursorLocation);
+  private boolean isCursorWithinCodeBlockActions() {
+    Point cursorLocation = actionsComposite.getDisplay().getCursorLocation();
+    Point actionsLocation = actionsComposite.toDisplay(0, 0);
+    Point actionsSize = actionsComposite.getSize();
+    Rectangle actionsBounds = new Rectangle(actionsLocation.x, actionsLocation.y, actionsSize.x, actionsSize.y);
+    return actionsBounds.contains(cursorLocation);
   }
 
   private boolean isFocusWithinCodeBlockActions() {