includes/tool-certification/style.html - Issue 29438582: Issue 5135 - Reduce font size and padding on smaller screens on acceptableads.com

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

Delta Between Two Patch Sets: includes/tool-certification/style.html

Issue 29438582: Issue 5135 - Reduce font size and padding on smaller screens on acceptableads.com (Closed) Base URL: https://hg.adblockplus.org/web.acceptableads.com

Left Patch Set: Handle responsive styling Created June 22, 2017, 3:27 p.m.

Right Patch Set: Fix alignment of hr under headings Created July 6, 2017, 10:25 a.m.

Left:
Right:

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

Jump to:

Left: Side by side diff | Download
Right: Side by side diff | Download

LEFT	RIGHT
1 <style>	1 <style>

2	2

3 .logos,	3 #logos,

4 .bg-accent blockquote	4 #logos-heading
juliandoucette 2017/06/26 10:31:20 .bg-accent was intended to be about background col .bg-accent was intended to be about background colour only. I suggest using another class for this. ire 2017/06/27 08:36:29 Done. Show quoted text On 2017/06/26 10:31:20, juliandoucette wrote: > .bg-accent was intended to be about background colour only. I suggest using > another class for this. Done.
5 {	5 {

6 padding-bottom: 32px;	6 padding-bottom: 32px;

7 }	7 }

8	8

9 .btn-primary
juliandoucette 2017/06/26 10:31:20 See comment in styles/about (it may be unnecessary See comment in styles/about (it may be unnecessary to style this button's margin individually) ire 2017/06/27 08:36:29 Done. Show quoted text On 2017/06/26 10:31:20, juliandoucette wrote: > See comment in styles/about (it may be unnecessary to style this button's margin > individually) Done.
10 {

11 margin-bottom: 32px;

12 }

13

14 @media(min-width: 544px)	9 @media(min-width: 544px)

15 {	10 {

16 .logos	11 #logos

17 {	12 {

18 padding-bottom: 96px;	13 padding-bottom: 96px;

19 }	14 }

20 .bg-accent blockquote	15 #logos-heading

21 {	16 {

22 padding-bottom: 64px;	17 padding-bottom: 64px;

23 }

24 .btn-primary

25 {

26 margin-bottom: 64px;

27 }	18 }

28 }	19 }

29	20

30 @media(min-width: 768px)	21 @media(min-width: 768px)

31 {	22 {

32 .masthead-img .embed	23 .masthead-img .embed

33 {	24 {

34 margin-top: 196px;	25 margin-top: 196px;

35 }	26 }

36 .btn-primary

37 {

38 margin-bottom: 96px;

39 }

40 }	27 }

41	28

42 </style>	29 </style>

LEFT	RIGHT