From ed933b8d9d523a9f4f7006d6e6d1c81901e5478a Mon Sep 17 00:00:00 2001
From: Daniel McCloy <dan@mccloy.info>
Date: Wed, 2 Oct 2024 12:12:20 -0500
Subject: [PATCH] make HTML repr for Forward match others (#12883)

---
 mne/html_templates/repr/_channels.html.jinja  | 18 +-------
 mne/html_templates/repr/forward.html.jinja    | 42 ++++++++-----------
 .../repr/static/_channels.html.jinja          | 17 ++++++++
 tutorials/forward/35_eeg_no_mri.py            |  2 +-
 4 files changed, 37 insertions(+), 42 deletions(-)
 create mode 100644 mne/html_templates/repr/static/_channels.html.jinja

diff --git a/mne/html_templates/repr/_channels.html.jinja b/mne/html_templates/repr/_channels.html.jinja
index 4f7646e9c80..1d057b9688f 100644
--- a/mne/html_templates/repr/_channels.html.jinja
+++ b/mne/html_templates/repr/_channels.html.jinja
@@ -11,23 +11,7 @@
 {%include 'static/_section_header_row.html.jinja' %}
 
 {% for channel_type, channels in (info | format_channels).items() %}
-{% set channel_names_good = channels["good"] | map(attribute='name_html') | join(', ') %}
-<tr class="repr-element {{ section_class_name }} {{ collapsed_row_class }}">
-    <td class="mne-repr-section-toggle"></td>
-    <td>{{ channel_type }}</td>
-    <td>
-        <button class="mne-ch-names-btn sd-sphinx-override sd-btn sd-btn-info sd-text-wrap sd-shadow-sm" onclick="alert('Good {{ channel_type}}:\n\n{{ channel_names_good | safe }}')" title="(Click to open in popup)&#13;&#13;{{ channel_names_good | safe }}">
-            {{ channels["good"] | length}}
-        </button>
-
-        {% if channels["bad"] %}
-        {% set channel_names_bad = channels["bad"] | map(attribute='name_html') | join(', ') %}
-        and <button class="mne-ch-names-btn sd-sphinx-override sd-btn sd-btn-info sd-text-wrap sd-shadow-sm" onclick="alert('Bad {{ channel_type}}:\n\n{{ channel_names_bad | safe }}')" title="(Click to open in popup)&#13;&#13;{{ channel_names_bad | safe }}">
-            {{ channels["bad"] | length}} bad
-        </button>
-        {% endif %}
-    </td>
-</tr>
+    {% include 'static/_channels.html.jinja' %}
 {% endfor %}
 
 <tr class="repr-element {{ section_class_name }} {{ collapsed_row_class }}">
diff --git a/mne/html_templates/repr/forward.html.jinja b/mne/html_templates/repr/forward.html.jinja
index 510a775c2b6..e8bce9649b1 100644
--- a/mne/html_templates/repr/forward.html.jinja
+++ b/mne/html_templates/repr/forward.html.jinja
@@ -1,35 +1,29 @@
 {%include '_js_and_css.html.jinja' %}
 
+{% set section = "Forward" %}
+{% set section_class_name = section | lower | append_uuid %}
+
+{# Collapse content during documentation build. #}
+{% if collapsed %}
+{% set collapsed_row_class = "mne-repr-collapsed" %}
+{% else %}
+{% set collapsed_row_class = "" %}
+{% endif %}
+
 <table class="table mne-repr-table">
+    {%include 'static/_section_header_row.html.jinja' %}
     {% for channel_type, channels in (info | format_channels).items() %}
-    {% set channel_names_good = channels["good"] | map(attribute='name_html') | join(', ') %}
-    <tr class="repr-element {{ section_class_name }} {{ collapsed_row_class }}">
-        <th>{{ channel_type }}</th>
-        <td>
-            <button class="mne-ch-names-btn sd-sphinx-override sd-button sd-btn-info sd-text-wrap sd-shadow-sm"
-                onclick="alert('Good {{ channel_type}}:\n\n{{ channel_names_good | safe }}')"
-                title="Show good channel names">
-                {{ channels["good"] | length}}
-            </button>
-
-            {% if channels["bad"] %}
-            {% set channel_names_bad = channels["bad"] | map(attribute='name_html') | join(', ') %}
-            and <button class="mne-ch-names-btn sd-sphinx-override sd-button sd-btn-info sd-text-wrap sd-shadow-sm"
-                onclick="alert('Bad {{ channel_type}}:\n\n{{ channel_names_bad | safe }}')"
-                title="Show bad channel names">
-                {{ channels["bad"] | length}} bad
-            </button>
-            {% endif %}
-        </td>
-    </tr>
+        {% include 'static/_channels.html.jinja' %}
     {% endfor %}
 
-    <tr>
-        <th>Source space</th>
+    <tr class="repr-element {{ section_class_name }} {{ collapsed_row_class }}">
+        <td class="mne-repr-section-toggle"></td>
+        <td>Source space</td>
         <td>{{ source_space_descr }}</td>
     </tr>
-    <tr>
-        <th>Source orientation</th>
+    <tr class="repr-element {{ section_class_name }} {{ collapsed_row_class }}">
+        <td class="mne-repr-section-toggle"></td>
+        <td>Source orientation</td>
         <td>{{ source_orientation }}</td>
     </tr>
 </table>
\ No newline at end of file
diff --git a/mne/html_templates/repr/static/_channels.html.jinja b/mne/html_templates/repr/static/_channels.html.jinja
new file mode 100644
index 00000000000..d40f62f5d6b
--- /dev/null
+++ b/mne/html_templates/repr/static/_channels.html.jinja
@@ -0,0 +1,17 @@
+{% set channel_names_good = channels["good"] | map(attribute='name_html') | join(', ') %}
+<tr class="repr-element {{ section_class_name }} {{ collapsed_row_class }}">
+    <td class="mne-repr-section-toggle"></td>
+    <td>{{ channel_type }}</td>
+    <td>
+        <button class="mne-ch-names-btn sd-sphinx-override sd-btn sd-btn-info sd-text-wrap sd-shadow-sm" onclick="alert('Good {{ channel_type}}:\n\n{{ channel_names_good | safe }}')" title="(Click to open in popup)&#13;&#13;{{ channel_names_good | safe }}">
+            {{ channels["good"] | length}}
+        </button>
+
+        {% if channels["bad"] %}
+        {% set channel_names_bad = channels["bad"] | map(attribute='name_html') | join(', ') %}
+        and <button class="mne-ch-names-btn sd-sphinx-override sd-btn sd-btn-info sd-text-wrap sd-shadow-sm" onclick="alert('Bad {{ channel_type}}:\n\n{{ channel_names_bad | safe }}')" title="(Click to open in popup)&#13;&#13;{{ channel_names_bad | safe }}">
+            {{ channels["bad"] | length}} bad
+        </button>
+        {% endif %}
+    </td>
+</tr>
diff --git a/tutorials/forward/35_eeg_no_mri.py b/tutorials/forward/35_eeg_no_mri.py
index a2deaa069b6..0fca916cabf 100644
--- a/tutorials/forward/35_eeg_no_mri.py
+++ b/tutorials/forward/35_eeg_no_mri.py
@@ -82,7 +82,7 @@
 fwd = mne.make_forward_solution(
     raw.info, trans=trans, src=src, bem=bem, eeg=True, mindist=5.0, n_jobs=None
 )
-print(fwd)
+fwd
 
 ##############################################################################
 # From here on, standard inverse imaging methods can be used!