ktmud commented on pull request #10056: URL: https://github.com/apache/incubator-superset/pull/10056#issuecomment-645688093
> What's the proper merge point? I also like the new colors. Just felt having a11y regression or two noticeably incompatible styles is worse than having an outdated style. How about we reduce the scope of this PR a little to only change the primary color from kazan to cartel blue + darker blue for text and leave btn-default and others untouched? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
