Skip to content

add default to scss variables#471

Merged
connors merged 1 commit intotwbs:masterfrom
christianseel:patch-1
Apr 1, 2014
Merged

add default to scss variables#471
connors merged 1 commit intotwbs:masterfrom
christianseel:patch-1

Conversation

@christianseel
Copy link
Copy Markdown
Contributor

See issue #444

Doing so allows one to override default variables when writing custom SASS that imports Ratchet's SASS.
@cvrebert cvrebert added the css label Mar 9, 2014
@XhmikosR XhmikosR added this to the 2.0.2 milestone Mar 10, 2014
@XhmikosR
Copy link
Copy Markdown
Member

@connors: this seems to make sense.

@glebm: since SASS is more of your area of expertise, can you share your thoughts about this?

connors added a commit that referenced this pull request Apr 1, 2014
add default to scss variables
@connors connors merged commit acdeb65 into twbs:master Apr 1, 2014
@connors connors mentioned this pull request Apr 1, 2014
@glebm
Copy link
Copy Markdown
Member

glebm commented Apr 1, 2014

@XhmikosR A bit of a late reply, but, yes, this is good stuff.

@XhmikosR
Copy link
Copy Markdown
Member

XhmikosR commented Apr 1, 2014

Better late than never. Thanks @glebm :)

@connors
Copy link
Copy Markdown
Collaborator

connors commented Apr 1, 2014

:) 👍

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants