From da6bf2d046b19137ad8ae5184d97be22b3c8bcef Mon Sep 17 00:00:00 2001
From: Daniel McCloy <dan@mccloy.info>
Date: Wed, 2 Oct 2024 10:41:55 -0500
Subject: [PATCH 1/4] refactor channels HTML repr

---
 mne/html_templates/repr/_channels.html.jinja   | 18 +-----------------
 .../repr/static/_channels.html.jinja           | 17 +++++++++++++++++
 2 files changed, 18 insertions(+), 17 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/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>

From 6c3dea4e76556ded474acb68c1fbf5ae0ddcbba5 Mon Sep 17 00:00:00 2001
From: Daniel McCloy <dan@mccloy.info>
Date: Wed, 2 Oct 2024 10:43:16 -0500
Subject: [PATCH 2/4] use channels template in fwd

---
 mne/html_templates/repr/forward.html.jinja | 22 ++--------------------
 1 file changed, 2 insertions(+), 20 deletions(-)

diff --git a/mne/html_templates/repr/forward.html.jinja b/mne/html_templates/repr/forward.html.jinja
index 510a775c2b6..e22dd6e0643 100644
--- a/mne/html_templates/repr/forward.html.jinja
+++ b/mne/html_templates/repr/forward.html.jinja
@@ -1,27 +1,9 @@
 {%include '_js_and_css.html.jinja' %}
 
 <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>

From 95afeb195f161266842527a3bc655b1b1a2eabfa Mon Sep 17 00:00:00 2001
From: Daniel McCloy <dan@mccloy.info>
Date: Wed, 2 Oct 2024 10:43:44 -0500
Subject: [PATCH 3/4] make fwd collapsible

---
 mne/html_templates/repr/forward.html.jinja | 20 ++++++++++++++++----
 1 file changed, 16 insertions(+), 4 deletions(-)

diff --git a/mne/html_templates/repr/forward.html.jinja b/mne/html_templates/repr/forward.html.jinja
index e22dd6e0643..e8bce9649b1 100644
--- a/mne/html_templates/repr/forward.html.jinja
+++ b/mne/html_templates/repr/forward.html.jinja
@@ -1,17 +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() %}
         {% 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

From 3c76cd64cb286f373f5adfcfea6282d17c6643fa Mon Sep 17 00:00:00 2001
From: Daniel McCloy <dan@mccloy.info>
Date: Wed, 2 Oct 2024 10:43:56 -0500
Subject: [PATCH 4/4] demo it in docs

---
 tutorials/forward/35_eeg_no_mri.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

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!