fix: Fix separate signature feature

pawamoy · pawamoy · commit da6e81c89789 · 2021-11-30T20:25:37.000+01:00
diff --git a/src/mkdocstrings/handlers/python/renderer.py b/src/mkdocstrings/handlers/python/renderer.py
@@ -132,24 +132,27 @@ def update_env(self, md: Markdown, config: dict) -> None:  # noqa: D102 (ignore
         self.env.filters["crossref"] = self.do_crossref
         self.env.filters["multi_crossref"] = self.do_multi_crossref
         self.env.filters["order_members"] = self.do_order_members
-        self.env.filters["format_code"] = self.do_format_code
+        self.env.filters["format_signature"] = self.do_format_signature
 
-    def do_format_code(self, code: str, line_length: int) -> str:
-        """Format the code using Black.
+    def do_format_signature(self, signature: str, line_length: int) -> str:
+        """Format a signature using Black.
 
         Parameters:
-            code: The code to format.
+            signature: The signature to format.
             line_length: The line length to give to Black.
 
         Returns:
             The same code, formatted.
         """
         from black import Mode, format_str
 
-        if len(code) >= line_length:
-            mode = Mode(line_length=line_length)
-            return format_str(f"def {code}: pass", mode=mode)[4:-6].strip()
-        return code
+        code = signature.strip()
+        if len(code) < line_length:
+            return code
+        mode = Mode(line_length=line_length)
+        formatted = format_str(f"def {code}: pass", mode=mode)
+        # remove starting `def ` and trailing `: pass`
+        return formatted[4:-5].strip()[:-1]
 
     def do_order_members(self, members: Sequence[Object | Alias], order: Order) -> Sequence[Object | Alias]:
         """Order members given an ordering method.
diff --git a/src/mkdocstrings/templates/python/material/_base/function.html b/src/mkdocstrings/templates/python/material/_base/function.html
@@ -23,15 +23,7 @@
           toc_label=function.name ~ "()") %}
 
         {% if config.separate_signature %}
-          <details>
-            <summary>Signature</summary>
-            {% filter highlight(language="python", inline=False) %}
-              {% filter format_code(config.line_length) %}        
-                {% if show_full_path %}{{ function.path }}{% else %}{{ function.name }}{% endif %}
-                {% include "signature.html" with context %}
-              {% endfilter %}
-            {% endfilter %}
-          </details>
+          {% if show_full_path %}{{ function.path }}{% else %}{{ function.name }}{% endif %}()
         {% else %}
           {% filter highlight(language="python", inline=True) %}
             {% if show_full_path %}{{ function.path }}{% else %}{{ function.name }}{% endif %}
@@ -45,6 +37,15 @@
 
       {% endfilter %}
 
+      {% if config.separate_signature %}
+        {% filter highlight(language="python", inline=False) %}
+          {% filter format_signature(config.line_length) %}        
+            {% if show_full_path %}{{ function.path }}{% else %}{{ function.name }}{% endif %}
+            {% include "signature.html" with context %}
+          {% endfilter %}
+        {% endfilter %}
+      {% endif %}
+
     {% else %}
       {% if config.show_root_toc_entry %}
         {% filter heading(heading_level,
