Update app.py

app.py

@@ -1,7 +1,6 @@
 import streamlit as st
 import importlib
 from io import BytesIO
-import time
 
 st.set_page_config(page_title="Multilingual Translator", page_icon="🌐", layout="centered")
 
@@ -15,60 +14,43 @@ def main():
 
     # Header
     st.markdown("<h1 style='text-align: center; color: #4285F4;'>Multilingual Translator</h1>", unsafe_allow_html=True)
-    st.markdown("<p style='text-align: center; color: #666;'>Translate text like Google Translate</p>", unsafe_allow_html=True)
 
-    # Language Controls
+    # Language and Input/Output Layout
     col1, col2 = st.columns(2)
     with col1:
         detected_options = lang_detect.detect_language(st.session_state.get("input_text", "")) if st.session_state.get("input_text", "").strip() and len(st.session_state.get("input_text", "").strip()) >= 10 else [("English", 1.0, "English")]
         source_lang = detected_options[0][0] if detected_options else "English"
         source_lang_code = next((k for k, v in LANGUAGES.items() if v[1] == source_lang), "en")
         source_lang_display = st.selectbox("Source", options=[LANGUAGES[source_lang_code][0]] + [v[0] for v in LANGUAGES.values()], index=0, key="source_lang")
-    with col2:
-        target_lang_display = st.selectbox("Target", options=[v[0] for v in LANGUAGES.values()], index=list(LANGUAGES.values()).index(LANGUAGES["hi"]), key="target_lang")
-        target_lang = next((k for k, v in LANGUAGES.items() if v[0] == target_lang_display), "hi")
-
-    # Input Section
-    if "input_text" not in st.session_state:
-        st.session_state.input_text = ""
-    with st.container():
-        input_type = st.radio("Input", ["Text", "File"], horizontal=True)
+        input_type = st.radio("Input", ["Text", "File"], horizontal=True, key="input_type")
         if input_type == "Text":
-            input_text = st.text_area("Text to Translate", height=150, key="input_text", on_change=trigger_translation, args=(translation, lang_detect, audio_processor,))
+            input_text = st.text_area("", height=200, key="input_text", on_change=trigger_translation, args=(translation, lang_detect, audio_processor,))
         else:
-            input_text = st.file_uploader("Upload File", type=["txt", "docx", "pdf"], key="file_input", on_change=trigger_translation, args=(translation, lang_detect, audio_processor,))
+            input_text = st.file_uploader("", type=["txt", "docx", "pdf"], key="file_input", on_change=trigger_translation, args=(translation, lang_detect, audio_processor,))
             if input_text:
                 st.session_state.input_text = input_text.read().decode("utf-8").strip()
-        st.markdown(f"<small style='color: grey;'>Chars: {len(st.session_state.input_text)}/1000</small>", unsafe_allow_html=True)
         st.button("Translate", key="translate_btn", on_click=trigger_translation, args=(translation, lang_detect, audio_processor,))
-
-    # Output Section
-    with st.container():
-        if "translated_text" in st.session_state and st.session_state.translated_text:
-            st.text_area("Translation", value=st.session_state.translated_text, height=150, key="output_text", disabled=True)
-            if st.button("🔊", key="audio_btn", on_click=play_audio, args=(audio_processor,)):
-                pass  # Handled by play_audio
+    with col2:
+        target_lang_display = st.selectbox("Target", options=[v[0] for v in LANGUAGES.values()], index=list(LANGUAGES.values()).index(LANGUAGES["hi"]), key="target_lang")
+        target_lang = next((k for k, v in LANGUAGES.items() if v[0] == target_lang_display), "hi")
+        if "translated_text" in st.session_state:
+            st.text_area("", value=st.session_state.translated_text, height=200, key="output_text", disabled=True)
+        if st.button("", key="audio_btn", on_click=play_audio, args=(audio_processor,), help="Play audio"):
+            pass  # Handled by play_audio
 
 def trigger_translation(translation, lang_detect, audio_processor):
     text = st.session_state.input_text.strip()
     if text:
-        with st.spinner("Translating..."):
-            try:
-                source_lang = next((k for k, v in LANGUAGES.items() if v[0] == st.session_state.source_lang), "en")
-                target_lang = next((k for k, v in LANGUAGES.items() if v[0] == st.session_state.target_lang), "hi")
-                if source_lang != "en" and target_lang != "en":
-                    temp = translation.translate(text, source_lang, "en")
-                    if not temp or len(temp.split()) < 2:
-                        raise ValueError("Intermediate translation failed")
-                    translated = translation.translate(temp, "en", target_lang)
-                else:
-                    translated = translation.translate(text, source_lang, target_lang)
-                if not translated or len(translated.split()) < 2:
-                    raise ValueError("Translation failed")
-                st.session_state.translated_text = translated
-            except Exception as e:
+        source_lang = next((k for k, v in LANGUAGES.items() if v[0] == st.session_state.source_lang), "en")
+        target_lang = next((k for k, v in LANGUAGES.items() if v[0] == st.session_state.target_lang), "hi")
+        if source_lang != "en" and target_lang != "en":
+            temp = translation.translate(text, source_lang, "en")
+            if temp and len(temp.split()) >= 2:
+                st.session_state.translated_text = translation.translate(temp, "en", target_lang)
+            else:
                 st.session_state.translated_text = text
-                st.warning(f"Error: {str(e)}. Using input as fallback.")
+        else:
+            st.session_state.translated_text = translation.translate(text, source_lang, target_lang) or text
 
 def play_audio(audio_processor):
     if "translated_text" in st.session_state and st.session_state.translated_text:
@@ -76,14 +58,10 @@ def play_audio(audio_processor):
         audio = audio_processor.text_to_speech(st.session_state.translated_text, target_lang)
         if audio and audio.getbuffer().nbytes > 0:
             st.audio(audio, format="audio/mp3")
-            st.success("Playing audio.")
         else:
             audio_fallback = audio_processor.text_to_speech(st.session_state.translated_text, "en")
             if audio_fallback and audio_fallback.getbuffer().nbytes > 0:
                 st.audio(audio_fallback, format="audio/mp3")
-                st.success("Playing fallback audio in English.")
-            else:
-                st.error("Audio failed. Try again.")
 
 if __name__ == "__main__":
     main()