ryanahamilton commented on PR #39355:
URL: https://github.com/apache/airflow/pull/39355#issuecomment-2092804548

   The screenshots illustrate that this will certainly be an appealing 
feature—I really wish the execution were as easy as this… The path that we 
should take (IMO) to enabling color mode in a sustainable manner is to migrate 
the rest of the UI to React. Within the React app, 
[Chakra](https://v2.chakra-ui.com/), the UI framework being employed, already 
has a robust [color mode 
support](https://v2.chakra-ui.com/docs/styled-system/color-mode) baked in. The 
problem is that the React app is nested within the aged Flask+Bootstrap app and 
the two frameworks don't share a common color mode system. I'm sure it would be 
possible to bridge the divide/logic between them, but one would have to 
determine if that is worth the effort versus investing time in migrating 
more/the remaining UI to the React app. Once the UI is entirely in React, 
enabling it will trivial.


-- 
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: commits-unsubscr...@airflow.apache.org

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

Reply via email to