Skip to content

Commit

Permalink
update timeline modal
Browse files Browse the repository at this point in the history
  • Loading branch information
mkumbobeaty committed Nov 5, 2024
1 parent ece2f09 commit 85fa1d0
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 11 deletions.
3 changes: 0 additions & 3 deletions web/src/beta/ui/fields/TimePeriodField/EditModal.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -15,13 +15,11 @@ type EditPanelProps = {
timePeriodValues?: TimePeriodFieldProp;
onClose?: () => void;
onChange?: (value?: TimePeriodFieldProp) => void;
setTimePeriodValues?: (value?: TimePeriodFieldProp) => void;
};

const EditModal: FC<EditPanelProps> = ({
visible,
timePeriodValues,
setTimePeriodValues,
onClose,
onChange
}) => {
Expand All @@ -38,7 +36,6 @@ const EditModal: FC<EditPanelProps> = ({
timePeriodValues,
onChange,
onClose,
setTimePeriodValues
});

const timezoneMatches = useMemo(() => {
Expand Down
6 changes: 2 additions & 4 deletions web/src/beta/ui/fields/TimePeriodField/hooks.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,14 +6,12 @@ type Props = {
timePeriodValues?: TimePeriodFieldProp;
onChange?: (value?: TimePeriodFieldProp) => void;
onClose?: () => void;
setTimePeriodValues?: (value?: TimePeriodFieldProp) => void;
};

export default ({
timePeriodValues,
onChange,
onClose,
setTimePeriodValues
}: Props) => {
const [warning, setWarning] = useState(false);

Expand Down Expand Up @@ -59,9 +57,9 @@ export default ({
default:
break;
}
setTimePeriodValues?.(updatedData);
onChange?.(updatedData);
},
[timePeriodValues, setTimePeriodValues]
[timePeriodValues, onChange]
);

const [disabledFields, setDisabledFields] = useState<string[]>([]);
Expand Down
6 changes: 2 additions & 4 deletions web/src/beta/ui/fields/TimePeriodField/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -33,15 +33,14 @@ const TimePeriodField: FC<TimePeriodFieldProps> = ({
const handleEditorModalClose = useCallback(() => {
setOpenEditModal(false);
if (!value) {
setTimePeriodValues(undefined);
onChange?.(undefined);
}
}, [value]);
}, [onChange, value]);

const handleEditorModalOpen = useCallback(() => setOpenEditModal(true), []);

const handleTimePeriodSettingDelete = useCallback(() => {
if (!value) return;
setTimePeriodValues(undefined);
onChange?.();
}, [value, onChange]);

Expand Down Expand Up @@ -114,7 +113,6 @@ const TimePeriodField: FC<TimePeriodFieldProps> = ({
onChange={onChange}
onClose={handleEditorModalClose}
visible={openEditModal}
setTimePeriodValues={setTimePeriodValues}
/>
)}
</CommonField>
Expand Down

0 comments on commit 85fa1d0

Please sign in to comment.