[@mantine/core] Fix Pill to respect disabled state (mantinedev#5959)

snturk · Mar 24, 2024 · 2685bae · 2685bae
1 parent e3e3bb8
commit 2685bae
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/packages/@mantine/core/src/components/Pill/Pill.tsx b/packages/@mantine/core/src/components/Pill/Pill.tsx
@@ -128,6 +128,7 @@ export const Pill = factory<PillFactory>((_props, ref) => {
           tabIndex={-1}
           aria-hidden
           unstyled={unstyled}
+          disabled={disabled}
           {...removeButtonProps}
           {...getStyles('remove', {
             className: removeButtonProps?.className,
@@ -139,6 +140,10 @@ export const Pill = factory<PillFactory>((_props, ref) => {
             removeButtonProps?.onMouseDown?.(event);
           }}
           onClick={(event) => {
+            if (disabled) {
+              event.preventDefault();
+              return;
+            }
             event.stopPropagation();
             onRemove?.();
             removeButtonProps?.onClick?.(event);

diff --git a/packages/@mantine/core/src/components/TagsInput/TagsInput.tsx b/packages/@mantine/core/src/components/TagsInput/TagsInput.tsx
@@ -301,6 +301,7 @@ export const TagsInput = factory<TagsInputFactory>((_props, ref) => {
     <Pill
       key={`${item}-${index}`}
       withRemoveButton={!readOnly}
+      disabled={disabled}
       onRemove={() => {
         setValue(_value.filter((i) => item !== i));
         onRemove?.(item);