Skip to content

Commit

Permalink
Remove offset from lines (#3854)
Browse files Browse the repository at this point in the history
<!--- Provide a general summary of your changes in the Title above -->

## Description

Ignores the offset property when rendering the line elements in the
`CartesianGrid` component. This property does not impact rendering.

<!--- Describe your changes in detail -->

## Related Issue
#3810 

## Motivation and Context

<!--- Why is this change required? What problem does it solve? -->
Solves issue #3810 

## How Has This Been Tested?

Manually checking to see that `offset` no longer exists for the line
elements when using the `CartesianGrid`


## Screenshots (if appropriate):

![image](https://github.com/recharts/recharts/assets/20285369/f8d4f5a9-1076-4327-90a4-b9e1710ae304)

## Types of changes

<!--- What types of changes does your code introduce? Put an `x` in all
the boxes that apply: -->

- [x] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing
functionality to change)

## Checklist:

<!--- Go over all the following points, and put an `x` in all the boxes
that apply. -->
<!--- If you're unsure about any of these, don't hesitate to ask. We're
here to help! -->

- [x] My code follows the code style of this project.
- [ ] My change requires a change to the documentation.
- [ ] I have updated the documentation accordingly.
- [ ] I have added tests to cover my changes.
- [x] I have added a storybook story or extended an existing story to
show my changes
- [x] All new and existing tests passed.
  • Loading branch information
branberry authored Oct 13, 2023
1 parent 9b6046b commit ad92abb
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/cartesian/CartesianGrid.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -92,8 +92,8 @@ export class CartesianGrid extends PureComponent<Props> {
lineItem = option(props);
} else {
const { x1, y1, x2, y2, key, ...others } = props;

lineItem = <line {...filterProps(others)} x1={x1} y1={y1} x2={x2} y2={y2} fill="none" key={key} />;
const { offset: __, ...restOfFilteredProps } = filterProps(others);
lineItem = <line {...restOfFilteredProps} x1={x1} y1={y1} x2={x2} y2={y2} fill="none" key={key} />;
}

return lineItem;
Expand Down

0 comments on commit ad92abb

Please sign in to comment.