- Implemented V-centering caret at CodeMap navigation

- Forced showing the whole suggestion list at startup of explicitly triggered auto-completion.

Author: oleg-shilo

src/CSScriptIntellisense/Plugin.cs

@@ -941,8 +941,11 @@ static void ShowSuggestionList(bool snippetsOnly, bool memberInfoWasShowing = fa
 
                 var document = Npp.GetCurrentDocument();
                 Action<ICompletionData> OnAccepted = data => OnAutocompletionAccepted(data, snippetsOnly);
-
-                autocompleteForm = new AutocompleteForm(OnAccepted, items, NppEditor.GetSuggestionHint());
+                
+                // no need to pass initial SuggestionHint as it may swallow (auto accept) the whole autocompletion window 
+                // in case of the hint to be the full match of one of the items. Just do it for a better UX 
+                
+                autocompleteForm = new AutocompleteForm(OnAccepted, items, null);
                 autocompleteForm.Left = point.X;
                 autocompleteForm.Top = point.Y + document.TextHeight(document.GetCurrentLineNumber());
                 autocompleteForm.FormClosed += (sender, e) =>

src/CSScriptNpp/Dialogs/CodeMapPanel.cs

@@ -74,8 +74,15 @@ void MembersList_MouseDoubleClick(object sender, MouseEventArgs e)
                         int prevLineEnd = document.PositionFromLine(currentLineNum) - Environment.NewLine.Length;
                         int topScrollOffset = currentLineNum - document.GetFirstVisibleLine();
 
-                        document.GotoLine(info.Line);
-                        document.SetFirstVisibleLine(info.Line - topScrollOffset);
+                        document.GotoLine(info.Line - 1); // document counts line as 0-based
+                        document.VerticalCentreCaret();
+
+                        //// GetLineVisible just does not work, it returns true always
+                        // var caretIsVisible = document.GetLineVisible(currentLineNum);
+                        // if (caretIsVisible)
+                        //     document.SetFirstVisibleLine(info.Line - topScrollOffset);
+                        // else
+                        //     document.EnsureVisible(info.Line - 1);
                     }
                 }
                 catch { } //it is expected to fail if the line does not contain the file content position spec. This is also the reason for not validating any "IndexOf" results.

src/Deploy_64.cmd

@@ -10,7 +10,7 @@ md "%plugin_root%\CSScriptNpp"
 md "%plugin_root%\CSScriptNpp\Mdbg"
 
 echo -------------------
-copy "output\plugins\CSScriptNpp.x64.dll" "%plugin_root%\CSScriptNpp.x64.dll"
+copy "output\plugins\CSScriptNpp.x64.dll" "%plugin_root%\CSScriptNpp.dll"
 xcopy output\plugins\CSScriptNpp\*.* "%plugin_root%\CSScriptNpp" /Y
 xcopy output\plugins\CSScriptNpp\Mdbg\*.* "%plugin_root%\CSScriptNpp\Mdbg" /Y