@instructure/ui-responsive 7.15.0 → 7.16.0

package/CHANGELOG.md

@@ -3,6 +3,12 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [7.16.0](https://github.com/instructure/instructure-ui/compare/v7.15.0...v7.16.0) (2022-02-08)
+
+### Features
+
+- **ui-responsive:** modify Responsive "props override" error to warning ([a2eb4b8](https://github.com/instructure/instructure-ui/commit/a2eb4b8))
+
 # [7.15.0](https://github.com/instructure/instructure-ui/compare/v7.14.1...v7.15.0) (2022-02-03)
 
 **Note:** Version bump only for package @instructure/ui-responsive

package/es/Responsive/index.js

@@ -2,6 +2,7 @@ import _classCallCheck from "@babel/runtime/helpers/esm/classCallCheck";
 import _createClass from "@babel/runtime/helpers/esm/createClass";
 import _inherits from "@babel/runtime/helpers/esm/inherits";
 import _createSuper from "@babel/runtime/helpers/esm/createSuper";
+import { warn as _warn } from "@instructure/console";
 import { error as _error } from "@instructure/console";
 
 /*
@@ -147,13 +148,13 @@ var Responsive = /*#__PURE__*/function (_Component) {
         var matchProps = props[match]; // Iterate over the props for the current match. If that the prop is
         // already in `mergedProps` that means that the prop was defined for
         // multiple breakpoints, and more than one of those breakpoints is being
-        // currently applied so we log an error.
+        // currently applied so we log a warning.
 
         Object.keys(matchProps).forEach(function (prop) {
           /*#__PURE__*/
 
           /*#__PURE__*/
-          _error(!(prop in mergedProps), ["[Responsive] The prop `".concat(prop, "` is defined at 2 or more breakpoints"), "which are currently applied at the same time. Its current value, `".concat(mergedProps[prop], "`,"), "will be overwritten as `".concat(matchProps[prop], "`.")].join(' '));
+          _warn(!(prop in mergedProps), ["[Responsive] The prop `".concat(prop, "` is defined at 2 or more breakpoints"), "which are currently applied at the same time. Its current value, `".concat(mergedProps[prop], "`,"), "will be overwritten as `".concat(matchProps[prop], "`.")].join(' '));
 
           mergedProps[prop] = matchProps[prop];
         });

package/lib/Responsive/index.js

@@ -167,11 +167,11 @@ var Responsive = /*#__PURE__*/function (_Component) {
         var matchProps = props[match]; // Iterate over the props for the current match. If that the prop is
         // already in `mergedProps` that means that the prop was defined for
         // multiple breakpoints, and more than one of those breakpoints is being
-        // currently applied so we log an error.
+        // currently applied so we log a warning.
 
         Object.keys(matchProps).forEach(function (prop) {
           /*#__PURE__*/
-          ( /*#__PURE__*/0, _console.error)(!(prop in mergedProps), ["[Responsive] The prop `".concat(prop, "` is defined at 2 or more breakpoints"), "which are currently applied at the same time. Its current value, `".concat(mergedProps[prop], "`,"), "will be overwritten as `".concat(matchProps[prop], "`.")].join(' '));
+          ( /*#__PURE__*/0, _console.warn)(!(prop in mergedProps), ["[Responsive] The prop `".concat(prop, "` is defined at 2 or more breakpoints"), "which are currently applied at the same time. Its current value, `".concat(mergedProps[prop], "`,"), "will be overwritten as `".concat(matchProps[prop], "`.")].join(' '));
           mergedProps[prop] = matchProps[prop];
         });
       });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@instructure/ui-responsive",
-  "version": "7.15.0",
+  "version": "7.16.0",
   "description": "A component that allows for rendering a component differently based on either the element or the viewport size",
   "author": "Instructure, Inc. Engineering and Product Design",
   "type": "commonjs",
@@ -23,19 +23,19 @@
   "license": "MIT",
   "dependencies": {
     "@babel/runtime": "^7.9.2",
-    "@instructure/console": "^7.15.0",
-    "@instructure/debounce": "^7.15.0",
-    "@instructure/ui-dom-utils": "^7.15.0",
-    "@instructure/ui-react-utils": "^7.15.0",
-    "@instructure/ui-testable": "^7.15.0",
-    "@instructure/ui-themeable": "^7.15.0",
-    "@instructure/ui-utils": "^7.15.0",
+    "@instructure/console": "^7.16.0",
+    "@instructure/debounce": "^7.16.0",
+    "@instructure/ui-dom-utils": "^7.16.0",
+    "@instructure/ui-react-utils": "^7.16.0",
+    "@instructure/ui-testable": "^7.16.0",
+    "@instructure/ui-themeable": "^7.16.0",
+    "@instructure/ui-utils": "^7.16.0",
     "prop-types": "^15"
   },
   "devDependencies": {
-    "@instructure/ui-babel-preset": "^7.15.0",
-    "@instructure/ui-color-utils": "^7.15.0",
-    "@instructure/ui-test-utils": "^7.15.0"
+    "@instructure/ui-babel-preset": "^7.16.0",
+    "@instructure/ui-color-utils": "^7.16.0",
+    "@instructure/ui-test-utils": "^7.16.0"
   },
   "peerDependencies": {
     "react": "^16.8"

package/src/Responsive/index.js

@@ -26,7 +26,7 @@ import React, { Component } from 'react'
 import PropTypes from 'prop-types'
 
 import { deepEqual } from '@instructure/ui-utils'
-import { error } from '@instructure/console/macro'
+import { error, warn } from '@instructure/console/macro'
 
 import {
   addElementQueryMatchListener,
@@ -163,9 +163,9 @@ class Responsive extends Component {
       // Iterate over the props for the current match. If that the prop is
       // already in `mergedProps` that means that the prop was defined for
       // multiple breakpoints, and more than one of those breakpoints is being
-      // currently applied so we log an error.
+      // currently applied so we log a warning.
       Object.keys(matchProps).forEach((prop) => {
-        error(
+        warn(
           !(prop in mergedProps),
           [
             `[Responsive] The prop \`${prop}\` is defined at 2 or more breakpoints`,