Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Adding "auto" to acceptable margin values #113

Merged
merged 2 commits into from Nov 13, 2012

Conversation

Projects
None yet
2 participants
Contributor

mindofmicah commented Oct 25, 2012

.mha replaces having a class to center block-level elements in a container. The rest are added to make this a complete set.

Added in 'auto' as an available value on margins
.mha replaces having a class to center block-level elements in a container. The rest are added to make this a complete set.

How would mta and mba behave?

Owner

mindofmicah replied Nov 12, 2012

Originally I had added them to 'complete the set' and intended to make use of the classes down the road; however after reading up on it, setting either margin-bottom or margin-top to auto effectively just sets the value to 0. Since that functionality is already handled in .mtn and .mbn, they could/should be removed. My repo has the rules affecting those properties removed now.

Removed the unnecessary declarations
margin-top:auto; and margin-bottom:auto are the same as margin-top:0; and margin-bottom:0;

stubbornella added a commit that referenced this pull request Nov 13, 2012

Merge pull request #113 from mindofmicah/master
Adding "auto" to acceptable margin values

@stubbornella stubbornella merged commit 7223cd1 into stubbornella:master Nov 13, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment