Updates

app.py

@@ -35,55 +35,51 @@ def reset_pdf_cache():
 def generate_pdf_report():
     pdf = FPDF()
     pdf.add_page()
-    pdf.set_font("Arial", style='B', size=12)
+    pdf.set_font("Arial", size=12)
     
-    # Header for PDF report (without emoji)
-    pdf.set_fill_color(255, 255, 255)  # White background
-    pdf.set_text_color(0, 0, 0)  # Black text
-    pdf.cell(0, 10, txt="Magic Math Quiz!", ln=True, align="C", fill=True)
-    pdf.ln(5)
-    pdf.set_font("Arial", size=8)
-    pdf.cell(0, 10, txt="By Ghassem Tofighi", ln=True, align="C", link="https://ghassem.com")
+    pdf.cell(200, 10, txt="Quiz Report", ln=True, align="C")
     pdf.ln(10)
 
-    for i, entry in enumerate(st.session_state.questions):
-        # Zebra background
-        if i % 2 == 0:
-            pdf.set_fill_color(245, 245, 245)  # Light gray
-        else:
-            pdf.set_fill_color(255, 255, 255)  # White
-
-        # Module Title
-        pdf.set_font("Arial", style='B', size=10)
-        pdf.multi_cell(0, 8, f"Module: {entry['module_title']}", border=1, fill=True)
-
-        # Question
-        pdf.set_font("Arial", style='B', size=10)
-        pdf.multi_cell(0, 8, f"Q{i+1}: {entry['question']}", border=1, fill=True)
-        
-        # Time Taken
-        pdf.set_font("Arial", size=10)
-        pdf.multi_cell(0, 8, f"Time Taken: {entry['time_taken']} seconds", border=1, fill=True)
-        
-        # Options
-        pdf.set_font("Arial", size=10)
-        options = ['a', 'b', 'c', 'd']
-        for j, option in enumerate(entry['options']):
-            if option == entry['correct_answer']:
-                pdf.set_text_color(0, 128, 0)  # Green for correct
-            elif option == entry['selected']:
-                pdf.set_text_color(255, 0, 0)  # Red for incorrect
-            else:
-                pdf.set_text_color(0, 0, 0)  # Default color for others
-
-            pdf.multi_cell(0, 6, f"{options[j]}. {option}", border=1, fill=True)  # Reduced vertical spacing
-        
-        pdf.set_text_color(0, 0, 0)  # Reset color
-        pdf.multi_cell(0, 6, f"Explanation: {entry['explanation']}", border=1, fill=True)  # Reduced vertical spacing
-        pdf.multi_cell(0, 6, f"Step-by-Step Solution: {', '.join(entry['step_by_step_solution'])}", border=1, fill=True)  # Reduced vertical spacing
+    for module, data in st.session_state.module_question_count.items():
+        correct_count = st.session_state.module_correct_count.get(module, 0)
+        total_count = data
+        pdf.cell(200, 10, txt=f"Module: {module}", ln=True, align="L")
+        pdf.ln(5)
+        pdf.cell(200, 10, txt=f"Correct Answers: {correct_count}/{total_count}", ln=True, align="L")
         pdf.ln(5)
 
-    return pdf.output(dest='S').encode('latin-1', 'replace')
+        for entry in st.session_state.questions:
+            if entry['module'] == module:
+                question, options, selected, correct, explanation, step_by_step_solution = (
+                    entry['question'], 
+                    entry['options'], 
+                    entry['selected'] if entry['selected'] is not None else "Not Answered",
+                    entry['correct_answer'], 
+                    entry['explanation'],
+                    entry['step_by_step_solution']
+                )
+                pdf.multi_cell(0, 10, f"Q: {question}")
+                for option in options:
+                    if option == correct:
+                        pdf.set_text_color(0, 128, 0)  # Green for correct
+                        pdf.multi_cell(0, 10, f"{option}")
+                    elif option == selected:
+                        pdf.set_text_color(255, 0, 0)  # Red for incorrect
+                        pdf.multi_cell(0, 10, f"{option}")
+                    else:
+                        pdf.set_text_color(0, 0, 0)  # Default color for others
+                        pdf.multi_cell(0, 10, f"   {option}")
+                pdf.set_text_color(0, 0, 0)  # Reset color
+                pdf.multi_cell(0, 10, f"Explanation: {explanation}")
+                pdf.ln(5)
+                pdf.multi_cell(0, 10, "Step-by-Step Solution:")
+                for step in step_by_step_solution:
+                    pdf.multi_cell(0, 10, step)
+                pdf.ln(10)
+
+        pdf.ln(10)  # Add space after each module
+
+    return pdf.output(dest='S').encode('latin1', 'replace')
 
 def load_modules():
     modules = {}
@@ -157,6 +153,7 @@ if selected_category:
         f"""
         <div style="background-color: #333; padding: 10px; border-radius: 5px; margin-top: 20px;">
         <span style='font-size: 18px; color: white;'>{selected_category} > {selected_module_data['title']}</span>
+        </div>
         """, 
         unsafe_allow_html=True
     )
@@ -165,7 +162,6 @@ if selected_category:
         st.markdown(
             f"""
             <a href='#' download='quiz_report.pdf' style='font-size: 18px; margin-left: 10px; color: white;' onclick='document.getElementById("pdf-link").click();'>📄 Download PDF Report</a>
-            </div>
             """, 
             unsafe_allow_html=True
         )