Skip to content
This repository has been archived by the owner. It is now read-only.

Introduce margin mixins #20

Merged
merged 4 commits into from Jun 9, 2014
Merged

Introduce margin mixins #20

merged 4 commits into from Jun 9, 2014

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented Jun 6, 2014

  • Switch from extending silent classes to using mixins for cleaner stylesheets

(This will affect Transition which has previously used silent classes)

fofr added 4 commits Jun 6, 2014
* They shouldn’t be extended, a mixin or variable should be used instead
* Create `default-label-margin` and `default-horizontal-margin`
* Use a mixin rather than extending silent classes
* Make !important an option, use it in margin helper classes
* Common usage should be `@include add-top-margin;` which will provide
the expect margins without an important side effect
bishboria added a commit that referenced this pull request Jun 9, 2014
@bishboria bishboria merged commit 2cf9b10 into master Jun 9, 2014
1 check passed
1 check passed
default "Build #30 succeeded on Jenkins"
Details
@bishboria bishboria deleted the margin-mixins branch Jun 9, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.