EnxDev commented on PR #35897: URL: https://github.com/apache/superset/pull/35897#issuecomment-3907211602
> > During manual testing, I observed that users can select a background color (e.g., error) for a row and then add a condition that sets the text color to the same value. This results in the text becoming invisible due to identical foreground and background colors. > > What are your thoughts on this? > > I don't think this is a problem with this merge request. The same can be done now. Is it even necessary to colorize both the cell and the text at the same time? I don’t think it’s necessary to color both, so I don’t believe it should be allowed. At the moment, I don’t see this as a major issue, but we can gather additional feedback. @rusackas , what do you think? -- 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]
