Skip to content

MessageContainerView updates#94

Merged
SD10 merged 4 commits into
v0.7.0from
unknown repository
Sep 2, 2017
Merged

MessageContainerView updates#94
SD10 merged 4 commits into
v0.7.0from
unknown repository

Conversation

@ghost

@ghost ghost commented Sep 2, 2017

Copy link
Copy Markdown

MessageContainerView updates
Currently background color of the containerView is ignored, this needs to be fixed

@ghost ghost closed this Sep 2, 2017
@ghost ghost reopened this Sep 2, 2017
@ghost

ghost commented Sep 2, 2017

Copy link
Copy Markdown
Author

Please ignore this pull request as it is trying to merge with master!

@SD10

SD10 commented Sep 2, 2017

Copy link
Copy Markdown
Member

@oyalhi It's fine we can change the base branch to 0.7.0 later. I forgot to create a new branch

@ghost

ghost commented Sep 2, 2017

Copy link
Copy Markdown
Author

Ok, thanks.

@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #94 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #94   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           3      3           
  Lines          77     77           
=====================================
  Hits           77     77

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a4ddfa5...37b6adb. Read the comment docs.

@SD10 SD10 changed the base branch from master to v0.7.0 September 2, 2017 08:51

@SD10 SD10 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@oyalhi We'll probably end up using images instead of drawing the bubbles but thanks for giving this branch something to work with ❤️ 🎉

@SD10 SD10 merged commit f3040f6 into MessageKit:v0.7.0 Sep 2, 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.

3 participants