Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed partials ordering and added missing responsive-embed one #18

Merged
merged 3 commits into from Jul 5, 2015

Conversation

mgtitimoli
Copy link
Contributor

This PR fixes this issue

'scrollspy': true,
'tab': true,
'affix': true
"transition": true,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No reason to change to "". I've got the linter set to single quotes for JS.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Np 👍 I'll revert this now, I've just standardized to double quotes, since in all the other places in this file this is the kind used.

@justin808
Copy link
Member

Just one minor issue with quotes.

@justin808
Copy link
Member

@mgtitimoli Can you please verify this change works over here:

https://github.com/justin808/bootstrap-sass-loader-example

I won't have time to test this for a few days. However, the code looks fine. I'd like to merge this if I can get at least one person to say that that there's no issues with this change. Otherwise, I'll get this in a few days.

@justin808
Copy link
Member

@mgtitimoli Any update?

@justin808 justin808 merged commit 3043f7e into shakacode:master Jul 5, 2015
@mgtitimoli mgtitimoli deleted the partials-ordering branch July 30, 2015 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong partials ordering
2 participants