Skip to content
This repository was archived by the owner on Dec 23, 2024. It is now read-only.

stan's version 1 approach#1

Open
stanchiang wants to merge 5 commits intoiOS-Connect:masterfrom
stanchiang:master
Open

stan's version 1 approach#1
stanchiang wants to merge 5 commits intoiOS-Connect:masterfrom
stanchiang:master

Conversation

@stanchiang
Copy link

opening a pull request to receive feedback on my implementation of the drag and drop project.


class DraggableView: UIView {

var panGestureRecognizer:UIPanGestureRecognizer!
Copy link
Member

Choose a reason for hiding this comment

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

Would love to see this not be an ImplicitlyUnwrappedOptional 😄

For this one specifically we can just initialize it directly.
let panGestureRecognizer = UIPanGestureRecognizer()
Then remove the declaration from init()

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants