dosubot[bot] commented on issue #34141: URL: https://github.com/apache/superset/issues/34141#issuecomment-3063518863
<!-- Answer --> This is a known UX gap: the "Green for increase, red for decrease" (and similar) conditional formatting options in Table Chart only work when a time comparison (time shift) is active. The code for built-in comparison color schemes hides these options unless a time shift is set, but the custom conditional formatting control still lets you select them even when no time comparison is present. As a result, picking these options without a time shift has no effect—they are ignored by the chart logic, which expects a comparison column to be available for the increase/decrease calculation. This can be confusing, since the UI allows the selection but nothing happens. There is currently no documentation or warning in the UI about this behavior. The control panel should ideally restrict or warn about these options when no time shift is set, since they will never have an effect in that case. This is a UI/UX issue, not a backend bug. For reference, see the relevant code and logic in the Table Chart controls and transformProps files: [controlPanel.tsx](https://github.com/apache/superset/blob/829e4d92d91ceae4b43b1ed3b063ffe45377799c/superset-frontend/plugins/plugin-chart-table/src/controlPanel.tsx) [transformProps.ts](https://github.com/apache/superset/blob/829e4d92d91ceae4b43b1ed3b063ffe45377799c/superset-frontend/plugins/plugin-chart-table/src/transformProps.ts) <!-- 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/3b6fad9c-9139-44af-a3fb-3b932a450fab?feedback_type=great_response) | [Irrelevant](https://app.dosu.dev/response-feedback/3b6fad9c-9139-44af-a3fb-3b932a450fab?feedback_type=irrelevant_answer) | [Incorrect](https://app.dosu.dev/response-feedback/3b6fad9c-9139-44af-a3fb-3b932a450fab?feedback_type=incorrect_sources) | [Verbose](https://app.dosu.dev/response-feedback/3b6fad9c-9139-44af-a3fb-3b932a450fab?feedback_type=too_verbose) | [Hallucination](https://app.dosu.dev/response-feedback/3b6fad9c-9139-44af-a3fb-3b932a450fab?feedback_type=hallucination) | [Report 🐛](https://app.dosu.dev/response-feedback/3b6fad9c-9139-44af-a3fb-3b932a450fab?feedback_type=bug_report) | [Other](https://app.dosu.dev/response-feedback/3b6fad9c-9139-44af-a3fb-3b932a450fab?feedback_type=other)</sup> [](https://go.dosu.dev/discord-bot) [! [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/34141) -- 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]
