Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Improve shadow doc in PhysicalShapeLayer #8114

Merged
merged 2 commits into from
Mar 11, 2019
Merged

Conversation

liyuqian
Copy link
Contributor

The old x is confusing as it often refers to x-axis in 2D graphics.
Replace it with t which has a nice interpretation of tangent.

The old x is confusing as it often refers to x-axis in 2D graphics.
Replace it with t which has a nice interpretation of tangent.
@liyuqian liyuqian requested a review from dnfield March 11, 2019 18:43
@liyuqian
Copy link
Contributor Author

Merge on yellow as the build_android is actually green.

@liyuqian liyuqian merged commit 56d8634 into flutter:master Mar 11, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 11, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Mar 12, 2019
flutter/engine@a71609f...56d8634

git log a71609f..56d8634 --no-merges --oneline
56d8634 Improve shadow doc in PhysicalShapeLayer (flutter/engine#8114)
bd0f908 Adds a platfromViewId to SemanticsNode (flutter/engine#8055)

The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff (mklim@google.com), and stop
the roller if necessary.
RBogie pushed a commit to RBogie/flutter-engine that referenced this pull request Apr 8, 2019
The old x is confusing as it often refers to x-axis in 2D graphics.
Replace it with t which has a nice interpretation of tangent.
RBogie added a commit to RBogie/flutter-engine that referenced this pull request Apr 8, 2019
RBogie added a commit to RBogie/flutter-engine that referenced this pull request Apr 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants