Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove the ratio_xy argument from complex radial gradients #2544

Merged
merged 1 commit into from Mar 19, 2018

Conversation

@mrobinson
Copy link
Member

mrobinson commented Mar 19, 2018

This is another step toward removing complex radial gradients completely
(#2439). A radius parameter is introduced to compute the ratio.
Currently this new parameter duplicates some of the functionality of
start-radius and end-radius, but once the calculation of these values
moves to prim_store we can remove those as well.


This change is Reviewable

This is another step toward removing complex radial gradients completely
(#2439). A radius parameter is introduced to compute the ratio.
Currently this new parameter duplicates some of the functionality of
start-radius and end-radius, but once the calculation of these values
moves to prim_store we can remove those as well.
@mrobinson mrobinson requested a review from glennw Mar 19, 2018
@glennw
glennw approved these changes Mar 19, 2018
@glennw
Copy link
Member

glennw commented Mar 19, 2018

@bors-servo
Copy link
Contributor

bors-servo commented Mar 19, 2018

📌 Commit f8137fd has been approved by glennw

@bors-servo
Copy link
Contributor

bors-servo commented Mar 19, 2018

Testing commit f8137fd with merge 99bde66...

bors-servo added a commit that referenced this pull request Mar 19, 2018
Remove the ratio_xy argument from complex radial gradients

This is another step toward removing complex radial gradients completely
(#2439). A radius parameter is introduced to compute the ratio.
Currently this new parameter duplicates some of the functionality of
start-radius and end-radius, but once the calculation of these values
moves to prim_store we can remove those as well.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/2544)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Mar 19, 2018

☀️ Test successful - status-appveyor, status-taskcluster, status-travis
Approved by: glennw
Pushing 99bde66 to master...

1 similar comment
@bors-servo
Copy link
Contributor

bors-servo commented Mar 19, 2018

☀️ Test successful - status-appveyor, status-taskcluster, status-travis
Approved by: glennw
Pushing 99bde66 to master...

@bors-servo bors-servo merged commit f8137fd into servo:master Mar 19, 2018
2 checks passed
2 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
homu Test successful
Details
@mrobinson mrobinson deleted the mrobinson:remove-ratio-xy branch Mar 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.