-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Initially tried working with the MessageKit framework but I couldn't … #2
base: master
Are you sure you want to change the base?
Conversation
…quite understand it. Borrowed some code from Kavsoft for adding a dynamically resizing message view.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you think it makes sense to finish chat view and link it a data source before merging to master?
UIResponder.keyboardDidShowNotification, object: nil, queue: .main) { | ||
(data) in | ||
|
||
let height1 = data.userInfo![UIResponder.keyboardFrameEndUserInfoKey] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[critical] forced unwrapping is not good, can cause a crash
struct ChatView_Previews: PreviewProvider { | ||
static var previews: some View { | ||
ChatView() | ||
} | ||
} | ||
|
||
|
||
struct chatInstance : View { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nitpick] let's follow Swift standards and name structs with the capital letter
@@ -479,6 +491,11 @@ | |||
package = 0DEE520725D0BA71003C6EB7 /* XCRemoteSwiftPackageReference "swift-protobuf" */; | |||
productName = SwiftProtobuf; | |||
}; | |||
8A713CCF2623EAD300F6D4BB /* MessageKit */ = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If MessageKit is not used can you please remove it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the proper way to do this?
Yeah, can do. I definitely need to get a better understanding on how the messages interface with the devices. |
Initially tried working with the MessageKit framework but I couldn't quite understand it. Borrowed some code from Kavsoft for adding a dynamically resizing message view.