fix: sort to ensure buttons do not swap places

app.py

@@ -171,6 +171,7 @@ def display_download_buttons():
         .collect()["representations_document"]
         .to_list()
     )
+    representations_documents.sort()
 
     # remove some old intermediate files
     _ = [file.unlink() for file in (Paths.STAGING / "pdfs_azure").glob("*.pdf")]
@@ -190,46 +191,38 @@ def display_download_buttons():
 
     # Add some spacing and better organization
     st.markdown("---")
-
-    for i, rep in enumerate(representations_documents):
-        report_path = Paths.SUMMARY / f"Summary_Documents-{rep}.pdf"
-        summaries_path = Paths.SUMMARY / f"Summary_of_Submitted_Responses-{rep}.pdf"
-
-        # Create a container for each report set
-        with st.container():
-            st.subheader(f"Report Set {i+1}: {rep}")
-
-            # Create columns with better spacing
-            col1, col2 = st.columns(2, gap="large")
-
-            with col1:
-                st.markdown("**Executive Report**")
+    # Create a container for the Executive Reports
+    with st.expander("**Executive Reports**"):
+        for i, rep in enumerate(representations_documents):
+            summaries_path = Paths.SUMMARY / f"Summary_of_Submitted_Responses-{rep}.pdf"
+            with st.container():
+                st.subheader(f"Executive Report for {rep}")
                 with open(summaries_path, "rb") as pdf_file:
                     st.download_button(
-                        label="Download",
+                        label="Download Executive Report",
                         data=pdf_file,
                         file_name=f"Summary_of_Submitted_Responses-{rep}.pdf",
                         mime="application/pdf",
-                        type="primary",
                         use_container_width=True,
-                        key=f"exec_{rep}",
+                        key=f"exec_{i}_{hash(rep)}",  # Ensure key uniqueness with index + hash
                     )
+                st.markdown("---")
 
-            with col2:
-                st.markdown("**Representation Summary**")
+    # Create a container for the Representation Summaries
+    with st.expander("**Representation Summaries**"):
+        for i, rep in enumerate(representations_documents):
+            report_path = Paths.SUMMARY / f"Summary_Documents-{rep}.pdf"
+            with st.container():
+                st.subheader(f"Representation Summary for {rep}")
                 with open(report_path, "rb") as pdf_file:
                     st.download_button(
-                        label="Download",
+                        label="Download Representation Summary",
                         data=pdf_file,
                         file_name=f"Summary_Documents-{rep}.pdf",
                         mime="application/pdf",
-                        type="primary",
                         use_container_width=True,
-                        key=f"rep_{rep}",
+                        key=f"rep_{i}_{hash(rep)}",  # Ensure key uniqueness with index + hash
                     )
-
-            # Add a separator between report sets
-            if i < len(representations_documents) - 1:
                 st.markdown("---")