Skip to content

Create a function to round image into an ellipse#2456

Open
Yithis wants to merge 1 commit into
facebook:mainfrom
Yithis:ellipse_rounding_feature
Open

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

Conversation

@Yithis

@Yithis Yithis commented Jan 14, 2020

Copy link
Copy Markdown
Contributor

Motivation

#2452

Test Plan

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

@Yithis

Yithis commented Jan 14, 2020

Copy link
Copy Markdown
Contributor Author

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

Yithis commented Jan 14, 2020

Copy link
Copy Markdown
Contributor Author

Next changes are here.

@oprisnik oprisnik left a comment

Copy link
Copy Markdown
Contributor

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.

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

Copy link
Copy Markdown
Contributor

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

3 participants