Updated tooltip positioning for the refactor

Author: billti

Default.sublime-keymap

@@ -73,15 +73,6 @@
             { "key": "selector", "operator": "equal", "operand": "text.find-refs" }
         ]
     },
-    {
-        "keys": [ "alt+,"],
-        "command": "typescript_signature_popup",
-        "context": [
-            { "key": "selector", "operator": "equal", "operand": "source.ts" },
-            { "key": "tooltip_supported", "operator": "equal", "operand": true}
-
-        ]
-    },
     {
         "keys": [ "ctrl+t", "ctrl+q" ],
         "command": "typescript_quick_info_doc",
@@ -136,6 +127,15 @@
             { "key": "is_popup_visible"}
         ]
     },    // In case when auto match is enabled, only format if not within {}
+    {
+        "keys": [ "alt+,"],
+        "command": "typescript_signature_popup",
+        "context": [
+            { "key": "selector", "operator": "equal", "operand": "source.ts" },
+            { "key": "tooltip_supported", "operator": "equal", "operand": true}
+
+        ]
+    },
     {
         "keys": [ "}" ],
         "command": "typescript_format_on_key",

TypeScript.sublime-commands

@@ -10,5 +10,6 @@
  { "caption" : "TypeScript: Rename", "command": "typescript_rename" },
  { "caption" : "TypeScript: PasteAndFormat", "command": "typescript_paste_and_format" },
  { "caption" : "TypeScript: FormatLine", "command": "typescript_format_line" },
+ { "caption" : "TypeScript: SignatureInfo", "command": "typescript_signature_popup" },
  { "caption" : "TypeScript: FormatBlock", "command": "typescript_format_brackets" }
 ]

libs/popup_manager.py

@@ -28,6 +28,9 @@ def __init__(self, proxy):
         self.signature_index = 0
         self.current_parameter = 0
 
+        # Used to avoid looping
+        self.suppress_onhidden = 0
+
     def queue_signature_popup(self, view):
         cursor = view.rowcol(view.sel()[0].begin())
         point = Location(cursor[0] + 1, cursor[1] + 1)
@@ -64,6 +67,7 @@ def on_response(self, responseJson, view):
         span_end = view.text_point(
             arg_span["end"]["line"] - 1,
             arg_span["end"]["offset"] - 1)
+
         arg_region = sublime.Region(span_start, span_end)
         view.add_regions('argSpan', [arg_region],
                          flags=sublime.HIDDEN)
@@ -76,15 +80,37 @@ def display(self):
         popup_text = PopupManager.html_template.substitute(popup_parts)
 
         log.debug('Displaying signature popup')
-        if not self.current_view.is_popup_visible():
-            self.current_view.show_popup(
-                popup_text,
-                sublime.COOPERATE_WITH_AUTO_COMPLETE,
-                on_navigate=self.on_navigate,
-                on_hide=self.on_hidden,
-                max_width=800)
-        else:
-            self.current_view.update_popup(popup_text)
+
+        arg_region = self.current_view.get_regions('argSpan')[0]
+        location = arg_region.begin()  # Default to start of arg list
+
+        # If the cursor is not in the first line of the arg list, set the popup
+        # location to first non-whitespace, or EOL, of the current line
+        cursor_point = self.current_view.sel()[0].begin()
+        opening_line = self.current_view.line(arg_region.begin())
+        if(not opening_line.contains(cursor_point)):
+            cursor_line_start = self.current_view.line(cursor_point).begin()
+            location = self.current_view.find(
+                r'\s*?(?=[\S\n\r]|$)',
+                cursor_line_start
+            ).end()
+
+        # Need to hide/redisplay in case location moved, but this triggers an
+        # on_hidden loop if we don't ignore the event (which occurs later)
+        # from the hide message
+        if self.current_view.is_popup_visible():
+            self.suppress_onhidden += 1
+            log.debug('+suppress_onhidden: {0}'.format(self.suppress_onhidden))
+            self.current_view.hide_popup()
+
+        self.current_view.show_popup(
+            popup_text,
+            sublime.COOPERATE_WITH_AUTO_COMPLETE,
+            on_navigate=self.on_navigate,
+            on_hide=self.on_hidden,
+            location=location,
+            max_width=800)
+
 
     def move_next(self):
         if not self.signature_help:
@@ -111,6 +137,13 @@ def on_navigate(self, loc):
     def on_hidden(self):
         log.debug('In popup on_hidden handler')
         if not self.current_view:
+            log.debug('No current view for popup session. Hiding popup')
+            return
+
+        # Did we hide the popup ourselves in order to redisplay it?
+        if self.suppress_onhidden > 0:
+            self.suppress_onhidden -= 1
+            log.debug('-suppress_onhidden: {0}'.format(self.suppress_onhidden))
             return
 
         cursor_region = self.current_view.sel()[0]
@@ -119,7 +152,7 @@ def on_hidden(self):
             argSpan = self.current_view.get_regions('argSpan')[0]
             if argSpan.contains(cursor_region):
                 log.debug('Was hidden while in region.  Redisplaying')
-                # Occurs on left/right movement.  Rerun to redisplay popup.
+                # Occurs on cursor movement.  Rerun to redisplay popup.
                 self.display()
         else:
             # Cleanup
@@ -200,7 +233,8 @@ def get_current_signature_parts(self):
             param = item["parameters"][self.current_parameter]
             activeParam = '<span class="param">{0}:</span> <i>{1}</i>'.format(
                 param["name"],
-                param["documentation"][0]["text"] if param["documentation"] else "")
+                param["documentation"][0]["text"]
+                    if param["documentation"] else "")
         else:
             activeParam = ''