jaegertracing/jaeger-ui

Update Tooltip Arrow Configuration in Ant Design Components

bupd opened this issue · 1 comments

bupd commented

What happened?

arrowPointatCenter

The Arrow Point at center for the tooltip has been officially deprecated in Antd. Since we are using antd v5.
We should stop using the arrow pointAtCenter and move to arrow: {{ pointAtCenter: true }}

We need to remove the old convention and move to the updated convention to avoid future issues.

Steps to reproduce

  1. With the deprecation of the arrowPointAtCenter as boolean property in Ant Design Tooltip component, it's crucial to update our codebase to adhere to the latest conventions provided by Ant Design version 5.

Expected behavior

To align with the latest Ant Design conventions and avoid future compatibility issues, we need to update our codebase to use the arrow property with pointAtCenter: true configuration instead of arrowPointAtCenter.

By updating the Tooltip arrow configuration to use the latest Ant Design conventions, we ensure compatibility with future versions of Ant Design and avoid potential deprecation warnings. This proactive approach helps maintain the stability and longevity of our codebase, contributing to better maintainability and scalability of our application.

Relevant log output

No response

Screenshot

No response

Additional context

No response

Jaeger backend version

No response

SDK

No response

Pipeline

No response

Stogage backend

No response

Operating system

No response

Deployment model

No response

Deployment configs

No response

bupd commented

Here are the Steps I am going to implement for solving this issue.

  1. Identify all occurrences of arrowPointAtCenter property usage in our codebase.
  2. Replace each instance with the updated arrow property with pointAtCenter: true configuration.
  3. Test the updated components to ensure they function as expected with the new configuration.