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

Better Debug Guidelines #20

Closed
Snugug opened this issue Jul 26, 2012 · 9 comments
Closed

Better Debug Guidelines #20

Snugug opened this issue Jul 26, 2012 · 9 comments

Comments

@Snugug
Copy link
Member

Snugug commented Jul 26, 2012

It seems as if the background-grid mixin isn't so smart, especially with non-uniform grids, and we should come up with a better mixin for this.

@scottkellum
Copy link
Member

Works with non-uniform grids for me, kinda sucks though. Variables need to be tweaked a little to get passed through. Would like to add a way to view padding to this.

@Snugug
Copy link
Member Author

Snugug commented Jul 26, 2012

Hmm, I just tried it with a non-uniform grid, did (4, 1, red) and it gave me 4 uniform sized columns. Is there something I'm missing there? Maybe it's just that the documentation needs to be better?

@scottkellum
Copy link
Member

$columns, $gutter, $color

you might have wanted to do
((4, 1), $color: red)

@Snugug
Copy link
Member Author

Snugug commented Jul 26, 2012

Nope, that's now giving me an asymetrical grid, but it's not the right grid.

@scottkellum
Copy link
Member

Not clear what you want, the first column there should be 4x the last column. (just two columns)

@Snugug
Copy link
Member Author

Snugug commented Jul 26, 2012

Oh! So do I need to put in my full columns stuff?

@Snugug
Copy link
Member Author

Snugug commented Jul 26, 2012

Yup, yup, that's what I needed to do.
So I guess, either better docs for this, or make it smarter?

@scottkellum
Copy link
Member

Want to pull existing vars? should work without anything (). unless some vars aren’t being pulled properly.

@Snugug
Copy link
Member Author

Snugug commented Jul 26, 2012

So it does. so it does.

@Snugug Snugug closed this as completed Jul 26, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants