fixed issues with BulkEdtior (dialog not closing and other dialog sho… (

#297)

* fixed issues with BulkEdtior (dialog not closing and other dialog showing wrong button)

- closes #296

* formatting

README.md

@@ -233,6 +233,7 @@ E.g., if it was used in a menu and the menu is red, the circle would be red.
 ### **WORK IN PROGRESS**
 * (foxriver76) fixed last change y-offset for some widgets
 * (foxriver76) fixed issue where JquiState did not respect data type
+* (foxriver76) fixed issues with BulkEdtior (dialog not closing and other dialog showing wrong button)
 
 ### 2.9.13 (2024-01-08)
 * (foxriver76) correctly detect IDs in bindings when they contain hash character

src/src/Vis/Widgets/JQui/BulkEditor.tsx

@@ -127,7 +127,7 @@ interface BulkEditorState {
     iconDialog: null | number;
     imageDialog: null | number;
     dialog: boolean;
-    dialogDelete: null | false | number;
+    dialogDelete: null | number;
     numbers?: boolean;
 }
 
@@ -394,7 +394,7 @@ class BulkEditor extends React.Component<BulkEditorProps, BulkEditorState> {
                     // @ts-expect-error this is fine
                     color="grey"
                     startIcon={<Close />}
-                    onClick={() => this.setState({ dialogDelete: false })}
+                    onClick={() => this.setState({ dialogDelete: null })}
                 >
                     {I18n.t('Cancel')}
                 </Button>
@@ -696,7 +696,7 @@ class BulkEditor extends React.Component<BulkEditorProps, BulkEditorState> {
                 {this.state.values[i]}
                 <IconButton
                     size="small"
-                    onClick={() => this.setState({ editDialog: { add: false, index: i, value: this.state.values[i] } })}
+                    onClick={() => this.setState({ editDialog: { add: true, index: i, value: this.state.values[i] } })}
                 >
                     <Edit />
                 </IconButton>