Skip to content
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

Last message is completely visible in chat #458

Closed
wants to merge 1 commit into from

Conversation

neeraj3029
Copy link
Member

Fixes #457

Changes: Changed the height of CGSize from 38 to 60. Now, the last message in chat can be seen completely

Screenshots for the change:
GIF -
expected

.

@neeraj3029
Copy link
Member Author

@jogendra Please review! 😃

@@ -113,7 +113,7 @@ extension ChatViewController: UICollectionViewDelegateFlowLayout {
} else if message.actionType == ActionType.video_play.rawValue || message.actionType == ActionType.audio_play.rawValue {
return CGSize(width: view.frame.width, height: 158)
}
return CGSize(width: view.frame.width, height: estimatedFrame.height + 38)
return CGSize(width: view.frame.width, height: estimatedFrame.height + 60)
Copy link
Member

Choose a reason for hiding this comment

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

Set edge inset instead of changing the height of the cell.

Copy link
Member Author

Choose a reason for hiding this comment

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

I tried various combinations of edge insets but none of them seemed to work out. It would be a great help if you could suggest any value set.

@neeraj3029 neeraj3029 closed this Dec 13, 2018
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.

2 participants