Using the same terminology between edit and delete.

Signed-off-by: angatupyry <fierrofenix@gmail.com>
open-rmf · Sep 18, 2023 · c6f3e13 · c6f3e13
1 parent 9173dc1
commit c6f3e13
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 10 deletions.
diff --git a/packages/dashboard/src/components/tasks/task-schedule.tsx b/packages/dashboard/src/components/tasks/task-schedule.tsx
@@ -158,7 +158,6 @@ export const SchedulerTask = () => {
           currentValue={EventScopes.CURRENT}
           allValue={EventScopes.ALL}
           value={value}
-          deleting={false}
           onChange={onChange}
         />
       </ConfirmationDialog>
@@ -331,7 +330,6 @@ export const SchedulerTask = () => {
             currentValue={EventScopes.CURRENT}
             allValue={EventScopes.ALL}
             value={eventScope}
-            deleting={true}
             onChange={(event: React.ChangeEvent<HTMLInputElement>) =>
               setEventScope(event.target.value)
             }

diff --git a/packages/react-components/lib/tasks/task-scheduled-edit-delete-popup.tsx b/packages/react-components/lib/tasks/task-scheduled-edit-delete-popup.tsx
@@ -5,15 +5,13 @@ interface EventEditDeletePopupProps {
   currentValue: string;
   allValue: string;
   value: string;
-  deleting: boolean;
   onChange: (event: React.ChangeEvent<HTMLInputElement>) => void;
 }
 
 export function EventEditDeletePopup({
   currentValue,
   allValue,
   value,
-  deleting,
   onChange,
 }: EventEditDeletePopupProps): JSX.Element {
   return (
@@ -24,15 +22,11 @@ export function EventEditDeletePopup({
         value={value}
         onChange={onChange}
       >
-        <FormControlLabel
-          value={currentValue}
-          control={<Radio />}
-          label={`${deleting ? 'This event' : 'Edit this instance'}`}
-        />
+        <FormControlLabel value={currentValue} control={<Radio />} label={'This event'} />
         <FormControlLabel
           value={allValue}
           control={<Radio />}
-          label={`${deleting ? 'All events' : 'Edit series'}`}
+          label={'All events in this schedule'}
         />
       </RadioGroup>
     </FormControl>