Skip to content

Create a function to round image into an ellipse#2456

Open
Yithis wants to merge 1 commit intofacebook:mainfrom
Yithis:ellipse_rounding_feature
Open

Create a function to round image into an ellipse#2456
Yithis wants to merge 1 commit intofacebook:mainfrom
Yithis:ellipse_rounding_feature

Conversation

@Yithis
Copy link
Copy Markdown
Contributor

@Yithis Yithis commented Jan 14, 2020

Motivation

#2452

Test Plan

Code was checked by loading different images in showcase app. Best examples are here and here.

@Yithis
Copy link
Copy Markdown
Contributor Author

Yithis commented Jan 14, 2020

Hope you have access to this link. I'm putting there my next changes, because as far as I know github do not support stacked PR.

@Yithis
Copy link
Copy Markdown
Contributor Author

Yithis commented Jan 14, 2020

Next changes are here.

Copy link
Copy Markdown
Contributor

@oprisnik oprisnik left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, I'm going to move this to the sample app.

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@oprisnik has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants