vogievetsky opened a new pull request #10454: URL: https://github.com/apache/druid/pull/10454
There was a styling regression in the web console that made the tick in the checkbox box not render.  This is what it looked like in previously:  After looking into it I decided to switch away from using blueprint's `Checkbox` control. For lists (like the one above) a more blueprint idiomatic way is to set an icon on a `MenuItem` this way does not require any css hacks. This is how it looks:   For all other places where a Checkbox was used (log tailing in the ShowLogs dialog and WarningChecklist) I switched the control to a switch - which I think is nicer because it is bigger and more satisfying to use. It is also more common throughout the UI.   I think it is an improvement in all cases. ---------------------------------------------------------------- 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]
