Skip to content

Commit 9bf819a

Browse files
committed
fix(modals): remove redundant closure wrapping modal close handler
Inline the signal write directly in each onclick to eliminate the intermediate `close` closure, resolving a clippy redundant_closure lint.
1 parent f66fd7d commit 9bf819a

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

nodedb-studio/src/modals/new_connection.rs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ use crate::state::ui::ModalKind;
1111
#[component]
1212
pub fn NewConnectionForm() -> Element {
1313
let mut modal = use_context::<Signal<Option<ModalKind>>>();
14-
let mut close = move |_| modal.set(None);
1514
rsx! {
1615
div { class: "modal-body",
1716
div { class: "form-field",
@@ -36,10 +35,10 @@ pub fn NewConnectionForm() -> Element {
3635
}
3736
}
3837
div { class: "modal-footer",
39-
button { class: "btn ghost", onclick: move |e| close(e), "Cancel" }
38+
button { class: "btn ghost", onclick: move |_| modal.set(None), "Cancel" }
4039
button { class: "btn", "Test" }
4140
button { class: "btn", "Save" }
42-
button { class: "btn primary", onclick: move |e| close(e), "Save & connect" }
41+
button { class: "btn primary", onclick: move |_| modal.set(None), "Save & connect" }
4342
}
4443
}
4544
}

0 commit comments

Comments
 (0)