dosubot[bot] commented on issue #36967:
URL: https://github.com/apache/superset/issues/36967#issuecomment-3721960150

   <!-- Answer -->
   You're right—this is not a user config issue. The Calendar Chart tooltip and 
highlight problems affect everyone using the default theme because the chart 
uses hardcoded grayscale tokens for tooltip styling, not the dynamic Ant Design 
tokens. In the Light theme, these tokens should provide good contrast, but if 
the computed colors are too similar (which can happen due to how they're 
generated), you get unreadable tooltips. Persistent highlights are a limitation 
of the CalHeatMap integration, which doesn't always clear highlight classes on 
mouseout.
   
   There are no existing bug reports, merged PRs, or documentation addressing 
this specific issue, so it's not currently tracked as a known bug. Fixing it at 
the code level would mean refactoring the Calendar Chart to use dynamic theme 
tokens or component-level theming, and improving how highlights are managed and 
cleared in the CalHeatMap integration. This would make the chart more robust 
and theme-aware going forward.
   
   If you or anyone else wants to contribute a fix, the relevant code is in 
`superset-frontend/plugins/legacy-plugin-chart-calendar/src/ReactCalendar.jsx`, 
where the tooltip and highlight styles are injected. The theme token 
computation logic is in the theme utility files. Refactoring to use more 
flexible, dynamic tokens (and ensuring highlight cleanup) would address the 
root cause. The Superset team welcomes detailed bug reports and 
contributions—clear steps to reproduce, screenshots, and environment details 
help a lot in getting these kinds of issues prioritized and fixed 
[source](https://github.com/apache/superset/wiki/Release-Process) 
[source](https://github.com/apache/superset/issues/36842).
   
   
   <!-- Dosu Comment Footer -->
   *To reply, just mention [@dosu](https://go.dosu.dev/dosubot).*
   
   ---
   <sup>How did I do? 
[Good](https://app.dosu.dev/response-feedback/9fe1543d-4cba-41fb-9bce-5f63292830a6?feedback_type=great_response)
 | 
[Irrelevant](https://app.dosu.dev/response-feedback/9fe1543d-4cba-41fb-9bce-5f63292830a6?feedback_type=irrelevant_answer)
 | 
[Incorrect](https://app.dosu.dev/response-feedback/9fe1543d-4cba-41fb-9bce-5f63292830a6?feedback_type=incorrect_sources)
 | 
[Verbose](https://app.dosu.dev/response-feedback/9fe1543d-4cba-41fb-9bce-5f63292830a6?feedback_type=too_verbose)
 | 
[Hallucination](https://app.dosu.dev/response-feedback/9fe1543d-4cba-41fb-9bce-5f63292830a6?feedback_type=hallucination)
 | [Report 
🐛](https://app.dosu.dev/response-feedback/9fe1543d-4cba-41fb-9bce-5f63292830a6?feedback_type=bug_report)
 | 
[Other](https://app.dosu.dev/response-feedback/9fe1543d-4cba-41fb-9bce-5f63292830a6?feedback_type=other)</sup>
   
   [![Chat with 
Dosu](https://dosu.dev/dosu-chat-badge.svg)](https://app.dosu.dev/a28d3c7e-a9d3-459e-9fb6-3a6f9ff4f357/ask?utm_source=github)&nbsp;[![Open
 in 
Cursor](https://dosu.dev/dosu-in-cursor.svg)](https://cursor.com/link/prompt?text=You%27re%20right%E2%80%94this%20is%20not%20a%20user%20config%20issue.%20The%20Calendar%20Chart%20tooltip%20and%20highlight%20problems%20affect%20everyone%20using%20the%20default%20theme%20because%20the%20chart%20uses%20hardcoded%20grayscale%20tokens%20for%20tooltip%20styling%2C%20not%20the%20dynamic%20Ant%20Design%20tokens.%20In%20the%20Light%20theme%2C%20these%20tokens%20should%20provide%20good%20contrast%2C%20but%20if%20the%20computed%20colors%20are%20too%20similar%20%28which%20can%20happen%20due%20to%20how%20they%27re%20generated%29%2C%20you%20get%20unreadable%20tooltips.%20Persistent%20highlights%20are%20a%20limitation%20of%20the%20CalHeatMap%20integration%2C%20which%20doesn%27t%20always%20clear%20highlight%20classes%20on%20mouseout.%0A%0AThere
 
%20are%20no%20existing%20bug%20reports%2C%20merged%20PRs%2C%20or%20documentation%20addressing%20this%20specific%20issue%2C%20so%20it%27s%20not%20currently%20tracked%20as%20a%20known%20bug.%20Fixing%20it%20at%20the%20code%20level%20would%20mean%20refactoring%20the%20Calendar%20Chart%20to%20use%20dynamic%20theme%20tokens%20or%20component-level%20theming%2C%20and%20improving%20how%20highlights%20are%20managed%20and%20cleared%20in%20the%20CalHeatMap%20integration.%20This%20would%20make%20the%20chart%20more%20robust%20and%20theme-aware%20going%20forward.%0A%0AIf%20you%20or%20anyone%20else%20wants%20to%20contribute%20a%20fix%2C%20the%20relevant%20code%20is%20in%20%60superset-frontend/plugins/legacy-plugin-chart-calendar/src/ReactCalendar.jsx%60%2C%20where%20the%20tooltip%20and%20highlight%20styles%20are%20injected.%20The%20theme%20token%20computation%20logic%20is%20in%20the%20theme%20utility%20files.%20Refactoring%20to%20use%20more%20flexible%2C%20dynamic%20tokens%20%28and%20ensuring%20hi
 
ghlight%20cleanup%29%20would%20address%20the%20root%20cause.%20The%20Superset%20team%20welcomes%20detailed%20bug%20reports%20and%20contributions%E2%80%94clear%20steps%20to%20reproduce%2C%20screenshots%2C%20and%20environment%20details%20help%20a%20lot%20in%20getting%20these%20kinds%20of%20issues%20prioritized%20and%20fixed%20%5Bsource%5D%28https%3A//github.com/apache/superset/wiki/Release-Process%29%20%5Bsource%5D%28https%3A//github.com/apache/superset/issues/36842%29.)&nbsp;[![Join
 
Discord](https://img.shields.io/badge/join-5865F2?logo=discord&logoColor=white&label=)](https://go.dosu.dev/discord-bot)&nbsp;[![Share
 on 
X](https://img.shields.io/badge/X-share-black)](https://twitter.com/intent/tweet?text=%40dosu_ai%20helped%20me%20solve%20this%20issue!&url=https%3A//github.com/apache/superset/issues/36967)


-- 
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