@etsoo/materialui 1.0.27 → 1.0.28

package/lib/SelectEx.d.ts

@@ -45,10 +45,6 @@ export declare type SelectExProps<T extends object, D extends DataTypes.Keys<T>
      * Item click handler
      */
     onItemClick?: (event: React.MouseEvent, option: T) => void;
-    /**
-     * Item mouse down handler
-     */
-    onItemMouseDown?: (event: React.MouseEvent, option: T) => void;
     /**
      * On load data handler
      */

package/lib/SelectEx.js

@@ -11,7 +11,7 @@ import { Utils } from '@etsoo/shared';
 export function SelectEx(props) {
     var _a;
     // Destruct
-    const { defaultValue, idField = 'id', error, helperText, inputRequired, itemIconRenderer, itemStyle, label, labelField = 'label', loadData, onItemChange, onItemClick, onItemMouseDown, onLoadData, multiple = false, name, options, search = false, autoAddBlankItem = search, value, onChange, fullWidth, ...rest } = props;
+    const { defaultValue, idField = 'id', error, helperText, inputRequired, itemIconRenderer, itemStyle, label, labelField = 'label', loadData, onItemChange, onItemClick, onLoadData, multiple = false, name, options, search = false, autoAddBlankItem = search, value, onChange, fullWidth, ...rest } = props;
     // Options state
     const [localOptions, setOptions] = React.useState([]);
     const isMounted = React.useRef(true);
@@ -135,8 +135,12 @@ export function SelectEx(props) {
         React.createElement(Select, { ref: divRef, value: localOptions.some((option) => itemChecked(getId(option)))
                 ? valueState !== null && valueState !== void 0 ? valueState : ''
                 : '', input: React.createElement(OutlinedInput, { notched: true, label: label, required: inputRequired }), labelId: labelId, name: name, multiple: multiple, onChange: (event, child) => {
-                if (onChange)
+                if (onChange) {
                     onChange(event, child);
+                    // event.preventDefault() will block executing
+                    if (event.defaultPrevented)
+                        return;
+                }
                 doItemChange(localOptions, event.target.value, true);
                 handleChange(event);
             }, renderValue: (selected) => {
@@ -160,10 +164,6 @@ export function SelectEx(props) {
                     if (onItemClick) {
                         onItemClick(event, option);
                     }
-                }, onMouseDown: (event) => {
-                    if (onItemMouseDown) {
-                        onItemMouseDown(event, option);
-                    }
                 }, style: itemStyle == null
                     ? undefined
                     : itemStyle(option) },

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@etsoo/materialui",
-    "version": "1.0.27",
+    "version": "1.0.28",
     "description": "TypeScript Material-UI Implementation",
     "main": "lib/index.js",
     "types": "lib/index.d.ts",

package/src/SelectEx.tsx

@@ -79,11 +79,6 @@ export type SelectExProps<
      */
     onItemClick?: (event: React.MouseEvent, option: T) => void;
 
-    /**
-     * Item mouse down handler
-     */
-    onItemMouseDown?: (event: React.MouseEvent, option: T) => void;
-
     /**
      * On load data handler
      */
@@ -124,7 +119,6 @@ export function SelectEx<
         loadData,
         onItemChange,
         onItemClick,
-        onItemMouseDown,
         onLoadData,
         multiple = false,
         name,
@@ -296,7 +290,12 @@ export function SelectEx<
                 name={name}
                 multiple={multiple}
                 onChange={(event, child) => {
-                    if (onChange) onChange(event, child);
+                    if (onChange) {
+                        onChange(event, child);
+
+                        // event.preventDefault() will block executing
+                        if (event.defaultPrevented) return;
+                    }
                     doItemChange(localOptions, event.target.value, true);
                     handleChange(event);
                 }}
@@ -333,11 +332,6 @@ export function SelectEx<
                                     onItemClick(event, option);
                                 }
                             }}
-                            onMouseDown={(event) => {
-                                if (onItemMouseDown) {
-                                    onItemMouseDown(event, option);
-                                }
-                            }}
                             style={
                                 itemStyle == null
                                     ? undefined