Refactor feedback visibility logic in analysis interface

app.py

@@ -574,7 +574,7 @@ def create_interface():
                     visible=False
                 )
                 flag_paper_btn = gr.Button("Submit Paper Feedback", visible=False)
-                analyze_btn = gr.Button("Analyze Connection Potential", variant="primary", interactive=False)
+                analyze_btn = gr.Button("Analyze Connection Potential", variant="primary", visible=False)
             with gr.Column(scale=1):
                 analysis_output = gr.Markdown(
                     value="# Synergy Analysis",
@@ -600,21 +600,13 @@ def create_interface():
         analysis_logger.setup([paper_details_output, analysis_output, analysis_feedback, analysis_comment], data_path + "flagged_analyses")
         paper_match_logger.setup([abstract_input, paper_details_output, paper_feedback, paper_comment], data_path + "flagged_paper_matches")
 
-        # Show analysis feedback components when analysis is displayed
-        def show_analysis_feedback():
-            return {
-                analysis_feedback: gr.update(visible=True),
-                analysis_comment: gr.update(visible=True),
-                flag_analysis_btn: gr.update(visible=True)
-            }
-
-        # Show paper feedback when paper is selected
-        def show_paper_feedback():
-            return {
-                paper_feedback: gr.update(visible=True),
-                paper_comment: gr.update(visible=True), 
-                flag_paper_btn: gr.update(visible=True)
-            }
+        # Show feedback component
+        def show_feedback():
+            return gr.update(visible=True), gr.update(visible=True), gr.update(visible=True)
+        
+        # Hide feedback component
+        def hide_feedback():
+            return gr.update(visible=False), gr.update(visible=False), gr.update(visible=False)
 
         # Display paper details when row is selected
         def on_select(evt: gr.SelectData, papers, query):
@@ -658,11 +650,13 @@ def create_interface():
             outputs=[paper_details_output, selected_paper_state],
             api_name=False,
         ).then(
-            lambda: gr.update(interactive=True),  # Enable analyze button when paper selected
+            lambda: gr.update(visible=True),  # Enable analyze button when paper selected
             outputs=[analyze_btn],
             api_name=False,
         ).then(
-            show_paper_feedback  # Add feedback UI when paper selected
+            show_feedback,
+            outputs=[paper_feedback, paper_comment, flag_paper_btn],
+            api_name=False,
         )
 
         # Connect analyze button to run analysis
@@ -672,8 +666,11 @@ def create_interface():
             outputs=[analysis_output],
             show_progress_on=[paper_details_output, analysis_output],
             api_name=False,
-        ).then(show_analysis_feedback)
-
+        ).then(
+            show_feedback,
+            outputs=[analysis_feedback, analysis_comment, flag_analysis_btn],
+            api_name=False,
+        )
         # Add flagging handlers
         flag_analysis_btn.click(
             lambda *args: analysis_logger.flag(list(args)),