-
Notifications
You must be signed in to change notification settings - Fork 881
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
Add spaces between property: value; in CSS #21
Conversation
Forgot spaces before opening brackets the first time around. |
Of course this is purely personal preference, but
Since CSS Guidelines is supposed to be a reference material, decreasing (although really small) amount of time needed to "parse" the code by utilizing the style that is more common should be a good thing. I'm for merging this in 👍 |
+1 |
Well, this is how Harry writes CSS (example), so it might be a bit hard to get this merged to his very own guidelines ;) |
@pim Actually this is out of context here on github. I have already asked @csswizardry about this on Twitter and he agreed: https://twitter.com/csswizardry/status/426406662015287296 I pointed out that given his method of using 2x spaces in markup between classes for legibility, it was inconsistent to remove spaces in his declarations what reduce legibility. Harry said he has already started adapting his style to be consistent with this and that he would be open to accepting this pull request. |
@inlikealion Oh, I understand. That’s good then – I prefer to use spaces too :) |
Add spaces between property: value; in CSS
Does what it says on the label.