Skip to content

Commit

Permalink
fix(doc): fix grammar errors in docs/comments
Browse files Browse the repository at this point in the history
Co-authored-by: Neil Kistner <neil.kistner@gmail.com>
  • Loading branch information
plouc and wyze committed Dec 18, 2020
1 parent 6276637 commit b5bbeda
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 7 deletions.
Expand Up @@ -148,7 +148,7 @@ const interpolateTextAnchor = (
* as `interpolateLink`, unfortunately `react-spring` does not support
* multiple output values from a single interpolation.
*
* We should revise this if `react-spring` add this feature at some point.
* We should revise this if `react-spring` adds this feature at some point.
*/
const interpolateTextPosition = (
startAngleValue: SpringValue<AnimatedProps['startAngle']>,
Expand Down
2 changes: 1 addition & 1 deletion packages/arcs/src/arc_link_labels/useArcLinks.ts
Expand Up @@ -23,7 +23,7 @@ export const useArcLinks = <
}: {
data: Datum[]
// arcs with a length below this (end angle - start angle in degrees)
// are gonna be excluded, this can be typically used to avoid having
// are going to be excluded, this can typically be used to avoid having
// overlapping labels.
skipAngle?: number
// offset from arc outer radius in pixels
Expand Down
2 changes: 1 addition & 1 deletion packages/arcs/src/centers.ts
Expand Up @@ -103,7 +103,7 @@ export const useArcCenters = <
// 1.0: outer radius
offset?: number
// arcs with a length below this (end angle - start angle in degrees)
// are gonna be excluded, this can be typically used to avoid having
// are going to be excluded, this can typically be used to avoid having
// overlapping labels.
skipAngle?: number
// this can be used to append extra properties to the centers,
Expand Down
4 changes: 2 additions & 2 deletions packages/arcs/src/interpolateArc.ts
Expand Up @@ -3,8 +3,8 @@ import { ArcGenerator } from './types'

/**
* Directly animating paths for arcs leads to sub-optimal results
* as the interpolation is gonna be linear while we deal with polar coordinates,
* this interpolator is gonna generate proper arc transitions.
* as the interpolation is going to be linear while we deal with polar coordinates,
* this interpolator is going to generate proper arc transitions.
* It should be used with the `useAnimatedArc` or `useArcsTransition` hooks.
*/
export const interpolateArc = (
Expand Down
2 changes: 1 addition & 1 deletion packages/arcs/src/useArcGenerator.ts
Expand Up @@ -8,7 +8,7 @@ import { ArcGenerator, Arc } from './types'
* Please note that both inner/outer radius aren't static
* and should come from the arc itself, while it requires
* more props on the arcs, it provides more flexibility
* because it's not limited to pie then but can also works
* because it's not limited to pie then but can also work
* with charts such as sunbursts.
*/
export const useArcGenerator = ({
Expand Down
2 changes: 1 addition & 1 deletion packages/arcs/src/utils.ts
Expand Up @@ -4,7 +4,7 @@ import { DatumWithArc } from './types'

/**
* Make sure an angle (expressed in radians)
* always fall in the range 0~2*PI.
* always falls in the range 0~2*PI.
*/
export const getNormalizedAngle = (angle: number) => {
let normalizedAngle = angle % (Math.PI * 2)
Expand Down

0 comments on commit b5bbeda

Please sign in to comment.