tylerchilds / cutestrap

A strong, independent CSS Framework. Only 2.7KB minified & gzipped.
https://www.cutestrap.com
GNU General Public License v3.0
1.57k stars 80 forks source link

remove dd text indent #38

Closed patrickhlauke closed 8 years ago

patrickhlauke commented 8 years ago

removes the text indent on dd, which causes very unsightly visual effect when the definition description is multiline (see example screenshot below)

before:

cute-dd-current

after:

cute-dd-fix

alternatively, suggest changing from text-indent to padding-left, which would then move all dd content to the right, not just the beginning of each line.

alternative:

cute-dd-alt

tylerchilds commented 8 years ago

I like the alternative solution, using padding to indent. It'll maintain the hierarchy of the content belonging to the term. I agree the current implementation is unsightly. Good ideas :)

patrickhlauke commented 8 years ago

pushed new commit with padding-left :)

tylerchilds commented 8 years ago

Awesome, thanks! :)