Skip to content

Commit ce6e3a6

Browse files
committed
add a create_effect to make sure external changes to the sorting signal are reflected in the table rendering.
closes #40
1 parent ee73925 commit ce6e3a6

File tree

1 file changed

+9
-7
lines changed

1 file changed

+9
-7
lines changed

src/components/table_content.rs

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -232,6 +232,7 @@ where
232232
let load_row_count = load_row_count.clone();
233233

234234
move |clear_row_count: bool| {
235+
logging::log!("clear");
235236
selection.clear();
236237
first_selected_index.set(None);
237238

@@ -247,22 +248,23 @@ where
247248
}
248249
}
249250

250-
set_reload_count.set(reload_count.get().overflowing_add(1).0);
251+
set_reload_count.set(reload_count.get_untracked().overflowing_add(1).0);
251252
}
252253
};
253254

254-
let on_head_click = {
255+
let on_head_click = move |event: TableHeadEvent| {
256+
sorting.update(move |sorting| sorting_mode.update_sorting_from_event(sorting, event));
257+
};
258+
259+
create_effect({
255260
let rows = Rc::clone(&rows);
256261
let clear = clear.clone();
257262

258-
move |event: TableHeadEvent| {
259-
sorting.update(move |sorting| sorting_mode.update_sorting_from_event(sorting, event));
260-
263+
move |_| {
261264
rows.borrow_mut().set_sorting(&sorting.get());
262-
263265
clear(false);
264266
}
265-
};
267+
});
266268

267269
create_effect({
268270
let rows = Rc::clone(&rows);

0 commit comments

Comments
 (0)