Update ui.py

ui.py

@@ -145,7 +145,7 @@ def render_data_table(tableDF, speakerNames, raw_to_display, currFile):
 # ---------------------------------------------------------------------------
 
 def render_speaker_samples_tab(speakerNames, raw_to_display, currFile):
-    """Render a table: Speaker (with inline ✎ rename + scrollable dropdown history) | Audio Sample | ↺ button."""
+    """Render a table: Speaker (with inline ✎ rename + history dropdown) | Audio Sample | ↺ button."""
     file_samples = st.session_state.speakerClips.get(currFile, {})
     has_waveform = currFile in st.session_state.speakerWaveforms
     has_samples  = bool(file_samples)
@@ -165,6 +165,8 @@ def render_speaker_samples_tab(speakerNames, raw_to_display, currFile):
     if not has_samples:
         st.info("Audio samples are only available for files analyzed from audio (not RTTM/CSV/TXT).")
 
+    NEW_NAME_SENTINEL = "＋ Type a new name…"
+
     for sp in speakerNames:
         display_name = raw_to_display.get(sp, sp)
         edit_key     = (currFile, sp)
@@ -174,62 +176,45 @@ def render_speaker_samples_tab(speakerNames, raw_to_display, currFile):
 
         # --- Speaker cell ---
         if is_editing:
+            select_key  = f"inline_rename_select_{currFile}_{sp}"
             input_key   = f"inline_rename_input_{currFile}_{sp}"
             confirm_key = f"inline_rename_confirm_{currFile}_{sp}"
             cancel_key  = f"inline_rename_cancel_{currFile}_{sp}"
 
-            history     = st.session_state.inline_rename_history
-            current_val = display_name if display_name != sp else ""
+            history      = st.session_state.inline_rename_history
+            current_val  = display_name if display_name != sp else ""
 
-            with row_cols[0]:
-                # Render a native <datalist> above the Streamlit input.
-                # The browser connects them by matching the input's list= attribute,
-                # which we set on the nearest text input via scoped JS.
-                if history:
-                    dl_id      = f"dl_{sp}_{currFile}".replace(" ", "_").replace(".", "_")
-                    options    = "\n".join(f'<option value="{n}"></option>' for n in history)
-                    st.markdown(
-                        f"""
-                        <datalist id="{dl_id}">{options}</datalist>
-                        <style>
-                          /* Make the native dropdown scrollable at ~160px */
-                          #{dl_id} {{ max-height: 160px; overflow-y: auto; }}
-                        </style>
-                        <script>
-                          (function() {{
-                            // Wait one tick for Streamlit to render the input,
-                            // then attach the datalist to the very next text input
-                            // that appears after this script tag.
-                            var dl = document.getElementById('{dl_id}');
-                            var observer = new MutationObserver(function() {{
-                              var inputs = document.querySelectorAll('input[data-testid="stTextInput"]');
-                              if (inputs.length) {{
-                                inputs[inputs.length - 1].setAttribute('list', '{dl_id}');
-                                observer.disconnect();
-                              }}
-                            }});
-                            observer.observe(document.body, {{ childList: true, subtree: true }});
-                            // Also try immediately
-                            var inputs = document.querySelectorAll('input[data-testid="stTextInput"]');
-                            if (inputs.length) inputs[inputs.length - 1].setAttribute('list', '{dl_id}');
-                          }})();
-                        </script>
-                        """,
-                        unsafe_allow_html=True,
-                    )
+            # Dropdown options: history entries first, then sentinel for free text
+            options = history + [NEW_NAME_SENTINEL]
+
+            # Pre-select current name if it's already in history
+            default_idx = history.index(current_val) if current_val in history else len(history)
 
-                new_name = st.text_input(
-                    "Rename",
-                    value=current_val,
-                    key=input_key,
+            with row_cols[0]:
+                chosen = st.selectbox(
+                    "Pick or type",
+                    options=options,
+                    index=default_idx,
+                    key=select_key,
                     label_visibility="collapsed",
-                    placeholder=f"Rename {sp}…",
                 )
 
+                # If sentinel selected, show a free-text input
+                if chosen == NEW_NAME_SENTINEL:
+                    new_name = st.text_input(
+                        "New name",
+                        value=current_val,
+                        key=input_key,
+                        label_visibility="collapsed",
+                        placeholder=f"Rename {sp}…",
+                    )
+                else:
+                    new_name = chosen
+
                 btn_col1, btn_col2 = st.columns(2)
                 if btn_col1.button("✓", key=confirm_key, help="Confirm rename"):
                     confirmed = new_name.strip()
-                    if confirmed:
+                    if confirmed and confirmed != NEW_NAME_SENTINEL:
                         apply_inline_rename(currFile, sp, confirmed)
                         if confirmed not in st.session_state.inline_rename_history:
                             st.session_state.inline_rename_history.append(confirmed)