Skip to content

IPad drawing - #140

Merged
akaDuality merged 26 commits into
mainfrom
iPadDrawing
Dec 24, 2022
Merged

akaDuality merged 26 commits into
mainfrom
iPadDrawing

Conversation

@akaDuality

@akaDuality akaDuality commented Nov 19, 2022 •

Copy link
Copy Markdown
Member
  • Draw only by Apple Pencil. Otherwise you can't scroll long document by finger
  • Scale to fit height, not width
  • Move files to packages at Preview

@akaDuality
akaDuality marked this pull request as draft November 19, 2022 11:06
@akaDuality akaDuality linked an issue Dec 3, 2022 that may be closed by this pull request
2 of 5 tasks
…dDrawing

# Conflicts:
#	Shared/Sources/Canvas/CanvasPresenter.swift
#	Shared/Sources/Document/Documents/VODesignDocument+UIKit.swift
#	Shared/Sources/Document/Documents/VODesignDocumentProtocol.swift
#	VoiceOver Preview/PreviewFeatures/Sources/DesignPreview/Canvas/VODesignPreviewView.swift
#	VoiceOver Preview/PreviewFeatures/Sources/DesignPreview/Canvas/VoiceOverElement.swift
#	VoiceOver Preview/PreviewFeatures/Sources/DesignPreview/Canvas/VoiceOverLayout.swift
#	VoiceOver Preview/PreviewFeatures/Sources/DesignPreview/Layout/VoiceOverElement.swift
#	VoiceOver Preview/PreviewFeatures/Sources/DesignPreview/VoiceOverElement.swift
@drucelweisse
drucelweisse marked this pull request as ready for review December 14, 2022 18:21
#endif

public class A11yDescription: Codable, Equatable {
public class A11yDescription: Codable, Equatable, ObservableObject {

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

We should split A11yDescription to A11yDescription and A11yDescriptionDto. Dto will be data transfer object for saving to file, but regular A11yDescription will be used for rendering

Comment thread VoiceOver Designer.xcodeproj/project.pbxproj Outdated
@akaDuality
akaDuality merged commit 607649d into main Dec 24, 2022
@akaDuality
akaDuality deleted the iPadDrawing branch December 24, 2022 09:41
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.

iPad version

2 participants