Also stop activating panes on setting the selection in VB6

rubberduck-vba · Feb 13, 2019 · eb592a1 · eb592a1
1 parent b01f450
commit eb592a1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 35 deletions.
diff --git a/Rubberduck.VBEditor.VB6/SafeComWrappers/VB/CodePane.cs b/Rubberduck.VBEditor.VB6/SafeComWrappers/VB/CodePane.cs
@@ -1,6 +1,4 @@
-using System;
-using Rubberduck.VBEditor.SafeComWrappers.Abstract;
-using Rubberduck.VBEditor.WindowsApi;
+using Rubberduck.VBEditor.SafeComWrappers.Abstract;
 using VB = Microsoft.Vbe.Interop.VB6;
 
 // ReSharper disable once CheckNamespace - Special dispensation due to conflicting file vs namespace priorities
@@ -90,35 +88,6 @@ private void SetSelection(int startLine, int startColumn, int endLine, int endCo
                 return;
             }
             Target.SetSelection(startLine, startColumn, endLine, endColumn);
-            ForceFocus();
-        }
-
-        private void ForceFocus()
-        {
-            if (IsWrappingNullReference)
-            {
-                return;
-            }
-
-            Show();
-
-            using (var vbe = VBE)
-            using (var mainWindow = vbe.MainWindow)
-            using (var window = Window)
-            {
-                var mainWindowHandle = mainWindow.Handle();
-                var handle = mainWindow.Handle().FindChildWindow(window.Caption);
-
-                if (handle != IntPtr.Zero)
-                {
-                    NativeMethods.ActivateWindow(handle, mainWindowHandle);
-                }
-                else
-                {
-                    _logger.Debug(
-                        "CodePane.ForceFocus() failed to get a handle on the MainWindow.");
-                }
-            }
         }
 
         public void Show()

diff --git a/Rubberduck.VBEditor.VBA/SafeComWrappers/VB/CodePane.cs b/Rubberduck.VBEditor.VBA/SafeComWrappers/VB/CodePane.cs
@@ -1,6 +1,4 @@
-using System;
-using Rubberduck.VBEditor.SafeComWrappers.Abstract;
-using Rubberduck.VBEditor.WindowsApi;
+using Rubberduck.VBEditor.SafeComWrappers.Abstract;
 using VB = Microsoft.Vbe.Interop;
 
 // ReSharper disable once CheckNamespace - Special dispensation due to conflicting file vs namespace priorities