Skip to content

Fix UITextView content size ambiguous issue - #310

Merged
nathantannar4 merged 3 commits into
MessageKit:v0.11.0from
zhongwuzw:master
Nov 1, 2017
Merged

nathantannar4 merged 3 commits into
MessageKit:v0.11.0from
zhongwuzw:master

Conversation

@zhongwuzw

Copy link
Copy Markdown
Member

No description provided.

@zhongwuzw

Copy link
Copy Markdown
Member Author

layout_issues

@nathantannar4 nathantannar4 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.

Looks like this is a better solution and it renders the correct result

SD10
SD10 previously approved these changes Nov 1, 2017

@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.

👍

@nathantannar4 nathantannar4 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.

We will also need to change the update constraint method from

placeholderLabelConstraintSet?.width?.constant = -(placeholderLabelInsets.left + placeholderLabelInsets.right)

to

placeholderLabelConstraintSet?.right?.constant = -placeholderLabelInsets.right

@SD10

SD10 commented Nov 1, 2017 •

Copy link
Copy Markdown
Member

@nathantannar4 I was going to ask if you didn't want to change your mind this time 🤣

That new label is too similar to the colors of existing labels 🤔

I also think labels on PRs are stupid. I never remember to add/remove them myself. I just created under review to let people know I saw their PR && in progress to get people off my back 😃

@zhongwuzw

Copy link
Copy Markdown
Member Author

@nathantannar4 , I updated the changes.

@SD10
SD10 changed the base branch from v0.11.0 to master November 1, 2017 07:16
nathantannar4
nathantannar4 previously approved these changes Nov 1, 2017
@nathantannar4

Copy link
Copy Markdown
Member

🔥

@nathantannar4

Copy link
Copy Markdown
Member

@zhongwuzw Can you just make a change log entry for this in the CHANGELOG.md ? These are non-breaking changes but we need to document them

@zhongwuzw
zhongwuzw dismissed stale reviews from nathantannar4 and SD10 via c9c632e November 1, 2017 07:54
@zhongwuzw

Copy link
Copy Markdown
Member Author

@nathantannar4 , I added the CHANGELOG entries. Forgive my little poor English .:cn::cn:

@nathantannar4
nathantannar4 changed the base branch from master to v0.11.0 November 1, 2017 08:54
@nathantannar4
nathantannar4 merged commit f2c5702 into MessageKit:v0.11.0 Nov 1, 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