Merge pull request #91 from coreui/dev-v2.5.3

v2.5.3

Author: xidedix

CHANGELOG.md

@@ -1,5 +1,17 @@
 ### [@coreui/react](https://coreui.io/) changelog
 
+##### `v2.5.3`
+- fix(SidebarNav): add missing `itemAttr` to item/item with children (optional)
+- chore: update demo styles `@coreui/coreui` to `v2.1.12`
+ 
+###### dependencies update
+- update `react` to `^16.10.1`
+- update `react-dom` to `^16.10.1`
+- update `react-router-dom` to `^5.1.2`
+- update `babel-eslint` to `^10.0.3`
+- update `eslint-plugin-react` to `^7.15.1`
+- update `webpack-dev-server` to `^3.8.1`
+
 ##### `v2.5.2`
 - fix(Switch): does not provide any keyboard accessibility - thanks @roastery-zz close #44
 - fix(Switch): checked props and state out of sync - thanks @gravitymedianet @jinixx

demo/src/scss/style.css

@@ -1,6 +1,6 @@
 {
   "name": "@coreui/react",
-  "version": "2.5.2",
+  "version": "2.5.3",
   "description": "CoreUI React Bootstrap 4 components",
   "license": "MIT",
   "author": {
@@ -45,22 +45,22 @@
   },
   "peerDependencies": {
     "@coreui/coreui": "^2.1.12",
-    "react": "^16.8.6",
-    "react-router-dom": "^5.0.1"
+    "react": "^16.10.1",
+    "react-router-dom": "^5.1.2"
   },
   "devDependencies": {
-    "babel-eslint": "^10.0.2",
+    "babel-eslint": "^10.0.3",
     "enzyme": "^3.10.0",
     "enzyme-adapter-react-16": "^1.14.0",
     "eslint": "^5.16.0",
     "eslint-plugin-import": "^2.18.2",
-    "eslint-plugin-react": "^7.14.3",
+    "eslint-plugin-react": "^7.15.1",
     "nwb": "^0.23.0",
-    "react": "^16.8.6",
-    "react-dom": "^16.8.6",
-    "react-router-dom": "^5.0.1",
+    "react": "^16.10.1",
+    "react-dom": "^16.10.1",
+    "react-router-dom": "^5.1.2",
     "sinon": "^5.1.1",
-    "webpack-dev-server": "^3.7.2"
+    "webpack-dev-server": "^3.8.1"
   },
   "repository": {
     "type": "git",

package.json

@@ -28,6 +28,7 @@ const BreadcrumbsItem = ({ match }) => {
   const routeName = findRouteName(match.url);
   if (routeName) {
     return (
+        // eslint-disable-next-line react/prop-types
       match.isExact ?
         <BreadcrumbItem active>{routeName}</BreadcrumbItem>
        :

src/Breadcrumb.js

@@ -30,6 +30,7 @@ const BreadcrumbsItem2 = ({ match }) => {
   const Link = router.Link;
   if (routeName) {
     return (
+        // eslint-disable-next-line react/prop-types
       match.isExact ?
         <BreadcrumbItem active>{routeName}</BreadcrumbItem>
        :

src/Breadcrumb2.js

@@ -49,7 +49,8 @@ router | inject `react-router-dom@5` object | _mandatory for @coreui/react ^2.5.
     text: 'NEW',
     class: ''                   // optional
   },
-  attributes: { target: '_blank', rel: "noreferrer noopener", disabled: false, hidden: false }, // (v2.1.0 up) optional valid JS object with JS API naming
+  attributes: { target: '_blank', rel: "noreferrer noopener", disabled: false, hidden: false }, // (v2.1.0 up) link html attributes - optional valid JS object with JS API naming
+  itemAttr: { id: 'item-1'},    // item html attributes - optional
 },
 ```
 - item with `children` array - works like `nav-dropdown-toggle` with `nav-dropdown-items`
@@ -59,7 +60,8 @@ router | inject `react-router-dom@5` object | _mandatory for @coreui/react ^2.5.
   url: '/icons',
   icon: 'icon-star',
   class: 'text-uppercase',             // (v2.5.1 up) optional
-  attributes: { class: 'bg-success' }, // (v2.5.1 up) optional valid JS object with JS API naming,
+  attributes: { class: 'bg-success' }, // (v2.5.1 up) nav-dropdown link html attributes - optional valid JS object with JS API naming,
+  itemAttr: { id: 'drop-1' },          // (v2.5.3 up) item html attributes - optional
   children: [
     {
       name: 'Flags',     // item options apply