[bugfix] Fix issue where an enabled element that has been disabled be… (#263)

* [bugfix] Fix issue where an enabled element that has been disabled before is not toggleable. Also update the disableElement API to disable left panel button as well if the left panel is disabled

* remove - in the viewer comment

Author: ZhijieZhang

src/redux/actions/exposedActions.js

@@ -75,7 +75,7 @@ export const closeElements = dataElements => dispatch => {
 export const toggleElement = dataElement => (dispatch, getState) => {
   const state = getState();
 
-  if (state.viewer.disabledElements[dataElement]) {
+  if (state.viewer.disabledElements[dataElement] && state.viewer.disabledElements[dataElement].disabled) {
     return;
   }
 

src/redux/actions/internalActions.js

@@ -5,7 +5,9 @@ import core from 'core';
 
 // viewer
 export const disableElement = (dataElement, priority) => (dispatch, getState) => {
-  if (dataElement === 'stylePopup') {
+  if (dataElement === 'leftPanel') {
+    dispatch(disableElements(['leftPanel', 'leftPanelButton'], priority));
+  } else if (dataElement === 'stylePopup') {
     dispatch(disableElements(['toolStylePopup', 'annotationStylePopup'], priority));
   } else {
     const currentPriority = selectors.getDisabledElementPriority(getState(), dataElement);
@@ -19,7 +21,9 @@ export const disableElements = (dataElements, priority) => (dispatch, getState)
   dispatch({ type: 'DISABLE_ELEMENTS', payload: { dataElements: filteredDataElements, priority } });
 };
 export const enableElement = (dataElement, priority) => (dispatch, getState) => {
-  if (dataElement === 'stylePopup') {
+  if (dataElement === 'leftPanel') {
+    dispatch(enableElements(['leftPanel', 'leftPanelButton'], priority));
+  } else if (dataElement === 'stylePopup') {
     dispatch(enableElements(['toolStylePopup', 'annotationStylePopup'], priority));
   } else {
     const currentPriority = selectors.getDisabledElementPriority(getState(), dataElement);