rsuite 5.24.1 → 5.25.0

package/Breadcrumb/styles/index.less

@@ -16,6 +16,10 @@
     &:focus {
       .tab-focus();
     }
+
+    &-active {
+      color: var(--rs-breadcrumb-item-active-text);
+    }
   }
 
   // Separator has gap with text

package/Button/styles/index.less

@@ -135,6 +135,7 @@
     color: var(--rs-btn-ghost-hover-text);
     background-color: transparent;
     border-color: var(--rs-btn-ghost-hover-border);
+    box-shadow: 0 0 0 1px var(--rs-btn-ghost-hover-border);
   });
 
   .button-pressed({
@@ -147,12 +148,15 @@
     color: var(--rs-btn-ghost-text);
     background-color: transparent;
     opacity: 0.3;
+    border-color: var(--rs-btn-ghost-border);
+    box-shadow:none;
   });
 }
 
 // Spectrum buttons
 each(@spectrum, .(@color) {
   @C200: var(~'--rs-@{color}-200');
+  @C300: var(~'--rs-@{color}-300');
   @C400: var(~'--rs-@{color}-400');
   @C500: var(~'--rs-@{color}-500');
   @C600: var(~'--rs-@{color}-600');
@@ -162,15 +166,15 @@ each(@spectrum, .(@color) {
 
   .rs-btn-@{color} {
     --rs-btn-primary-bg: @C500;
-    --rs-btn-primary-hover-bg: @C600;
-    --rs-btn-primary-active-bg: @C700;
+    --rs-btn-primary-hover-bg: @C700;
+    --rs-btn-primary-active-bg: @C800;
     --rs-btn-subtle-hover-bg: @C500;
-    --rs-btn-subtle-hover-text: #fff;
+    --rs-btn-subtle-hover-text: @B200;
     --rs-btn-subtle-active-bg: @C600;
-    --rs-btn-subtle-active-text: #fff;
+    --rs-btn-subtle-active-text: @B800;
     --rs-btn-ghost-border: @C700;
     --rs-btn-ghost-text: @C700;
-    --rs-btn-ghost-hover-border: @C800;
+    --rs-btn-ghost-hover-border: @C900;
     --rs-btn-ghost-hover-text: @C800;
     --rs-btn-ghost-active-border: @C900;
     --rs-btn-ghost-active-text: @C900;
@@ -178,12 +182,12 @@ each(@spectrum, .(@color) {
     --rs-btn-link-hover-text: @C800;
     --rs-btn-link-active-text: @C900;
     --rs-iconbtn-primary-addon: @C600;
-    --rs-iconbtn-primary-activated-addon: @C700;
-    --rs-iconbtn-primary-pressed-addon: @C800;
+    --rs-iconbtn-primary-activated-addon: @C800;
+    --rs-iconbtn-primary-pressed-addon: @C900;
 
     .dark-mode({
       --rs-btn-primary-bg: @C700;
-      --rs-btn-primary-hover-bg: @C600;
+      --rs-btn-primary-hover-bg: @C500;
       --rs-btn-primary-active-bg: @C400;
       --rs-btn-subtle-hover-bg: @C600;
       --rs-btn-subtle-hover-text: #fff;
@@ -199,8 +203,8 @@ each(@spectrum, .(@color) {
       --rs-btn-link-hover-text: @C400;
       --rs-btn-link-active-text: @C200;
       --rs-iconbtn-primary-addon: @C600;
-      --rs-iconbtn-primary-activated-addon: @C500;
-      --rs-iconbtn-primary-pressed-addon: @C400;
+      --rs-iconbtn-primary-activated-addon: @C400;
+      --rs-iconbtn-primary-pressed-addon: @C300;
     });
 
     .high-contrast-mode({

package/CHANGELOG.md

@@ -1,3 +1,21 @@
+# [5.25.0](https://github.com/rsuite/rsuite/compare/v5.24.1...v5.25.0) (2023-01-20)
+
+### Bug Fixes
+
+- **Breadcrumb:** improve font color for active item ([#3028](https://github.com/rsuite/rsuite/issues/3028)) ([1efcbf9](https://github.com/rsuite/rsuite/commit/1efcbf90a345ee93928ca13c181c103d247e869d))
+- **Breadcrumb:** rendered breadcrumbs as span when href is not specified ([#3027](https://github.com/rsuite/rsuite/issues/3027)) ([8ad16e4](https://github.com/rsuite/rsuite/commit/8ad16e4d572b55892a50c0cad747554039b43398))
+- **Button:** fix disabled icon button bug in dark mode ([#3020](https://github.com/rsuite/rsuite/issues/3020)) ([8975eff](https://github.com/rsuite/rsuite/commit/8975eff6b6e9a4ec9c2ed4c58cfbdf4f23e5d94a))
+- **DatePicker:** fix first click not working on month selection ([#3019](https://github.com/rsuite/rsuite/issues/3019)) ([42eeaa3](https://github.com/rsuite/rsuite/commit/42eeaa3cc5e19442157f771f02dab850f257c2dc))
+- **Whisper:** add type checking for child components ([#3012](https://github.com/rsuite/rsuite/issues/3012)) ([c189119](https://github.com/rsuite/rsuite/commit/c18911903141423d79eb79d5313bc47dd974705e))
+
+### Features
+
+- **CustomProvider:** add support for disableRipple on `<CustomProvider>` ([#3030](https://github.com/rsuite/rsuite/issues/3030)) ([c6bb7ad](https://github.com/rsuite/rsuite/commit/c6bb7ade76ea843b43e799c5cb797b6b5e69d98f))
+
+### Performance Improvements
+
+- **Button:** improved button color when hovered and activated ([#3022](https://github.com/rsuite/rsuite/issues/3022)) ([7973696](https://github.com/rsuite/rsuite/commit/7973696c2854d7199fc27d040dffe45dc356dc9a))
+
 ## [5.24.1](https://github.com/rsuite/rsuite/compare/v5.24.0...v5.24.1) (2023-01-12)
 
 ### Bug Fixes

package/Picker/styles/index.less

@@ -233,14 +233,6 @@
 
   .rs-picker-cleanable.rs-picker-has-value & {
     padding-right: (@dropdown-toggle-padding-right + @picker-toggle-clean-width);
-
-    .rs-@{date-picker-prefix}&,
-    .rs-@{date-range-picker-prefix}& {
-      padding-right: (
-        @dropdown-toggle-padding-right + @picker-toggle-clean-width +
-          @picker-item-content-padding-vertical
-      );
-    }
   }
 }
 
@@ -261,6 +253,11 @@
     color: var(--rs-state-error);
   }
 
+  &:hover svg path {
+    stroke: var(--rs-state-error);
+    stroke-width: 1;
+  }
+
   .rs-@{date-picker-prefix} &,
   .rs-@{date-range-picker-prefix} & {
     right: (@picker-toggle-clean-right + @picker-item-content-padding-vertical);

package/Picker/styles/mixin.less

@@ -99,28 +99,18 @@
   }
 
   .rs-picker-has-value.rs-picker-cleanable & {
-    padding-right: (
-      @padding-horizontal + @dropdown-caret-width + @dropdown-caret-padding + @clean-btn-width +
-        @clean-btn-margin
-    );
+    padding-right: (@padding-horizontal + @dropdown-caret-width + @dropdown-caret-padding);
   }
 
   .rs-picker-toggle-caret,
   .rs-picker-toggle-clean {
     top: @padding-vertical;
+    right: @padding-horizontal;
 
     .rs-picker-default & {
       top: @padding-vertical - @picker-default-toggle-border-width;
     }
   }
-
-  .rs-picker-toggle-caret {
-    right: @padding-horizontal;
-  }
-
-  .rs-picker-toggle-clean {
-    right: (@dropdown-caret-width + @padding-horizontal + @clean-btn-width - @clean-btn-margin);
-  }
 }
 
 .picker-button-caret-lg() {
@@ -156,19 +146,12 @@
   .rs-picker-toggle-clean,
   .rs-picker-toggle-caret {
     top: @top;
+    right: @padding-horizontal;
 
     .rs-picker-subtle & {
       top: @top - @picker-default-toggle-border-width;
     }
   }
-
-  .rs-picker-toggle-caret {
-    right: @padding-horizontal;
-  }
-
-  .rs-picker-toggle-clean {
-    right: (@padding-horizontal + @toggle-clean-padding + @calendar-caret-width);
-  }
 }
 
 .date-picker-button-caret-lg() {

package/cjs/@types/global.d.ts

@@ -1 +1,7 @@
 declare const __DEV__: boolean;
+
+declare namespace Chai {
+  interface Assertion {
+    toHaveError(expectedMessage?: string): Assertion;
+  }
+}

package/cjs/Breadcrumb/BreadcrumbItem.js

@@ -19,7 +19,7 @@ var _utils = require("../utils");
 
 var BreadcrumbItem = /*#__PURE__*/_react.default.forwardRef(function (props, ref) {
   var _props$as = props.as,
-      Component = _props$as === void 0 ? _SafeAnchor.default : _props$as,
+      Component = _props$as === void 0 ? props.href ? _SafeAnchor.default : 'span' : _props$as,
       href = props.href,
       _props$classPrefix = props.classPrefix,
       classPrefix = _props$classPrefix === void 0 ? 'breadcrumb-item' : _props$classPrefix,

package/cjs/Calendar/MonthDropdown.js

@@ -78,8 +78,7 @@ var MonthDropdown = /*#__PURE__*/_react.default.forwardRef(function (props, ref)
 
     return false;
   }, [disabledMonth]);
-
-  var rowRenderer = function rowRenderer(_ref) {
+  var rowRenderer = (0, _react.useCallback)(function (_ref) {
     var index = _ref.index,
         style = _ref.style;
 
@@ -115,8 +114,7 @@ var MonthDropdown = /*#__PURE__*/_react.default.forwardRef(function (props, ref)
         year: year
       });
     })));
-  };
-
+  }, [date, isMonthDisabled, merge, prefix, rowCount, startYear]);
   var classes = merge(className, withClassPrefix(), {
     show: show
   });

package/cjs/CustomProvider/CustomProvider.d.ts

@@ -38,6 +38,8 @@ export interface CustomValue<T = Locale> {
      * A Map of toast containers
      */
     toasters?: React.MutableRefObject<Map<string, ToastContainerInstance>>;
+    /** If true, the ripple effect is disabled. Affected components include: Button, Nav.Item, Pagination. */
+    disableRipple?: boolean;
 }
 export interface CustomProviderProps<T = Locale> extends Partial<CustomValue<T>> {
     /** Supported themes */

package/cjs/CustomProvider/CustomProvider.js

@@ -32,7 +32,8 @@ var CustomProvider = function CustomProvider(props) {
       classPrefix = _props$classPrefix === void 0 ? (0, _prefix.getClassNamePrefix)() : _props$classPrefix,
       theme = props.theme,
       container = props.toastContainer,
-      rest = (0, _objectWithoutPropertiesLoose2.default)(props, ["children", "classPrefix", "theme", "toastContainer"]);
+      disableRipple = props.disableRipple,
+      rest = (0, _objectWithoutPropertiesLoose2.default)(props, ["children", "classPrefix", "theme", "toastContainer", "disableRipple"]);
 
   var toasters = _react.default.useRef(new Map());
 
@@ -46,9 +47,10 @@ var CustomProvider = function CustomProvider(props) {
     return (0, _extends2.default)({
       classPrefix: classPrefix,
       theme: theme,
-      toasters: toasters
+      toasters: toasters,
+      disableRipple: disableRipple
     }, rest);
-  }, [classPrefix, theme, rest]);
+  }, [classPrefix, theme, disableRipple, rest]);
 
   (0, _react.useEffect)(function () {
     if (_DOMHelper.canUseDOM && theme) {

package/cjs/Overlay/OverlayTrigger.js

@@ -393,6 +393,10 @@ var OverlayTrigger = /*#__PURE__*/_react.default.forwardRef(function (props, ref
     } : speaker);
   };
 
+  if (typeof children === 'object' && children.type === _react.default.Fragment || typeof children === 'string') {
+    console.error('[rsuite] The OverlayTrigger component does not accept strings or Fragments as child.');
+  }
+
   return /*#__PURE__*/_react.default.createElement(_react.default.Fragment, null, typeof children === 'function' ? children(triggerEvents, triggerRef) : /*#__PURE__*/_react.default.cloneElement(children, (0, _extends2.default)({
     ref: triggerRef,
     'aria-describedby': controlId

package/cjs/Picker/PickerToggle.js

@@ -247,7 +247,7 @@ var PickerToggle = /*#__PURE__*/_react.default.forwardRef(function (props, ref)
       closeLabel: 'Clear'
     },
     onClick: handleClean
-  }), caret && /*#__PURE__*/_react.default.createElement(Caret, {
+  }), caret && !showCleanButton && /*#__PURE__*/_react.default.createElement(Caret, {
     className: prefix(_templateObject2 || (_templateObject2 = (0, _taggedTemplateLiteralLoose2.default)(["caret"])))
   })));
 });

package/cjs/Picker/PickerToggleTrigger.js

@@ -1,7 +1,5 @@
 "use strict";
 
-var _interopRequireWildcard = require("@babel/runtime/helpers/interopRequireWildcard");
-
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
 
 exports.__esModule = true;
@@ -11,7 +9,7 @@ var _extends2 = _interopRequireDefault(require("@babel/runtime/helpers/extends")
 
 var _objectWithoutPropertiesLoose2 = _interopRequireDefault(require("@babel/runtime/helpers/objectWithoutPropertiesLoose"));
 
-var _react = _interopRequireWildcard(require("react"));
+var _react = _interopRequireDefault(require("react"));
 
 var _pick = _interopRequireDefault(require("lodash/pick"));
 
@@ -19,8 +17,6 @@ var _OverlayTrigger = _interopRequireDefault(require("../Overlay/OverlayTrigger"
 
 var _utils = require("../utils");
 
-var _CustomProvider = require("../CustomProvider");
-
 var omitTriggerPropKeys = ['onEntered', 'onExited', 'onEnter', 'onEntering', 'onExit', 'onExiting', 'open', 'defaultOpen', 'onHide', 'container', 'containerPadding', 'preventOverflow'];
 exports.omitTriggerPropKeys = omitTriggerPropKeys;
 var pickTriggerPropKeys = [].concat(omitTriggerPropKeys, ['disabled', 'plaintext', 'readOnly', 'loading']);
@@ -34,12 +30,15 @@ var PickerToggleTrigger = /*#__PURE__*/_react.default.forwardRef(function (props
       trigger = _props$trigger === void 0 ? 'click' : _props$trigger,
       rest = (0, _objectWithoutPropertiesLoose2.default)(props, ["pickerProps", "speaker", "placement", "trigger"]);
   var pickerTriggerProps = (0, _pick.default)(pickerProps, pickTriggerPropKeys);
-  var context = (0, _react.useContext)(_CustomProvider.CustomContext);
+
+  var _useCustom = (0, _utils.useCustom)(),
+      rtl = _useCustom.rtl;
+
   return /*#__PURE__*/_react.default.createElement(_OverlayTrigger.default, (0, _extends2.default)({}, rest, pickerTriggerProps, {
     disabled: pickerTriggerProps.disabled || pickerTriggerProps.loading,
     ref: ref,
     trigger: trigger,
-    placement: (0, _utils.placementPolyfill)(placement, context === null || context === void 0 ? void 0 : context.rtl),
+    placement: (0, _utils.placementPolyfill)(placement, rtl),
     speaker: speaker
   }));
 });

package/cjs/Ripple/Ripple.js

@@ -39,6 +39,9 @@ var getPosition = function getPosition(target, event) {
 };
 
 var Ripple = /*#__PURE__*/_react.default.forwardRef(function (props, ref) {
+  var _useCustom = (0, _utils.useCustom)(),
+      disableRipple = _useCustom.disableRipple;
+
   var _props$as = props.as,
       Component = _props$as === void 0 ? 'span' : _props$as,
       className = props.className,
@@ -88,6 +91,11 @@ var Ripple = /*#__PURE__*/_react.default.forwardRef(function (props, ref) {
       };
     }
   }, [handleMouseDown]);
+
+  if (disableRipple) {
+    return null;
+  }
+
   return /*#__PURE__*/_react.default.createElement(Component, (0, _extends2.default)({}, rest, {
     className: classes,
     ref: (0, _utils.mergeRefs)(triggerRef, ref)

package/cjs/utils/useCustom.js

@@ -41,7 +41,8 @@ function useCustom(keys, overrideLocale) {
       rtl = _useContext$rtl === void 0 ? getDefaultRTL() : _useContext$rtl,
       formatDate = _useContext.formatDate,
       parseDate = _useContext.parseDate,
-      toasters = _useContext.toasters;
+      toasters = _useContext.toasters,
+      disableRipple = _useContext.disableRipple;
 
   var componentLocale = (0, _extends2.default)({}, locale === null || locale === void 0 ? void 0 : locale.common, typeof keys === 'string' ? locale === null || locale === void 0 ? void 0 : locale[keys] : typeof keys === 'object' ? mergeObject(keys.map(function (key) {
     return locale === null || locale === void 0 ? void 0 : locale[key];
@@ -69,6 +70,7 @@ function useCustom(keys, overrideLocale) {
     locale: componentLocale,
     rtl: rtl,
     toasters: toasters,
+    disableRipple: disableRipple,
     formatDate: formatDate || defaultFormatDate,
     parseDate: parseDate || defaultParseDate
   };