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

   <!-- Greeting -->
   Hi @ismaelricopalasios-maker! I'm [Dosu](https://go.dosu.dev/dosubot) and 
I’m helping the superset team.
   
   <!-- Answer -->
   Superset's theming system expects all necessary tokens, like colorText and 
colorTextSecondary, to be defined in the active theme. There is no fallback 
logic (such as colorTextSecondary || colorText) implemented in the codebase, 
and components should not rely on such fallbacks. If a token is missing, the 
system either uses Ant Design's default or the value is simply absent. This 
design matches the recommendation to use a single theme color variable for 
consistent behavior across themes, as relying on fallbacks can cause the kind 
of visual inconsistencies you described when switching themes. Defining and 
using only one variable (like theme.colorText) is the intended and most robust 
approach for consistency across all themes[^1].
   
   [^1]: [Superset Theme System 
Source](https://github.com/apache/superset/blob/829e4d92d91ceae4b43b1ed3b063ffe45377799c/superset-frontend/packages/superset-ui-core/src/theme/Theme.tsx)
   
   
   <!-- 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/dbfdeff4-d3be-4544-ade2-2440fea3aff8?feedback_type=great_response)
 | 
[Irrelevant](https://app.dosu.dev/response-feedback/dbfdeff4-d3be-4544-ade2-2440fea3aff8?feedback_type=irrelevant_answer)
 | 
[Incorrect](https://app.dosu.dev/response-feedback/dbfdeff4-d3be-4544-ade2-2440fea3aff8?feedback_type=incorrect_sources)
 | 
[Verbose](https://app.dosu.dev/response-feedback/dbfdeff4-d3be-4544-ade2-2440fea3aff8?feedback_type=too_verbose)
 | 
[Hallucination](https://app.dosu.dev/response-feedback/dbfdeff4-d3be-4544-ade2-2440fea3aff8?feedback_type=hallucination)
 | [Report 
šŸ›](https://app.dosu.dev/response-feedback/dbfdeff4-d3be-4544-ade2-2440fea3aff8?feedback_type=bug_report)
 | 
[Other](https://app.dosu.dev/response-feedback/dbfdeff4-d3be-4544-ade2-2440fea3aff8?feedback_type=other)</sup>&nbsp;&nbsp;[![Chat
 with 
Dosu](https://dosu.dev/dosu-chat-badge.svg)](https://app.dosu.dev/a28d3c7e-a9d3-459e-9fb6-3a6f9ff4f357/ask?utm_source=githu
 b)&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/35100)


-- 
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: notifications-unsubscr...@superset.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org
For additional commands, e-mail: notifications-h...@superset.apache.org

Reply via email to