salcode / bootstrap-genesis

WordPress Genesis Child Theme setup to use Bootstrap, Sass, and Grunt
MIT License
184 stars 63 forks source link

Code formatting #114

Closed codenameEli closed 8 years ago

codenameEli commented 8 years ago

Indentation was not there for meta/head tags. Could be intentional by you, just throwing it out there. PR incoming

salcode commented 8 years ago

I'm not sure how I feel about how this affects the rendered markup

Original on the left, your PR on the right

image

This is an interesting question, do we do indentation based on final rendered markup or source code?

salcode commented 8 years ago

My gut is we keep it the way it currently is.

Since there hasn't been any activity on this, I'm going to close this issue. If this specific issue still exists, we can reopen. If there is something tangentially related, we can create a new issue.