rusackas commented on pull request #15265: URL: https://github.com/apache/superset/pull/15265#issuecomment-865960588
One small detail to consider... I noticed that the icons are a little high up, vertically. It seems the `anticon` class has a `vertical-align` of `-.125em` for some reason. If I override that with `vertical-align: middle;` it aligns much better with the text.  While this _sounds_ like a sensible global change, I'm not sure what fallout/misalignments it might cause. We can pus this PR through as is, or stop and poke around at this. Your call. If you want to merge as is, maybe we can make a project board of "Things to try when we have visual regression testing" -- 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]
