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

Fix bottom label origin bugs #326

Merged
merged 2 commits into from
Nov 5, 2017
Merged

Fix bottom label origin bugs #326

merged 2 commits into from
Nov 5, 2017

Conversation

SD10
Copy link
Member

@SD10 SD10 commented Nov 4, 2017

I think this could resolve #231.

The problem was simpler than when I was trying to diagnose it from the UI. Nonetheless, this is an improvement if I've over-simplified it and missed something.

I also caught a problem with the bottom label origin for the .messageLeading case and resolved that as well. Not sure how that regressed. Please let me know if it was a problem for anyone.

TODO:

  • CHANGELOG entry

@zhongwuzw zhongwuzw requested review from FraDeliro and removed request for FraDeliro November 4, 2017 09:04
@SD10 SD10 merged commit 64e26b4 into master Nov 5, 2017
@SD10 SD10 deleted the bugfix/cell-origins branch November 5, 2017 01:51
@SD10 SD10 mentioned this pull request Nov 12, 2017
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.

MessageContainerView origin needs to consider AvatarView size
1 participant