Remove defaultProps from ScatterPlot.jsx (#2618)

## Which problem is this PR solving?
- Part of #2596 

## Description of the changes
- 

## How was this change tested?
- 

## Checklist
- [ ] I have read
https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
- [ ] I have signed all commits
- [ ] I have added unit tests for the new functionality
- [ ] I have run lint and test steps successfully
  - for `jaeger`: `make lint test`
  - for `jaeger-ui`: `npm run lint` and `npm run test`

---------

Signed-off-by: cs-308-2023 <[email protected]>
Co-authored-by: Yuri Shkuro <[email protected]>

packages/jaeger-ui/src/components/SearchTracePage/SearchResults/ScatterPlot.jsx

@@ -23,9 +23,13 @@ import { ONE_MILLISECOND, formatDuration } from '../../../utils/date';
 import 'react-vis/dist/style.css';
 import './ScatterPlot.css';
 
-export default function ScatterPlot(props) {
-  const { data, onValueClick, calculateContainerWidth } = props;
-
+export default function ScatterPlot({
+  data,
+  onValueClick,
+  // JSDOM does not, as of 2023, have a layout engine,
+  // so allow tests to supply a mock width as a workaround
+  calculateContainerWidth = container => container.clientWidth,
+}) {
   const containerRef = useRef(null);
   const [containerWidth, setContainerWidth] = useState(0);
 
@@ -103,8 +107,3 @@ ScatterPlot.propTypes = {
   onValueClick: PropTypes.func.isRequired,
   calculateContainerWidth: PropTypes.func,
 };
-
-ScatterPlot.defaultProps = {
-  // JSDOM does not, as of 2023, have a layout engine, so allow tests to supply a mock width as a workaround.
-  calculateContainerWidth: container => container.clientWidth,
-};