innovark37 opened a new pull request, #40420: URL: https://github.com/apache/superset/pull/40420
### SUMMARY Fixes Mixed Timeseries value labels using the wrong Y-axis formatter when Query A and Query B are assigned to different Y axes. Previously, value labels were formatted based on the query slot instead of the assigned axis: - Query A used the primary Y-axis formatter. - Query B used the secondary Y-axis formatter. This was incorrect when a query was assigned to the opposite axis. The fix selects the formatter from the series’ assigned `yAxisIndex` / `yAxisIndexB`, matching tooltip and Y-axis tick formatting behavior. Bar value-label clipping is also aligned with the assigned Y-axis bounds. ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF BEFORE <img width="1911" height="913" alt="image" src="https://github.com/user-attachments/assets/75322566-3163-4921-98d4-0d31b334b810"; /> <img width="333" height="156" alt="image" src="https://github.com/user-attachments/assets/1600d2c9-a975-4916-9b55-be582a5dae41"; /> AFTER <img width="1911" height="902" alt="{1984C82B-389C-49BE-B8E3-8927D5D2D69D}" src="https://github.com/user-attachments/assets/83157594-4b31-47b8-b7c5-0551584e4919"; /> <img width="356" height="190" alt="{43516732-6307-4374-B0CA-F1E83BCF1F55}" src="https://github.com/user-attachments/assets/0a2d7d61-8a3b-400d-be11-030187572ceb"; /> ### TESTING INSTRUCTIONS Manual verification: 1. Create a Mixed Timeseries chart. 2. Configure Query A as a line chart and assign it to the secondary Y-axis. 3. Configure Query B as a bar chart and assign it to the primary Y-axis. 4. Enable value labels with Show Values. 5. Set the primary Y-axis format to `.0%`. 6. Set the secondary Y-axis format to `,.1f`. 7. Verify Query A line value labels use the secondary Y-axis format. 8. Verify Query B bar value labels use the primary Y-axis format. 9. Reverse the Y-axis assignments and verify labels follow the assigned axis. ### ADDITIONAL INFORMATION <!--- Check any relevant boxes with "x" --> <!--- HINT: Include "Fixes #nnn" if you are fixing an existing issue --> - [x] Has associated issue: Fixes #40419 - [ ] Required feature flags: - [ ] Changes UI - [ ] Includes DB Migration (follow approval process in [SIP-59](https://github.com/apache/superset/issues/13351)) - [ ] Migration is atomic, supports rollback & is backwards-compatible - [ ] Confirm DB migration upgrade and downgrade tested - [ ] Runtime estimates and downtime expectations provided - [ ] Introduces new feature or API - [ ] Removes existing feature or API -- 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]
