Update app.py

app.py

@@ -430,12 +430,12 @@ async def process_text_with_progress(
 ):
     # Input validation
     if not text or text.strip() == "":
-        # For error case, return empty strings for markdown links and None for audio,
-        # along with visibility updates for error_output
+        # When an error occurs or input is invalid, return None for audio and empty strings
+        # for markdown, along with updates to error_output.
         return (
-            None,  # audio_output
-            gr.update(value="", visible=False),  # srt_download_link
-            gr.update(value="", visible=False),  # audio_download_link
+            None, # audio_output
+            gr.update(value="", visible=False), # srt_download_link
+            gr.update(value="", visible=False), # audio_download_link
             gr.update(value="Please enter some text to convert to speech.", visible=True) # error_output
         )
     
@@ -462,6 +462,8 @@ async def process_text_with_progress(
         )
         
         # Generate Markdown links for download that open in a new tab
+        # For gr.Markdown, return the HTML string directly for the 'value'
+        # and use gr.update for visibility.
         srt_download_link_html = f'<a href="file={srt_path}" download="subtitles.srt" target="_blank">Download SRT</a>'
         audio_download_link_html = f'<a href="file={audio_path}" download="audio.mp3" target="_blank">Download Audio</a>'
 
@@ -596,8 +598,8 @@ with gr.Blocks(title="Advanced TTS with Configurable SRT Generation") as app:
     
     # Add error message component - it should be visible initially
     # or its visibility handled in the return of the function
-    error_output = gr.Textbox(label="Status", visible=True) # Changed to visible=True for easier debugging.
-                                                            # You can set it to False if you want it hidden by default.
+    # It's better to keep it hidden initially and show only on error
+    error_output = gr.Textbox(label="Status", visible=False) 
     
     with gr.Row():
         with gr.Column():
@@ -605,8 +607,18 @@ with gr.Blocks(title="Advanced TTS with Configurable SRT Generation") as app:
         with gr.Column():
             # Change gr.File to gr.Markdown for download links
             # Set initial values to empty string and visible=False to hide them on load
-            srt_download_link = gr.Markdown(value="", visible=False, label="Download SRT")
-            audio_download_link = gr.Markdown(value="", visible=False, label="Download Audio")
+            # The label argument for gr.Markdown is deprecated in newer Gradio versions
+            # and might contribute to unexpected behavior or warnings.
+            # It's better to set the label directly in the value if needed, or use a separate gr.Textbox for label.
+            srt_download_link = gr.Markdown(value="", visible=False) # Removed label here
+            audio_download_link = gr.Markdown(value="", visible=False) # Removed label here
+            
+            # Add a separate Textbox or Markdown for "Download SRT" and "Download Audio" labels
+            # if you need persistent labels for the download section.
+            gr.Markdown("**Download SRT:**")
+            srt_download_placeholder = gr.Markdown(value="", visible=False) # Placeholder for the actual link
+            gr.Markdown("**Download Audio:**")
+            audio_download_placeholder = gr.Markdown(value="", visible=False) # Placeholder for the actual link
     
     # Handle button click with manual error handling instead of .catch()
     submit_btn.click(
@@ -622,9 +634,9 @@ with gr.Blocks(title="Advanced TTS with Configurable SRT Generation") as app:
         ],
         outputs=[
             audio_output,
-            srt_download_link,
-            audio_download_link,
-            error_output # Only one error_output here, as it's a single component
+            srt_download_placeholder, # Map to the new placeholder
+            audio_download_placeholder, # Map to the new placeholder
+            error_output
         ],
         api_name="generate"
     )