2023-12-28 06:46:28 +00:00
|
|
|
import React, { useEffect, useMemo, useRef, useState } from "react";
|
|
|
|
|
import styled from "styled-components";
|
|
|
|
|
|
|
|
|
|
// eslint-disable-next-line @typescript-eslint/ban-ts-comment
|
|
|
|
|
//@ts-ignore
|
|
|
|
|
import script from "!!raw-loader!./customWidgetscript.js";
|
|
|
|
|
|
|
|
|
|
// eslint-disable-next-line @typescript-eslint/ban-ts-comment
|
|
|
|
|
//@ts-ignore
|
|
|
|
|
import appsmithConsole from "!!raw-loader!./appsmithConsole.js";
|
|
|
|
|
|
|
|
|
|
// eslint-disable-next-line @typescript-eslint/ban-ts-comment
|
|
|
|
|
//@ts-ignore
|
|
|
|
|
import css from "!!raw-loader!./reset.css";
|
|
|
|
|
import clsx from "clsx";
|
|
|
|
|
import type { AppThemeProperties } from "entities/AppTheming";
|
chore: misc updates to custom widget (#30114)
#### PR fixes following issue(s)
Fixes https://github.com/appsmithorg/appsmith/issues/29991
Fixes https://github.com/appsmithorg/appsmith/issues/30154
Fixes https://github.com/appsmithorg/appsmith/issues/30020
Fixes https://github.com/appsmithorg/appsmith/issues/30019
Fixes https://github.com/appsmithorg/appsmith/issues/30130
Fixes https://github.com/appsmithorg/appsmith/issues/30159
Fixes https://github.com/appsmithorg/appsmith/issues/30223
#### Media
> A video or a GIF is preferred. when using Loom, don’t embed because it
looks like it’s a GIF. instead, just link to the video
>
>
#### Type of change
> Please delete options that are not relevant.
- Bug fix (non-breaking change which fixes an issue)
- New feature (non-breaking change which adds functionality)
- Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- Chore (housekeeping or task changes that don't impact user perception)
- This change requires a documentation update
>
>
>
## Testing
>
#### How Has This Been Tested?
> Please describe the tests that you ran to verify your changes. Also
list any relevant details for your test configuration.
> Delete anything that is not relevant
- [ ] Manual
- [ ] JUnit
- [ ] Jest
- [ ] Cypress
>
>
#### Test Plan
> Add Testsmith test cases links that relate to this PR
>
>
#### Issues raised during DP testing
> Link issues raised during DP testing for better visiblity and tracking
(copy link from comments dropped on this PR)
>
>
>
## Checklist:
#### Dev activity
- [ ] My code follows the style guidelines of this project
- [ ] I have performed a self-review of my own code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] PR is being merged under a feature flag
#### QA activity:
- [ ] [Speedbreak
features](https://github.com/appsmithorg/TestSmith/wiki/Guidelines-for-test-plans#speedbreakers-)
have been covered
- [ ] Test plan covers all impacted features and [areas of
interest](https://github.com/appsmithorg/TestSmith/wiki/Guidelines-for-test-plans#areas-of-interest-)
- [ ] Test plan has been peer reviewed by project stakeholders and other
QA members
- [ ] Manually tested functionality on DP
- [ ] We had an implementation alignment call with stakeholders post QA
Round 2
- [ ] Cypress test cases have been added and approved by SDET/manual QA
- [ ] Added `Test Plan Approved` label after Cypress tests were reviewed
- [ ] Added `Test Plan Approved` label after JUnit tests were reviewed
<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit
- **New Features**
- Custom widgets now support analytics events, enhancing visibility into
user interactions.
- Template selection, layout controls, and reference triggers in the
Custom Widget Builder are now integrated with analytics.
- Added new style options for custom widgets, including `primaryColor`,
`backgroundColor`, `borderRadius`, and `boxShadow`.
- **Bug Fixes**
- Corrected a typo in the constant title for better clarity.
- Updated help text for the Container Widget to accurately describe the
widget's border edge.
- **Enhancements**
- Improved user interface with additional styling for reference names in
the Custom Widget Builder.
- Enhanced debugger functionality with `useCallback` optimization and
new analytics logging.
- **Refactor**
- Streamlined the property pane by introducing a new `LabelContainer`
styled component.
- Refined the handling of dynamic binding paths to ignore certain
properties efficiently.
- **Documentation**
- Added a new constant for the default model documentation URL in the
Custom Widget Builder.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
2024-01-16 05:22:17 +00:00
|
|
|
import WidgetStyleContainer from "components/designSystems/appsmith/WidgetStyleContainer";
|
|
|
|
|
import type { BoxShadow } from "components/designSystems/appsmith/WidgetStyleContainer";
|
|
|
|
|
import type { Color } from "constants/Colors";
|
|
|
|
|
import { connect } from "react-redux";
|
|
|
|
|
import type { AppState } from "@appsmith/reducers";
|
|
|
|
|
import { combinedPreviewModeSelector } from "selectors/editorSelectors";
|
|
|
|
|
import { getAppMode } from "@appsmith/selectors/applicationSelectors";
|
|
|
|
|
import { APP_MODE } from "entities/App";
|
|
|
|
|
import { getWidgetPropsForPropertyPane } from "selectors/propertyPaneSelectors";
|
2023-12-28 06:46:28 +00:00
|
|
|
|
|
|
|
|
const StyledIframe = styled.iframe<{ width: number; height: number }>`
|
|
|
|
|
width: ${(props) => props.width - 8}px;
|
|
|
|
|
height: ${(props) => props.height - 8}px;
|
|
|
|
|
`;
|
|
|
|
|
|
|
|
|
|
const OverlayDiv = styled.div`
|
|
|
|
|
position: absolute;
|
|
|
|
|
top: 0;
|
|
|
|
|
left: 0;
|
|
|
|
|
width: 100%;
|
|
|
|
|
height: 100%;
|
|
|
|
|
`;
|
|
|
|
|
|
|
|
|
|
const EVENTS = {
|
|
|
|
|
CUSTOM_WIDGET_READY: "CUSTOM_WIDGET_READY",
|
|
|
|
|
CUSTOM_WIDGET_READY_ACK: "CUSTOM_WIDGET_READY_ACK",
|
|
|
|
|
CUSTOM_WIDGET_UPDATE_MODEL: "CUSTOM_WIDGET_UPDATE_MODEL",
|
|
|
|
|
CUSTOM_WIDGET_TRIGGER_EVENT: "CUSTOM_WIDGET_TRIGGER_EVENT",
|
|
|
|
|
CUSTOM_WIDGET_MODEL_CHANGE: "CUSTOM_WIDGET_MODEL_CHANGE",
|
|
|
|
|
CUSTOM_WIDGET_UI_CHANGE: "CUSTOM_WIDGET_UI_CHANGE",
|
|
|
|
|
CUSTOM_WIDGET_MESSAGE_RECEIVED_ACK: "CUSTOM_WIDGET_MESSAGE_RECEIVED_ACK",
|
|
|
|
|
CUSTOM_WIDGET_THEME_UPDATE: "CUSTOM_WIDGET_THEME_UPDATE",
|
|
|
|
|
};
|
|
|
|
|
|
|
|
|
|
// this is the padding set by the canvas
|
|
|
|
|
const WIDGET_PADDING = 8;
|
|
|
|
|
|
|
|
|
|
function CustomComponent(props: CustomComponentProps) {
|
|
|
|
|
const iframe = useRef<HTMLIFrameElement>(null);
|
|
|
|
|
|
|
|
|
|
const [loading, setLoading] = React.useState(true);
|
|
|
|
|
|
|
|
|
|
const [isIframeReady, setIsIframeReady] = useState(false);
|
|
|
|
|
|
|
|
|
|
const theme = useMemo(() => {
|
|
|
|
|
return {
|
|
|
|
|
...props.theme?.colors,
|
|
|
|
|
borderRadius: props.theme?.borderRadius?.appBorderRadius,
|
|
|
|
|
boxShadow: props.theme?.boxShadow?.appBoxShadow,
|
|
|
|
|
};
|
|
|
|
|
}, [props.theme]);
|
|
|
|
|
|
|
|
|
|
useEffect(() => {
|
|
|
|
|
const handler = (event: MessageEvent) => {
|
|
|
|
|
const iframeWindow =
|
|
|
|
|
iframe.current?.contentWindow ||
|
|
|
|
|
iframe.current?.contentDocument?.defaultView;
|
|
|
|
|
|
|
|
|
|
if (event.source === iframeWindow) {
|
|
|
|
|
// Sending acknowledgement for all messages since we're queueing all the postmessage from iframe
|
|
|
|
|
iframe.current?.contentWindow?.postMessage(
|
|
|
|
|
{
|
|
|
|
|
type: EVENTS.CUSTOM_WIDGET_MESSAGE_RECEIVED_ACK,
|
|
|
|
|
key: event.data.key,
|
|
|
|
|
success: true,
|
|
|
|
|
},
|
|
|
|
|
"*",
|
|
|
|
|
);
|
|
|
|
|
|
|
|
|
|
const message = event.data;
|
|
|
|
|
|
|
|
|
|
switch (message.type) {
|
|
|
|
|
case EVENTS.CUSTOM_WIDGET_READY:
|
|
|
|
|
setIsIframeReady(true);
|
|
|
|
|
iframe.current?.contentWindow?.postMessage(
|
|
|
|
|
{
|
|
|
|
|
type: EVENTS.CUSTOM_WIDGET_READY_ACK,
|
|
|
|
|
model: props.model,
|
|
|
|
|
ui: {
|
|
|
|
|
width: props.width - WIDGET_PADDING,
|
|
|
|
|
height: props.height - WIDGET_PADDING,
|
|
|
|
|
},
|
|
|
|
|
mode: props.renderMode,
|
|
|
|
|
theme,
|
|
|
|
|
},
|
|
|
|
|
"*",
|
|
|
|
|
);
|
|
|
|
|
break;
|
|
|
|
|
case EVENTS.CUSTOM_WIDGET_UPDATE_MODEL:
|
|
|
|
|
props.update(message.data);
|
|
|
|
|
break;
|
|
|
|
|
case EVENTS.CUSTOM_WIDGET_TRIGGER_EVENT:
|
|
|
|
|
props.execute(message.data.eventName, message.data.contextObj);
|
|
|
|
|
break;
|
|
|
|
|
case "UPDATE_HEIGHT":
|
|
|
|
|
const height = message.data.height;
|
|
|
|
|
|
|
|
|
|
if (height) {
|
|
|
|
|
iframe.current!.style.height = `${height}px`;
|
|
|
|
|
}
|
|
|
|
|
break;
|
|
|
|
|
case "CUSTOM_WIDGET_CONSOLE_EVENT":
|
|
|
|
|
props.onConsole &&
|
|
|
|
|
props.onConsole(message.data.type, message.data.args);
|
|
|
|
|
break;
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
};
|
|
|
|
|
|
|
|
|
|
window.addEventListener("message", handler, false);
|
|
|
|
|
|
|
|
|
|
return () => window.removeEventListener("message", handler, false);
|
|
|
|
|
}, [props.model, props.width, props.height]);
|
|
|
|
|
|
|
|
|
|
useEffect(() => {
|
|
|
|
|
if (iframe.current && iframe.current.contentWindow && isIframeReady) {
|
|
|
|
|
iframe.current.contentWindow.postMessage(
|
|
|
|
|
{
|
|
|
|
|
type: EVENTS.CUSTOM_WIDGET_MODEL_CHANGE,
|
|
|
|
|
model: props.model,
|
|
|
|
|
},
|
|
|
|
|
"*",
|
|
|
|
|
);
|
|
|
|
|
}
|
|
|
|
|
}, [props.model]);
|
|
|
|
|
|
|
|
|
|
useEffect(() => {
|
|
|
|
|
if (iframe.current && iframe.current.contentWindow && isIframeReady) {
|
|
|
|
|
iframe.current.contentWindow.postMessage(
|
|
|
|
|
{
|
|
|
|
|
type: EVENTS.CUSTOM_WIDGET_UI_CHANGE,
|
|
|
|
|
ui: {
|
|
|
|
|
width: props.width - 8,
|
|
|
|
|
height: props.height - 8,
|
|
|
|
|
},
|
|
|
|
|
},
|
|
|
|
|
"*",
|
|
|
|
|
);
|
|
|
|
|
}
|
|
|
|
|
}, [props.width, props.height]);
|
|
|
|
|
|
|
|
|
|
useEffect(() => {
|
|
|
|
|
if (iframe.current && iframe.current.contentWindow && isIframeReady) {
|
|
|
|
|
iframe.current.contentWindow.postMessage(
|
|
|
|
|
{
|
|
|
|
|
type: EVENTS.CUSTOM_WIDGET_THEME_UPDATE,
|
|
|
|
|
theme,
|
|
|
|
|
},
|
|
|
|
|
"*",
|
|
|
|
|
);
|
|
|
|
|
}
|
|
|
|
|
}, [theme]);
|
|
|
|
|
|
|
|
|
|
const srcDoc = `
|
|
|
|
|
<html>
|
|
|
|
|
<head>
|
|
|
|
|
<style>${css}</style>
|
|
|
|
|
</head>
|
|
|
|
|
<body>
|
|
|
|
|
<script type="text/javascript">${
|
|
|
|
|
props.onConsole ? appsmithConsole : ""
|
|
|
|
|
}</script>
|
|
|
|
|
<script type="module">
|
|
|
|
|
${script}
|
|
|
|
|
main();
|
|
|
|
|
</script>
|
|
|
|
|
${props.srcDoc.html}
|
|
|
|
|
<script type="module">
|
|
|
|
|
${props.srcDoc.js}
|
|
|
|
|
</script>
|
|
|
|
|
<style>${props.srcDoc.css}</style>
|
|
|
|
|
</body>
|
|
|
|
|
</html>
|
|
|
|
|
`;
|
|
|
|
|
|
|
|
|
|
useEffect(() => {
|
|
|
|
|
setLoading(true);
|
|
|
|
|
}, [srcDoc]);
|
|
|
|
|
|
|
|
|
|
return (
|
|
|
|
|
<div
|
|
|
|
|
className={clsx({
|
|
|
|
|
"bp3-skeleton": loading,
|
|
|
|
|
})}
|
|
|
|
|
>
|
|
|
|
|
{props.needsOverlay && <OverlayDiv data-testid="iframe-overlay" />}
|
chore: misc updates to custom widget (#30114)
#### PR fixes following issue(s)
Fixes https://github.com/appsmithorg/appsmith/issues/29991
Fixes https://github.com/appsmithorg/appsmith/issues/30154
Fixes https://github.com/appsmithorg/appsmith/issues/30020
Fixes https://github.com/appsmithorg/appsmith/issues/30019
Fixes https://github.com/appsmithorg/appsmith/issues/30130
Fixes https://github.com/appsmithorg/appsmith/issues/30159
Fixes https://github.com/appsmithorg/appsmith/issues/30223
#### Media
> A video or a GIF is preferred. when using Loom, don’t embed because it
looks like it’s a GIF. instead, just link to the video
>
>
#### Type of change
> Please delete options that are not relevant.
- Bug fix (non-breaking change which fixes an issue)
- New feature (non-breaking change which adds functionality)
- Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- Chore (housekeeping or task changes that don't impact user perception)
- This change requires a documentation update
>
>
>
## Testing
>
#### How Has This Been Tested?
> Please describe the tests that you ran to verify your changes. Also
list any relevant details for your test configuration.
> Delete anything that is not relevant
- [ ] Manual
- [ ] JUnit
- [ ] Jest
- [ ] Cypress
>
>
#### Test Plan
> Add Testsmith test cases links that relate to this PR
>
>
#### Issues raised during DP testing
> Link issues raised during DP testing for better visiblity and tracking
(copy link from comments dropped on this PR)
>
>
>
## Checklist:
#### Dev activity
- [ ] My code follows the style guidelines of this project
- [ ] I have performed a self-review of my own code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] PR is being merged under a feature flag
#### QA activity:
- [ ] [Speedbreak
features](https://github.com/appsmithorg/TestSmith/wiki/Guidelines-for-test-plans#speedbreakers-)
have been covered
- [ ] Test plan covers all impacted features and [areas of
interest](https://github.com/appsmithorg/TestSmith/wiki/Guidelines-for-test-plans#areas-of-interest-)
- [ ] Test plan has been peer reviewed by project stakeholders and other
QA members
- [ ] Manually tested functionality on DP
- [ ] We had an implementation alignment call with stakeholders post QA
Round 2
- [ ] Cypress test cases have been added and approved by SDET/manual QA
- [ ] Added `Test Plan Approved` label after Cypress tests were reviewed
- [ ] Added `Test Plan Approved` label after JUnit tests were reviewed
<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit
- **New Features**
- Custom widgets now support analytics events, enhancing visibility into
user interactions.
- Template selection, layout controls, and reference triggers in the
Custom Widget Builder are now integrated with analytics.
- Added new style options for custom widgets, including `primaryColor`,
`backgroundColor`, `borderRadius`, and `boxShadow`.
- **Bug Fixes**
- Corrected a typo in the constant title for better clarity.
- Updated help text for the Container Widget to accurately describe the
widget's border edge.
- **Enhancements**
- Improved user interface with additional styling for reference names in
the Custom Widget Builder.
- Enhanced debugger functionality with `useCallback` optimization and
new analytics logging.
- **Refactor**
- Streamlined the property pane by introducing a new `LabelContainer`
styled component.
- Refined the handling of dynamic binding paths to ignore certain
properties efficiently.
- **Documentation**
- Added a new constant for the default model documentation URL in the
Custom Widget Builder.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
2024-01-16 05:22:17 +00:00
|
|
|
<WidgetStyleContainer
|
|
|
|
|
backgroundColor={props.backgroundColor}
|
|
|
|
|
borderColor={props.borderColor}
|
|
|
|
|
borderRadius={props.borderRadius}
|
|
|
|
|
borderWidth={props.borderWidth}
|
|
|
|
|
boxShadow={props.boxShadow}
|
|
|
|
|
widgetId={props.widgetId}
|
|
|
|
|
>
|
|
|
|
|
<StyledIframe
|
|
|
|
|
height={props.height}
|
|
|
|
|
loading="lazy"
|
|
|
|
|
onLoad={() => {
|
|
|
|
|
setLoading(false);
|
|
|
|
|
}}
|
|
|
|
|
ref={iframe}
|
|
|
|
|
sandbox="allow-scripts allow-downloads"
|
|
|
|
|
srcDoc={srcDoc}
|
|
|
|
|
width={props.width}
|
|
|
|
|
/>
|
|
|
|
|
</WidgetStyleContainer>
|
2023-12-28 06:46:28 +00:00
|
|
|
</div>
|
|
|
|
|
);
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
export interface CustomComponentProps {
|
|
|
|
|
execute: (eventName: string, contextObj: Record<string, unknown>) => void;
|
|
|
|
|
update: (data: Record<string, unknown>) => void;
|
|
|
|
|
model: Record<string, unknown>;
|
|
|
|
|
srcDoc: {
|
|
|
|
|
html: string;
|
|
|
|
|
js: string;
|
|
|
|
|
css: string;
|
|
|
|
|
};
|
|
|
|
|
width: number;
|
|
|
|
|
height: number;
|
|
|
|
|
onLoadingStateChange?: (state: string) => void;
|
|
|
|
|
needsOverlay?: boolean;
|
|
|
|
|
onConsole?: (type: string, message: string) => void;
|
|
|
|
|
renderMode: "EDITOR" | "DEPLOYED" | "BUILDER";
|
|
|
|
|
theme: AppThemeProperties;
|
chore: misc updates to custom widget (#30114)
#### PR fixes following issue(s)
Fixes https://github.com/appsmithorg/appsmith/issues/29991
Fixes https://github.com/appsmithorg/appsmith/issues/30154
Fixes https://github.com/appsmithorg/appsmith/issues/30020
Fixes https://github.com/appsmithorg/appsmith/issues/30019
Fixes https://github.com/appsmithorg/appsmith/issues/30130
Fixes https://github.com/appsmithorg/appsmith/issues/30159
Fixes https://github.com/appsmithorg/appsmith/issues/30223
#### Media
> A video or a GIF is preferred. when using Loom, don’t embed because it
looks like it’s a GIF. instead, just link to the video
>
>
#### Type of change
> Please delete options that are not relevant.
- Bug fix (non-breaking change which fixes an issue)
- New feature (non-breaking change which adds functionality)
- Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- Chore (housekeeping or task changes that don't impact user perception)
- This change requires a documentation update
>
>
>
## Testing
>
#### How Has This Been Tested?
> Please describe the tests that you ran to verify your changes. Also
list any relevant details for your test configuration.
> Delete anything that is not relevant
- [ ] Manual
- [ ] JUnit
- [ ] Jest
- [ ] Cypress
>
>
#### Test Plan
> Add Testsmith test cases links that relate to this PR
>
>
#### Issues raised during DP testing
> Link issues raised during DP testing for better visiblity and tracking
(copy link from comments dropped on this PR)
>
>
>
## Checklist:
#### Dev activity
- [ ] My code follows the style guidelines of this project
- [ ] I have performed a self-review of my own code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] PR is being merged under a feature flag
#### QA activity:
- [ ] [Speedbreak
features](https://github.com/appsmithorg/TestSmith/wiki/Guidelines-for-test-plans#speedbreakers-)
have been covered
- [ ] Test plan covers all impacted features and [areas of
interest](https://github.com/appsmithorg/TestSmith/wiki/Guidelines-for-test-plans#areas-of-interest-)
- [ ] Test plan has been peer reviewed by project stakeholders and other
QA members
- [ ] Manually tested functionality on DP
- [ ] We had an implementation alignment call with stakeholders post QA
Round 2
- [ ] Cypress test cases have been added and approved by SDET/manual QA
- [ ] Added `Test Plan Approved` label after Cypress tests were reviewed
- [ ] Added `Test Plan Approved` label after JUnit tests were reviewed
<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit
- **New Features**
- Custom widgets now support analytics events, enhancing visibility into
user interactions.
- Template selection, layout controls, and reference triggers in the
Custom Widget Builder are now integrated with analytics.
- Added new style options for custom widgets, including `primaryColor`,
`backgroundColor`, `borderRadius`, and `boxShadow`.
- **Bug Fixes**
- Corrected a typo in the constant title for better clarity.
- Updated help text for the Container Widget to accurately describe the
widget's border edge.
- **Enhancements**
- Improved user interface with additional styling for reference names in
the Custom Widget Builder.
- Enhanced debugger functionality with `useCallback` optimization and
new analytics logging.
- **Refactor**
- Streamlined the property pane by introducing a new `LabelContainer`
styled component.
- Refined the handling of dynamic binding paths to ignore certain
properties efficiently.
- **Documentation**
- Added a new constant for the default model documentation URL in the
Custom Widget Builder.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
2024-01-16 05:22:17 +00:00
|
|
|
borderColor?: Color;
|
|
|
|
|
backgroundColor?: Color;
|
|
|
|
|
borderWidth?: number;
|
|
|
|
|
borderRadius?: number;
|
|
|
|
|
boxShadow?: BoxShadow;
|
|
|
|
|
widgetId: string;
|
2023-12-28 06:46:28 +00:00
|
|
|
}
|
|
|
|
|
|
chore: misc updates to custom widget (#30114)
#### PR fixes following issue(s)
Fixes https://github.com/appsmithorg/appsmith/issues/29991
Fixes https://github.com/appsmithorg/appsmith/issues/30154
Fixes https://github.com/appsmithorg/appsmith/issues/30020
Fixes https://github.com/appsmithorg/appsmith/issues/30019
Fixes https://github.com/appsmithorg/appsmith/issues/30130
Fixes https://github.com/appsmithorg/appsmith/issues/30159
Fixes https://github.com/appsmithorg/appsmith/issues/30223
#### Media
> A video or a GIF is preferred. when using Loom, don’t embed because it
looks like it’s a GIF. instead, just link to the video
>
>
#### Type of change
> Please delete options that are not relevant.
- Bug fix (non-breaking change which fixes an issue)
- New feature (non-breaking change which adds functionality)
- Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- Chore (housekeeping or task changes that don't impact user perception)
- This change requires a documentation update
>
>
>
## Testing
>
#### How Has This Been Tested?
> Please describe the tests that you ran to verify your changes. Also
list any relevant details for your test configuration.
> Delete anything that is not relevant
- [ ] Manual
- [ ] JUnit
- [ ] Jest
- [ ] Cypress
>
>
#### Test Plan
> Add Testsmith test cases links that relate to this PR
>
>
#### Issues raised during DP testing
> Link issues raised during DP testing for better visiblity and tracking
(copy link from comments dropped on this PR)
>
>
>
## Checklist:
#### Dev activity
- [ ] My code follows the style guidelines of this project
- [ ] I have performed a self-review of my own code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] PR is being merged under a feature flag
#### QA activity:
- [ ] [Speedbreak
features](https://github.com/appsmithorg/TestSmith/wiki/Guidelines-for-test-plans#speedbreakers-)
have been covered
- [ ] Test plan covers all impacted features and [areas of
interest](https://github.com/appsmithorg/TestSmith/wiki/Guidelines-for-test-plans#areas-of-interest-)
- [ ] Test plan has been peer reviewed by project stakeholders and other
QA members
- [ ] Manually tested functionality on DP
- [ ] We had an implementation alignment call with stakeholders post QA
Round 2
- [ ] Cypress test cases have been added and approved by SDET/manual QA
- [ ] Added `Test Plan Approved` label after Cypress tests were reviewed
- [ ] Added `Test Plan Approved` label after JUnit tests were reviewed
<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit
- **New Features**
- Custom widgets now support analytics events, enhancing visibility into
user interactions.
- Template selection, layout controls, and reference triggers in the
Custom Widget Builder are now integrated with analytics.
- Added new style options for custom widgets, including `primaryColor`,
`backgroundColor`, `borderRadius`, and `boxShadow`.
- **Bug Fixes**
- Corrected a typo in the constant title for better clarity.
- Updated help text for the Container Widget to accurately describe the
widget's border edge.
- **Enhancements**
- Improved user interface with additional styling for reference names in
the Custom Widget Builder.
- Enhanced debugger functionality with `useCallback` optimization and
new analytics logging.
- **Refactor**
- Streamlined the property pane by introducing a new `LabelContainer`
styled component.
- Refined the handling of dynamic binding paths to ignore certain
properties efficiently.
- **Documentation**
- Added a new constant for the default model documentation URL in the
Custom Widget Builder.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
2024-01-16 05:22:17 +00:00
|
|
|
/**
|
|
|
|
|
* TODO: Balaji soundararajan - to refactor code to move out selected widget details to platform
|
|
|
|
|
*/
|
|
|
|
|
export const mapStateToProps = (
|
|
|
|
|
state: AppState,
|
|
|
|
|
ownProps: CustomComponentProps,
|
|
|
|
|
) => {
|
|
|
|
|
const isPreviewMode = combinedPreviewModeSelector(state);
|
|
|
|
|
const appMode = getAppMode(state);
|
|
|
|
|
|
|
|
|
|
return {
|
|
|
|
|
needsOverlay:
|
|
|
|
|
appMode == APP_MODE.EDIT &&
|
|
|
|
|
!isPreviewMode &&
|
|
|
|
|
ownProps.widgetId !== getWidgetPropsForPropertyPane(state)?.widgetId,
|
|
|
|
|
};
|
|
|
|
|
};
|
|
|
|
|
|
|
|
|
|
export default connect(mapStateToProps)(CustomComponent);
|