Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

Request: Add layout-padding #830

Closed
epelc opened this issue Dec 3, 2014 · 4 comments
Closed

Request: Add layout-padding #830

epelc opened this issue Dec 3, 2014 · 4 comments

Comments

@epelc
Copy link
Contributor

epelc commented Dec 3, 2014

For some reason layout-padding was replaced with layout-margin and according to @ThomasBurleson it has been depreciated. However they are both very useful and I find it hard to argue one is more useful than the other so I propose we have both. Is anyone against adding it back?

I recently started a post on the forums but I don't think it has much traffic yet so I opened this issue.

@epelc
Copy link
Contributor Author

epelc commented Dec 3, 2014

I also noticed layout-margin doesn't play nicely with 'layout-fill when they are on the same element since they both set margin.

@ThomasBurleson ThomasBurleson added this to the Backlog milestone Dec 3, 2014
@ThomasBurleson
Copy link
Contributor

@ajoslin - can you comment on this plz.

@ajoslin
Copy link
Contributor

ajoslin commented Dec 3, 2014

layout-padding was removed because I saw so many cases where the padding wasn't working correctly.

But then, there are cases where it does work correctly. As you said @epelc, let's add it back with margin. That's the best solution.

@ajoslin ajoslin closed this as completed in 5caa22b Dec 3, 2014
@epelc
Copy link
Contributor Author

epelc commented Dec 3, 2014

@ajoslin Thanks for this

@Splaktar Splaktar removed this from the - Backlog milestone Feb 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants