21128: changed rowClick to toggle instead of select Arvados-DCO-1.1-Signed-off-by...
authorLisa Knox <lisaknox83@gmail.com>
Fri, 17 Nov 2023 14:27:52 +0000 (09:27 -0500)
committerLisa Knox <lisaknox83@gmail.com>
Fri, 17 Nov 2023 14:27:52 +0000 (09:27 -0500)
src/store/details-panel/details-panel-action.ts
src/store/multiselect/multiselect-actions.tsx
src/store/multiselect/multiselect-reducer.tsx

index ef5f6095c07725c29a6a02f49d23c56d3286941c..dfb708f25a80a62559f1784bfaae920173fb957d 100644 (file)
@@ -13,7 +13,7 @@ import { FilterBuilder } from 'services/api/filter-builder';
 import { OrderBuilder } from 'services/api/order-builder';
 import { CollectionResource } from 'models/collection';
 import { extractUuidKind, ResourceKind } from 'models/resource';
-import { selectOne } from 'store/multiselect/multiselect-actions';
+import { toggleOne } from 'store/multiselect/multiselect-actions';
 
 export const SLIDE_TIMEOUT = 500;
 
@@ -37,7 +37,7 @@ export const loadDetailsPanel = (uuid: string) =>
                     break;
             }
         }
-        dispatch<any>(selectOne(uuid))
+        dispatch<any>(toggleOne(uuid))
         dispatch(detailsPanelActions.LOAD_DETAILS_PANEL(uuid));
     };
 
index c5ad4948b5a2ee569ba6f3ae9ddf555f84a9f8b2..6eef131dab65a2198e0b674734f49cf190220cf5 100644 (file)
@@ -9,6 +9,7 @@ export const multiselectActionContants = {
     SET_CHECKEDLIST: "SET_CHECKEDLIST",
     SELECT_ONE: 'SELECT_ONE',
     DESELECT_ONE: "DESELECT_ONE",
+    TOGGLE_ONE: 'TOGGLE_ONE'
 };
 
 export const toggleMSToolbar = (isVisible: boolean) => {
@@ -35,6 +36,12 @@ export const deselectOne = (uuid: string) => {
     };
 };
 
+export const toggleOne = (uuid: string) => {
+    return dispatch => {
+        dispatch({ type: multiselectActionContants.TOGGLE_ONE, payload: uuid });
+    };
+};
+
 export const multiselectActions = {
     toggleMSToolbar,
     setCheckedListOnStore,
index 8b70ceccb0788a9cff31bc40909c7bcdbdbcacd2..099a1b5bf836c9836ac8f143405edf8f1424112d 100644 (file)
@@ -15,7 +15,7 @@ const multiselectToolbarInitialState = {
     checkedList: {},
 };
 
-const { TOGGLE_VISIBLITY, SET_CHECKEDLIST, SELECT_ONE, DESELECT_ONE } = multiselectActionContants;
+const { TOGGLE_VISIBLITY, SET_CHECKEDLIST, SELECT_ONE, DESELECT_ONE, TOGGLE_ONE } = multiselectActionContants;
 
 export const multiselectReducer = (state: MultiselectToolbarState = multiselectToolbarInitialState, action) => {
     switch (action.type) {
@@ -27,6 +27,8 @@ export const multiselectReducer = (state: MultiselectToolbarState = multiselectT
             return { ...state, checkedList: { ...state.checkedList, [action.payload]: true } };
         case DESELECT_ONE:
             return { ...state, checkedList: { ...state.checkedList, [action.payload]: false } };
+        case TOGGLE_ONE:
+            return { ...state, checkedList: { ...state.checkedList, [action.payload]: !state.checkedList[action.payload] } };
         default:
             return state;
     }