geido edited a comment on pull request #14505: URL: https://github.com/apache/superset/pull/14505#issuecomment-836865951
@mihir174 I would like to add that the 8px padding does not look very pretty. I think the original 16px padding works much better. Look at the screenshot below of how the control panel in the center looks like with 8px compared to the one at the very left. Additionally, the spacing of the "Run" and "Save" buttons is different now, which makes the layout look unbalanced. In general, my humble opinion is that the 16px seems to give more breath to the layout. <img width="1680" alt="Screen Shot 2021-05-10 at 18 48 52" src="https://user-images.githubusercontent.com/60598000/117687358-64541000-b1c0-11eb-9c81-749983d6c50f.png";> -- 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]
![https://user-images.githubusercontent.com/60598000/117687358-64541000-b1c0-11eb-9c81-749983d6c50f.png"](https://user-images.githubusercontent.com/60598000/117687358-64541000-b1c0-11eb-9c81-749983d6c50f.png"){kind=link}