rename perviousSelectedTabId to

previousSelectedTabId

Author: dev-javascript

README.md

@@ -524,12 +524,12 @@ instance
 
 ```js
 const [TabList, PanelList, ready] = useDynTabs({
-  onSelect: function ({currentSelectedTabId, perviousSelectedTabId}) {
+  onSelect: function ({currentSelectedTabId, previousSelectedTabId}) {
     // you can use 'this' here which refers to the instance
   },
 });
 // or
-instance.setOption('onSelect', ({currentSelectedTabId, perviousSelectedTabId}) => {}).refresh();
+instance.setOption('onSelect', ({currentSelectedTabId, previousSelectedTabId}) => {}).refresh();
 ```
 
 ### onOpen
@@ -858,7 +858,7 @@ Parameters:
 
 ```js
 instance.on('onSelect', function (params) {
-  const {currentSelectedTabId, perviousSelectedTabId} = params;
+  const {currentSelectedTabId, previousSelectedTabId} = params;
   // can use 'this' here which refers to the instance
 });
 ```
@@ -877,7 +877,7 @@ Parameters:
 **Example**
 
 ```js
-instance.one('onSelect', function ({currentSelectedTabId, perviousSelectedTabId}) {
+instance.one('onSelect', function ({currentSelectedTabId, previousSelectedTabId}) {
   // can use 'this' here which refers to the instance
 });
 ```
@@ -897,7 +897,7 @@ Parameters:
 
 ```js
 const onSelectHandler = function (params) {
-  const {currentSelectedTabId, perviousSelectedTabId} = params;
+  const {currentSelectedTabId, previousSelectedTabId} = params;
   // can use 'this' here which refers to the instance
   this.off('onSelect', onSelectHandler);
 };
@@ -1023,6 +1023,8 @@ const open_tab_3 = function () {
 };
 ```
 
+- First parameter of onSelect function is an object and has perviousSelectedTabId property which is deprecated. you should use previousSelectedTabId property instead of perviousSelectedTabId property.
+
 ## Test
 
 ```js

example/basic/com/basic/service/index.js

@@ -35,7 +35,7 @@ export default {
         const context = this;
         return true;
       },
-      onSelect: ({currentSelectedTabId, perviousSelectedTabId}) => {
+      onSelect: ({currentSelectedTabId, previousSelectedTabId}) => {
         const context = this;
       },
       beforeClose: function (e, id) {

src/index.test.js

@@ -191,8 +191,16 @@ describe('select method : ', () => {
     expect(op.onInit.mock.calls.length === 2).toBe(true);
     expect(op.onChange.mock.calls.length === 1).toBe(true);
     expect(op.onSelect.mock.calls.length === 1).toBe(true);
-    expect(op.onSelect.mock.calls[0][0]).toEqual({currentSelectedTabId: '2', perviousSelectedTabId: '1'});
-    expect(onSelect.mock.calls[0][0]).toEqual({currentSelectedTabId: '2', perviousSelectedTabId: '1'});
+    expect(op.onSelect.mock.calls[0][0]).toEqual({
+      currentSelectedTabId: '2',
+      perviousSelectedTabId: '1',
+      previousSelectedTabId: '1',
+    });
+    expect(onSelect.mock.calls[0][0]).toEqual({
+      currentSelectedTabId: '2',
+      perviousSelectedTabId: '1',
+      previousSelectedTabId: '1',
+    });
     expect(op.onInit).toHaveBeenCalledBefore(op.onChange);
     expect(op.onChange).toHaveBeenCalledBefore(op.onSelect);
     expect(op.onSelect).toHaveBeenCalledBefore(onSelect);

src/utils/api/api.factory.js

@@ -180,6 +180,7 @@ Helper.setNoneEnumProps(_apiProps, {
       isSwitched &&
         this.trigger('onSelect', this.userProxy, {
           currentSelectedTabId: newState.selectedTabID,
+          previousSelectedTabId: oldState.selectedTabID,
           perviousSelectedTabId: oldState.selectedTabID,
         });
     }