Circuit-Forge/peregrine
2
0
Fork 0
Code Issues 22 Pull requests 2 Projects Releases 6 Packages Wiki Activity Actions

fix: remove st.rerun() from dialog nav buttons — caused dialog to close on Next/Back

Browse source
This commit is contained in:
pyr0ball 2026-03-03 13:28:26 -08:00
parent 7ea166ab26
commit 1cd4b361de
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
app/feedback.py
View file

@ -72,7 +72,7 @@ def _feedback_dialog(page: str) -> None:
with col_cancel:
if st.button("Cancel"):
_clear_feedback_state()
st.rerun()
st.rerun() # intentionally closes the dialog
with col_next:
if st.button(
"Next →",
@ -81,7 +81,7 @@ def _feedback_dialog(page: str) -> None:
or not st.session_state.get("fb_desc", "").strip(),
):
st.session_state.fb_step = 2
st.rerun()
# no st.rerun() — button click already re-renders the dialog
# ═════════════════════════════════════════════════════════════════════════
# STEP 2 — Consent + attachments
@ -136,7 +136,7 @@ def _feedback_dialog(page: str) -> None:
)
if st.button("🗑 Remove screenshot"):
st.session_state.pop("fb_screenshot", None)
st.rerun()
# no st.rerun() — button click already re-renders the dialog
# ── Attribution consent ───────────────────────────────────────────────
st.divider()
@ -159,7 +159,7 @@ def _feedback_dialog(page: str) -> None:
with col_back:
if st.button("← Back"):
st.session_state.fb_step = 1
st.rerun()
# no st.rerun() — button click already re-renders the dialog
with col_submit:
if st.button("Submit Feedback", type="primary"):