weeheavy/pelican-theme
1
0
Fork 0
forked from github/pelican
Code Pull requests Activity

Use a top (instead of bottom) border in hentry.

Browse source 
`hentry` uses `clear: both` with the intention of placing a separator line (1px border) between each entry. However, it is wrongly using `border-bottom` instead of `border-top` to make that separator. CSS `clear` makes space to clear the **preceding** floats. Hence a `border-bottom` won't enjoy that `clear` effect.
This commit is contained in:
Nam Nguyen 2021-01-05 10:40:31 -08:00 • committed by GitHub
commit b12443f48a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
pelican/themes/notmyidea/static/css/main.css
View file

@ -384,10 +384,10 @@ div.figure p.caption, figure p.caption { /* margin provided by figure */
.hentry {
display: block;
clear: both;
border-bottom: 1px solid #eee;
border-top: 1px solid #eee;
padding: 1.5em 0;
}
li:last-child .hentry, #content > .hentry {border: 0; margin: 0;}
li:first-child .hentry, #content > .hentry {border: 0; margin: 0;}
#content > .hentry {padding: 1em 0;}
.hentry img{display : none ;}
.entry-title {font-size: 3em; margin-bottom: 10px; margin-top: 0;}