onChange event takes closedTabIDs and

openedTabIDs as a parameter

Author: dev-javascript

src/useDynamicTabs/useDynamicTabs.js

@@ -28,9 +28,9 @@ function useDynamicTabs(getDeps, options = {}) {
   }, []);
   useLayoutEffect(() => {
     const oldState = api.getCopyPerviousData(),
-      [openedTabsId, closedTabsId] = api.helper.getArraysDiff(state.openTabIDs, oldState.openTabIDs),
+      [openedTabIDs, closedTabIDs] = api.helper.getArraysDiff(state.openTabIDs, oldState.openTabIDs),
       isSwitched = oldState.selectedTabID !== state.selectedTabID;
-    api.onChange({newState: state, oldState, closedTabsId, openedTabsId, isSwitched});
+    api.onChange({newState: state, oldState, closedTabIDs, openedTabIDs, isSwitched});
   }, [state]);
   useLayoutEffect(() => {
     api.trigger('_onFlushEffects', api.userProxy, {currentData: api.getCopyData(), instance: api.userProxy});

src/utils/api/api.factory.js

@@ -163,28 +163,16 @@ const _apiProps = {
   },
 };
 Helper.setNoneEnumProps(_apiProps, {
-  // getInitialState: function () {
-  //     if (!this._initialState) {
-  //         const { selectedTabID, tabs, defaultPanelComponent } = this.optionsManager.options, openTabIDs = [];
-  //         tabs.map(tab => {
-  //             const newTab = this._addTab(tab, { defaultPanelComponent });
-  //             openTabIDs.push(newTab.id);
-  //         });
-  //         this._initialState = {
-  //             selectedTabID: selectedTabID + '', //make sure it is type of string
-  //             openTabIDs
-  //         };
-  //     }
-  //     return this._initialState;
-  // },
-  onChange: function ({newState, oldState, closedTabsId, openedTabsId, isSwitched}) {
-    if (isSwitched || openedTabsId.length || closedTabsId.length) {
+  onChange: function ({newState, oldState, closedTabIDs, openedTabIDs, isSwitched}) {
+    if (isSwitched || openedTabIDs.length || closedTabIDs.length) {
       this.trigger('onChange', this.userProxy, {
         currentData: {...newState},
         perviousData: {...oldState},
+        closedTabIDs,
+        openedTabIDs,
       });
-      openedTabsId.length && this.trigger('onOpen', this.userProxy, openedTabsId);
-      closedTabsId.length && this.trigger('onClose', this.userProxy, closedTabsId);
+      openedTabIDs.length && this.trigger('onOpen', this.userProxy, openedTabIDs);
+      closedTabIDs.length && this.trigger('onClose', this.userProxy, closedTabIDs);
       isSwitched &&
         this.trigger('onSelect', this.userProxy, {
           currentSelectedTabId: newState.selectedTabID,