Skip to content

Commit 8b8dd8d

Browse files
committed
Add MouseEnter & MouseMove Event in PopupCloser
This commit adds MouseEnter and MouseMove event in the PopupCloser, as Edge browser can only react to mouse movements and not mouse clicks. This also makes the beahviour of PopupCloser consistent with AbstractHoverInformationControl:Closer. fixes eclipse-platform/eclipse.platform.swt#2072
1 parent 1315540 commit 8b8dd8d

File tree

1 file changed

+8
-0
lines changed
  • bundles/org.eclipse.jface.text/src/org/eclipse/jface/text/contentassist

1 file changed

+8
-0
lines changed

bundles/org.eclipse.jface.text/src/org/eclipse/jface/text/contentassist/PopupCloser.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -109,6 +109,9 @@ public void install(ContentAssistant contentAssistant, Table table, AdditionalIn
109109

110110
fDisplay.addFilter(SWT.Deactivate, this);
111111
fDisplay.addFilter(SWT.MouseUp, this);
112+
113+
fDisplay.addFilter(SWT.MouseMove, this);
114+
fDisplay.addFilter(SWT.MouseEnter, this);
112115
}
113116
}
114117

@@ -130,6 +133,9 @@ public void uninstall() {
130133

131134
fDisplay.removeFilter(SWT.Deactivate, this);
132135
fDisplay.removeFilter(SWT.MouseUp, this);
136+
137+
fDisplay.removeFilter(SWT.MouseMove, this);
138+
fDisplay.removeFilter(SWT.MouseEnter, this);
133139
}
134140
}
135141

@@ -206,6 +212,8 @@ else if (event.type == SWT.MouseVerticalWheel)
206212
}
207213
break;
208214

215+
case SWT.MouseEnter:
216+
case SWT.MouseMove:
209217
case SWT.MouseUp:
210218
if (fAdditionalInfoController == null || fAdditionalInfoController.getInternalAccessor().isReplaceInProgress())
211219
break;

0 commit comments

Comments
 (0)