geido commented 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 it looks with 8px. The spacing of the "Run" and "Save" buttons is different now, which makes the layout look unbalanced. In general, the 16px seems to give more breath to the layout. With the 8px it seems very packed. Just my humble opinion. <img width="1387" alt="Screen Shot 2021-05-10 at 18 45 56" src="https://user-images.githubusercontent.com/60598000/117686915-ff98b580-b1bf-11eb-992c-f61443153a56.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/117686915-ff98b580-b1bf-11eb-992c-f61443153a56.png"](https://user-images.githubusercontent.com/60598000/117686915-ff98b580-b1bf-11eb-992c-f61443153a56.png"){kind=link}