susiwen8 opened a new pull request, #21599:
URL: https://github.com/apache/echarts/pull/21599

   ## Summary
   
   - Pass radar indicator indexes through tooltip markup fragments so 
`tooltip.valueFormatter(value, dataIndex)` receives the indicator index for 
radar values.
   - Add a unit regression for the tooltip markup callback path.
   - Add a manual HTML case to `test/tooltip-valueFormatter.html`.
   
   ## Root Cause
   
   Radar builds one tooltip `nameValue` block per indicator, but those blocks 
did not carry their indicator position. The common tooltip markup layer already 
forwards `fragment.dataIndex` to `valueFormatter`, so radar value formatters 
received `undefined`.
   
   ## Related
   
   Fixes #20109
   
   ## Testing
   
   - `npx jest --config test/ut/jest.config.cjs --coverage=false 
test/ut/spec/component/tooltip/tooltip.test.ts`
   - `npm run checktype`
   - `npm run lint`
   - Screenshot/manual verification of radar tooltip values showing `idx:0`, 
`idx:1`, and `idx:2`
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to