Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Without this fix there are duplicated Key Binding Triggers on WebKit as the fix provided by BrowserKeyBindingForwarderInstaller is only an Issue on Chrome and Edge
  • Loading branch information
N1k145 committed Apr 29, 2024
1 parent 10fee67 commit e60c9a1
Showing 1 changed file with 4 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@
import org.apache.logging.log4j.Logger;
import org.eclipse.core.commands.common.EventManager;
import org.eclipse.core.resources.IMarker;
import org.eclipse.core.runtime.Platform;
import org.eclipse.e4.core.contexts.IEclipseContext;
import org.eclipse.glsp.graph.GModelElement;
import org.eclipse.glsp.ide.editor.GLSPServerManager;
Expand Down Expand Up @@ -332,7 +333,9 @@ protected void mouseEnteredBrowser(final MouseEvent event) {

protected void installBrowserFunctions() {
// browser functions are automatically disposed with the browser
new BrowserKeyBindingForwarderInstaller(context.get(IBindingService.class)).install(browser);
if (Platform.getOS().equals(Platform.WS_WIN32)) {
new BrowserKeyBindingForwarderInstaller(context.get(IBindingService.class)).install(browser);
}
new BrowserFocusControlInstaller().install(browser);
new BrowserContextMenuInstaller().install(browser);
}
Expand Down

0 comments on commit e60c9a1

Please sign in to comment.