Skip to content

[WC-3297] Signature web#2138

Open
gjulivan wants to merge 8 commits intomainfrom
signature-web
Open

[WC-3297] Signature web#2138
gjulivan wants to merge 8 commits intomainfrom
signature-web

Conversation

@gjulivan
Copy link
Copy Markdown
Collaborator

Pull request type


Description

@gjulivan gjulivan requested a review from a team as a code owner March 16, 2026 13:14
@gjulivan gjulivan changed the title Signature web [WC-3297] Signature web Mar 16, 2026
Comment thread packages/pluggableWidgets/signature-web/src/__tests__/AppEvents.spec.tsx Outdated
Comment thread packages/pluggableWidgets/signature-web/src/components/Alert.tsx Outdated
Comment thread packages/pluggableWidgets/signature-web/src/ui/SignaturePreview.css Outdated
Comment thread packages/pluggableWidgets/signature-web/src/assets/Signature.icon.dark.png Outdated
Comment thread packages/pluggableWidgets/signature-web/src/package.xml Outdated
Comment thread packages/pluggableWidgets/signature-web/src/Signature.editorConfig.ts Outdated
Comment thread packages/pluggableWidgets/signature-web/src/Signature.tsx Outdated
Comment thread packages/pluggableWidgets/signature-web/CHANGELOG.md
Comment thread packages/pluggableWidgets/signature-web/package.json Outdated
Comment thread packages/pluggableWidgets/signature-web/package.json
Comment thread packages/pluggableWidgets/signature-web/package.json Outdated
@gjulivan gjulivan force-pushed the signature-web branch 8 times, most recently from 1edd154 to c439717 Compare March 26, 2026 08:45
Comment thread packages/pluggableWidgets/signature-web/README.md Outdated
Comment thread packages/pluggableWidgets/signature-web/package.json Outdated
Comment thread packages/pluggableWidgets/signature-web/package.json Outdated
Comment thread packages/pluggableWidgets/signature-web/mendix-pluggable-widgets-tools.tgz Outdated
Comment thread packages/pluggableWidgets/signature-web/src/Signature.xml Outdated
Comment thread packages/pluggableWidgets/signature-web/src/utils/customTypes.ts
Comment thread packages/pluggableWidgets/signature-web/src/utils/useResizeObserver.ts Outdated
Comment thread packages/pluggableWidgets/signature-web/src/utils/useSignaturePad.ts Outdated
Comment thread packages/pluggableWidgets/signature-web/src/utils/Utils.ts
Comment thread packages/pluggableWidgets/signature-web/src/Signature.tsx Outdated
@gjulivan gjulivan force-pushed the signature-web branch 7 times, most recently from f4f70c2 to bafb254 Compare March 27, 2026 11:57
Comment thread packages/pluggableWidgets/signature-web/src/components/SizeContainer.ts Outdated
Comment on lines +19 to +43
return (
<div
ref={ref}
className={classNames(className, "size-box")}
style={{
position: "relative",
...wrapperStyle,
...style
}}
>
<div
className={classNames("size-box-inner", classNameInner)}
aria-disabled={readOnly}
style={{
position: "absolute",
top: 0,
right: 0,
bottom: 0,
left: 0
}}
>
{children}
</div>
</div>
);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's move all inline styles to sass files.

import SignaturePad, { Options } from "signature_pad";
import { SignatureProps } from "./customTypes";

function usePrevious<T>(value: T): T | null {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't understand why we need this hook. Why we need keep attribute in ref and use effect to update it?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is duplicate of tools we have in 'platform' plugin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants