static/css/main.css - Issue 29720642: Noissue - Added .body-container to provide padding-top for body content and avoid navbar overlap

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments.

Keyboard Shortcuts

	File
u :	up to issue
m :	publish + mail comments
M :	edit review message
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line
<Enter> :	respond to / edit current comment
d :	mark current comment as done

	Issue
u :	up to list of issues
m :	publish + mail comments
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue
# :	close issue

	Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :	save comment
<Esc> :	cancel edit

Side by Side Diff: static/css/main.css

Issue 29720642: Noissue - Added .body-container to provide padding-top for body content and avoid navbar overlap (Closed) Base URL: https://hg.adblockplus.org/web.adblockplus.org

Patch Set: Created March 12, 2018, 5:02 p.m.

Left:
Right:

Use n/p to move between diff chunks; N/P to move between comments.

Jump to:

View unified diff | Download patch

OLD	NEW
1 *	1 *

2 {	2 {

3 font-family: Arial, sans;	3 font-family: Arial, sans;

4 font-size: 16px;	4 font-size: 16px;

5 }	5 }

6	6

7 body	7 body

8 {	8 {

9 margin: 0;	9 margin: 0;

10 line-height: 1.5;	10 line-height: 1.5;

11 }	11 }

12	12

	13 /** Provides padding necessary for fixed navbar */

	14 .body-container
	ire 2018/03/13 08:31:36 NIT: We could apply this style directly to the bod NIT: We could apply this style directly to the body element and instead just apply a class to the body where we want it. I don't think I have a strong preference either way, but just bringing it up in case you do. juliandoucette 2018/03/13 12:14:03 Ack. I thought of that last night as I was going t Show quoted text On 2018/03/13 08:31:36, ire wrote: > NIT: We could apply this style directly to the body element and instead just > apply a class to the body where we want it. I don't think I have a strong > preference either way, but just bringing it up in case you do. Ack. I thought of that last night as I was going to sleep XD. Dono why it didn't occur to me before. Will-do.
	15 {

	16 padding-top: 4em;

	17 }

	18

13 #content,	19 #content,

14 #blog	20 #blog

15 {	21 {

16 padding-bottom: 2em;	22 padding-bottom: 2em;

17 padding-top: 4em;

18 }	23 }

19	24

20 a img	25 a img

21 {	26 {

22 border: none;	27 border: none;

23 }	28 }

24	29

25 a:link, a:visited	30 a:link, a:visited

26 {	31 {

27 color: #555;	32 color: #555;

(...skipping 765 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
793 left: 50%;	798 left: 50%;

794 width: 80px;	799 width: 80px;

795 margin-top: -60px;	800 margin-top: -60px;

796 margin-left: -40px;	801 margin-left: -40px;

797 }	802 }

798	803

799 .hide-disclaimer .video-play	804 .hide-disclaimer .video-play

800 {	805 {

801 margin-top: -40px;	806 margin-top: -40px;

802 }	807 }

OLD	NEW

« no previous file with comments | « static/css/index.css ('k') | templates/default.tmpl » ('j') | templates/default.tmpl » ('J')