remove fn passed to child generate_remove_row_id_for_row since not required

Author: ltfschoen

examples/invoice/src/components/Cell.rs

@@ -13,14 +13,8 @@ use crate::js;
 use crate::state::{Editing, State, Text};
 
 #[component]
-pub fn Cell(
-    col: usize,
-    row: usize,
-    state: &Hook<State>,
-    generate_remove_row_id_for_row: impl Fn(usize) -> String,
-) -> impl View + '_ {
+pub fn Cell(col: usize, row: usize, state: &Hook<State>) -> impl View + '_ {
     // debug!("Cell get_text source {:?} {:?}", col, row);
-    let id_to_destroy = generate_remove_row_id_for_row(row);
     let value: &str;
     if col <= (state.details.table.columns.len() - 1) {
         value = state
@@ -41,12 +35,6 @@ pub fn Cell(
             Ok(r) => r,
             Err(e) => return,
         };
-        // TODO - we're not using the `row_id` to remove rows,
-        // so we can remove `generate_remove_row_id_for_row`
-        let row_id = match event.target().get_attribute("id") {
-            Some(r) => r,
-            None => return,
-        };
 
         state.remove_row_main(row_usize);
     });
@@ -103,7 +91,6 @@ pub fn Cell(
                 <td.destroy-container>
                     <button.destroy
                         data={row}
-                        id={id_to_destroy}
                         onclick={onremove_row}
                     />
                 </td>
@@ -139,7 +126,6 @@ pub fn Cell(
                 <td.destroy-container>
                     <button.destroy
                         data={row}
-                        id={id_to_destroy}
                         onclick={onremove_row}
                     />
                 </td>
@@ -162,7 +148,6 @@ pub fn Cell(
                 <td.destroy-container>
                     <button.destroy
                         data={row}
-                        id={id_to_destroy}
                         onclick={onremove_row}
                     />
                 </td>

examples/invoice/src/components/Editor.rs

@@ -192,10 +192,6 @@ pub fn Editor() -> impl View {
         let process_row_value_for_label_for_table =
             |label: &str| -> String { get_row_value_for_label_for_table(&label, &state) };
 
-        let generate_remove_row_id_for_row = |row: usize| -> String {
-            return "destroy-main-".to_string() + &i32::try_from(row).unwrap().to_string();
-        };
-
         view! {
             <div .invoice-wrapper>
                 <section .invoiceapp>
@@ -298,12 +294,10 @@ pub fn Editor() -> impl View {
                             <tbody>
                             {
                                 for state.main.table.rows().map(move |row| view! {
-                                    <tr
-                                        id={generate_remove_row_id_for_row(row)}
-                                    >
+                                    <tr>
                                     {
                                         for state.main.table.columns().map(move |col| view! {
-                                            <Cell {col} {row} {state} {generate_remove_row_id_for_row} />
+                                            <Cell {col} {row} {state} />
                                         })
                                     }
                                     </tr>