search

ContextInstitute / bfcom

http://bfn.context.org
GNU General Public License v3.0
4 stars 1 forks source link

Added bottom border to header #59

Closed DavidScottBernstein closed 5 years ago

DavidScottBernstein commented 5 years ago

Fixed #49

iangilman commented 5 years ago

@DavidScottBernstein We could certainly do it with an hr, but I think a more flexible and cleaner approach would be to apply a border-bottom to the header. You may need to adjust the header's padding and margins to make it look good, but that's good stuff to do anyway. Looks like the class to style is probably .header-grid; you can find it near the bottom of style.css.

Don't forget to resize your browser to see what it looks like on mobile. Looks like you might need to adjust the grid-template-rows when on mobile, or the header is too tall.

iangilman commented 5 years ago

Closing this in favor of #61.