search

ScalesCSS / scalescss

A design-free, modular collection of OOCSS patterns, framework settings, and utilities for quickly building responsive web projects
http://scalescss.github.io/scalescss/
MIT License
37 stars 15 forks source link

set line-height on headings #25

Closed yodasw16 closed 6 years ago

yodasw16 commented 7 years ago

Add option to set a different line-height for headings. I think having an overall line-height setting as well as individual variables to set a different line-height for each heading level makes the most sense.

This would need to be implemented in the typography package. There is currently a $base-line-height variable set on the body. We would need a new variable called $heading-line-height and then one for each heading level that has the default variable as their value.

digilou commented 6 years ago

Are you still looking for help, @yodasw16? It looks like you need additions in your typography file?

yodasw16 commented 6 years ago

Sorry, I fell behind this week. I just added a CONTRIBUTING.md file to the repo. Take a look and thanks for your interest!

digilou commented 6 years ago

@yodasw16 I will take a look! This is a neat project you have going on.

yodasw16 commented 6 years ago

Thanks! Just added a few details to the issue above.

yodasw16 commented 6 years ago

Thanks again for contributing!