Skip to content

Add SwiftUI view#73

Open
symes-s wants to merge 3 commits intoalankarmisra:masterfrom
symes-s:SwiftUI
Open

Add SwiftUI view#73
symes-s wants to merge 3 commits intoalankarmisra:masterfrom
symes-s:SwiftUI

Conversation

@symes-s
Copy link
Copy Markdown

@symes-s symes-s commented Aug 9, 2025

Added a SwiftUI wrapper for the UIView

See the preview for example usage

public func canvasViewDidEndUsingTool(_ canvasView: PKCanvasView) {

public func canvasViewDrawingDidChange(_ canvasView: PKCanvasView) {
Copy link
Copy Markdown
Author

@symes-s symes-s Aug 11, 2025

Choose a reason for hiding this comment

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

This change was necessary to get the signature output to update correctly. If this is going to break things, an extra method could be added to the SwiftSignatureViewDelegate instead.

@alankarmisra alankarmisra self-requested a review October 21, 2025 08:37
@lchamp
Copy link
Copy Markdown

lchamp commented Nov 5, 2025

@alankarmisra are you planning to review and merge soon?

Could you also fix the error caused by #75 (comment) and mentioned here: #76.
Indded the package is still targeting iOS12 and as such this gives the following error:
Capture d’écran 2025-11-05 à 14 35 38

@lchamp
Copy link
Copy Markdown

lchamp commented Nov 6, 2025

Could you also fix the error caused by #75 (comment) and mentioned here: #76. Inde ed the package is still targeting iOS12 and as such this gives the following error:

PR #77 should solve this issue and changes the version to 3.2.6

With the current PR #73 of symes-s you could add SwiftUI support (could change version to 3.3.0).

@alankarmisra
Copy link
Copy Markdown
Owner

alankarmisra commented Nov 16, 2025

@symes-s I'll look into this for the next release. Been a bit overwhelmed with regular life :/

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants