fix: Trigger Save layout only on title (#2039)

* fix: Trigger Save layout only on title

* chore: code improvement

* fix: Lint issues resolved

Co-authored-by: Palash Gupta <palashgdev@gmail.com>
This commit is contained in:
Chintan Sudani 2023-01-13 17:29:51 +05:30 committed by GitHub
parent a82bbe1a72
commit d9535e7a8d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 27 additions and 14 deletions

View File

@ -213,19 +213,29 @@ function GridCardGraph({
); );
}; };
const handleOnView = (): void => {
onToggleModal(setModal);
};
const handleOnDelete = (): void => {
onToggleModal(setDeleteModal);
};
const isEmptyLayout = widget?.id === 'empty' || isEmpty(widget); const isEmptyLayout = widget?.id === 'empty' || isEmpty(widget);
if (state.error && !isEmptyLayout) { if (state.error && !isEmptyLayout) {
return ( return (
<> <>
{getModals()} {getModals()}
<div className="drag-handle">
<WidgetHeader <WidgetHeader
parentHover={hovered} parentHover={hovered}
title={widget?.title} title={widget?.title}
widget={widget} widget={widget}
onView={(): void => onToggleModal(setModal)} onView={handleOnView}
onDelete={(): void => onToggleModal(setDeleteModal)} onDelete={handleOnDelete}
/> />
</div>
<ErrorContainer>{state.errorMessage}</ErrorContainer> <ErrorContainer>{state.errorMessage}</ErrorContainer>
</> </>
@ -255,6 +265,7 @@ function GridCardGraph({
}} }}
> >
{!isEmptyLayout && ( {!isEmptyLayout && (
<div className="drag-handle">
<WidgetHeader <WidgetHeader
parentHover={hovered} parentHover={hovered}
title={widget?.title} title={widget?.title}
@ -262,6 +273,7 @@ function GridCardGraph({
onView={(): void => onToggleModal(setModal)} onView={(): void => onToggleModal(setModal)}
onDelete={(): void => onToggleModal(setDeleteModal)} onDelete={(): void => onToggleModal(setDeleteModal)}
/> />
</div>
)} )}
{!isEmptyLayout && getModals()} {!isEmptyLayout && getModals()}

View File

@ -72,6 +72,7 @@ function GraphLayout({
useCSSTransforms useCSSTransforms
allowOverlap={false} allowOverlap={false}
onLayoutChange={onLayoutChangeHandler} onLayoutChange={onLayoutChangeHandler}
draggableHandle=".drag-handle"
> >
{layouts.map(({ Component, ...rest }) => { {layouts.map(({ Component, ...rest }) => {
const currentWidget = (widgets || [])?.find((e) => e.id === rest.i); const currentWidget = (widgets || [])?.find((e) => e.id === rest.i);