Skip to content

Check for SkParagraph enable status, and pass it to the created engine#939

Open
limitless-brain wants to merge 3 commits intoryanheise:minorfrom
limitless-brain:add-android-skparagraph-to-engine-args
Open

Check for SkParagraph enable status, and pass it to the created engine#939
limitless-brain wants to merge 3 commits intoryanheise:minorfrom
limitless-brain:add-android-skparagraph-to-engine-args

Conversation

@limitless-brain
Copy link
Copy Markdown

Check for SkParagraph enable status, and pass it to the created engine bug fix for the issue.

@ryanheise
Copy link
Copy Markdown
Owner

Thanks @limitless-brain for the PR. I'd definitely like to incorporate your contribution. I have left some comments regarding the diff that I'd like you to address before I can review it and see which lines you have actually changed. In addition to those two comments, there are many other locations where formatting changes should be undone before review.

@limitless-brain
Copy link
Copy Markdown
Author

Thanks @limitless-brain for the PR. I'd definitely like to incorporate your contribution. I have left some comments regarding the diff that I'd like you to address before I can review it and see which lines you have actually changed. In addition to those two comments, there are many other locations where formatting changes should be undone before review.

Noted, I will undo the formatting.

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