Update ui/insights_ui_generator.py

ui/insights_ui_generator.py

@@ -8,40 +8,59 @@ logger = logging.getLogger(__name__)
 
 def format_report_for_display(report_data: Optional[pd.Series]) -> str:
     """
-    Generates a complete Markdown string for a single report, including a dynamic title
-    based on the report's type and creation date.
-
-    Args:
-        report_data: A pandas Series representing a single row from the agentic analysis DataFrame.
-                     It should contain 'comprehensive_analysis_text', 'report_type', and 'Created Date'.
-
-    Returns:
-        A Markdown formatted string for the report.
+    Enhanced report formatting with Medium-style typography and layout.
+    This function replaces the original format_report_for_display.
     """
     if report_data is None or report_data.empty:
-        return "## Report Details\n\n*Please select a report from the library to view its contents.*"
-
-    # Assuming the main report text is in a column named 'comprehensive_analysis_text'
-    # You may need to adjust this column name based on your Bubble.io data structure.
+        return """
+        <div class="empty-state">
+            <div class="empty-state-icon">📄</div>
+            <div class="empty-state-title">Report Not Available</div>
+            <div class="empty-state-description">
+                The selected report could not be loaded. Please try selecting a different report 
+                or refresh the page.
+            </div>
+        </div>
+        """
+
+    # Extract report data
     report_text = report_data.get('report_text', '*Report content not found.*')
     report_type = report_data.get('report_type')
-    created_date_str = report_data.get('Created Date') # Bubble's default field name
-
-    title = "Comprehensive Analysis Report"  # Default title
+    created_date_str = report_data.get('Created Date')
 
+    # Generate dynamic title and subtitle
+    title = "Comprehensive Analysis Report"
+    subtitle = ""
+    
     try:
-        if report_type == 'Quarte':  # As per your request for "Quarter" type
-            title = "Quarterly Insights Report"
+        if report_type == 'Quarter':
+            title = "📈 Quarterly Insights Report"
+            subtitle = "Strategic analysis of your quarterly performance"
         elif report_type == 'Week' and pd.notna(created_date_str):
-            # Bubble dates are typically in ISO format, e.g., '2024-06-11T14:30:00.000Z'
+            # Ensure pandas is used for datetime conversion
             created_date = pd.to_datetime(created_date_str)
-            day_name = created_date.strftime('%A')  # e.g., 'Tuesday'
-            title = f"{day_name}'s Weekly Update Report"
+            day_name = created_date.strftime('%A')
+            title = f"📊 {day_name}'s Weekly Update"
+            subtitle = f"Weekly performance analysis for {created_date.strftime('%B %d, %Y')}"
     except Exception as e:
         logger.error(f"Error generating dynamic report title: {e}")
-        # In case of an error, the default title will be used.
-
-    return f"## {title}\n\n{report_text.strip()}"
+        # In case of an error, default title and no subtitle will be used.
+
+    # Format the report content with the new HTML structure
+    formatted_content = f"""
+    <div class="report-header">
+        <h1>{title}</h1>
+        {f'<p style="font-size: 1.1rem; color: #6b6b6b; margin-bottom: 0;">{subtitle}</p>' if subtitle else ''}
+    </div>
+    
+    <div class="report-body">
+        {report_text.strip()}
+    </div>
+    """
+    
+    # Wrap the entire content in a div with the 'markdown' class, which Gradio Markdown component expects
+    # and which is styled by the provided CSS.
+    return formatted_content
 
 
 def format_report_to_markdown(report_string: Optional[str]) -> str:
@@ -76,8 +95,8 @@ def extract_key_results_for_selection(
 
     Args:
         actionable_okrs_and_tasks_dict: The dictionary representation of TaskExtractionOutput,
-                                      typically `orchestration_results["actionable_okrs_and_tasks"]`.
-                                      Expected structure: {'okrs': List[OKR_dict], ...}
+                                        typically `orchestration_results["actionable_okrs_and_tasks"]`.
+                                        Expected structure: {'okrs': List[OKR_dict], ...}
 
     Returns:
         A list of dictionaries, where each dictionary represents a Key Result:
@@ -237,3 +256,4 @@ def format_single_okr_for_display(
         md_parts.append("\n*No Key Results matching the 'accepted' filter for this objective.*")
 
     return "\n".join(md_parts)
+