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

@else should be placed on same line as previous curly brace #6295

Closed
wants to merge 6 commits into from
Closed

@else should be placed on same line as previous curly brace #6295

wants to merge 6 commits into from

Conversation

marcobiedermann
Copy link
Contributor

@rafibomb
Copy link
Member

Thanks! Standards are good - can't auto-merge this but we'll get it in soon.

@rafibomb rafibomb added the scss label Feb 19, 2015
@rafibomb rafibomb added this to the 5.5.2 milestone Feb 19, 2015
@marcobiedermann
Copy link
Contributor Author

I just got one question. What if there is a comment above the @else condition?
See https://github.com/zurb/foundation/blob/master/scss%2Ffoundation%2Fcomponents%2F_grid.scss#L54

@gakimball
Copy link
Contributor

You could move the comment inside the block, and then you'd be able to pair all the @else blocks together.

@marcobiedermann
Copy link
Contributor Author

Ok, I'll fix it soon

Conflicts:
	scss/foundation/components/_orbit.scss
Conflicts:
	scss/foundation/components/_dropdown.scss
	scss/foundation/components/_global.scss
	scss/foundation/components/_grid.scss
	scss/foundation/components/_switches.scss
Conflicts:
	js/foundation/foundation.abide.js
	scss/foundation/components/_range-slider.scss
@rafibomb rafibomb modified the milestones: 5.5.2, 5.5.3 May 13, 2015
@gakimball
Copy link
Contributor

Hey, really sorry for letting this get super stale! Now that Foundation 6 is out, we won't be making any more codebase improvements to Foundation 5. We're always looking for contributions to the new codebase, though :)

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.

None yet

4 participants