includes/index/style.html - Issue 29499714: Issue 4912 - Update media coverage links on eyeo.com

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: includes/index/style.html

Issue 29499714: Issue 4912 - Update media coverage links on eyeo.com (Closed) Base URL: https://hg.adblockplus.org/web.eyeo.com

Patch Set: Created July 27, 2017, 4:46 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
(Empty)
	1 <style>

	2

	3 #media-links

	4 {

	5 text-align: center;

	6 }

	7

	8 #media-links img

	9 {

	10 max-height: 30px;

	11 max-width: 100%;
	juliandoucette 2017/07/31 20:00:28 What does this do? What does this do? ire 2017/08/01 10:04:05 On smaller screens, the image shouldn't expand pas Show quoted text On 2017/07/31 20:00:28, juliandoucette wrote: > What does this do? On smaller screens, the image shouldn't expand past 100%
	12 }

	13

	14 #media-links a

	15 {

	16 display: inline-block;

	17 position: relative;

	18 margin: 0px 10px 20px;

	19 }

	20

	21 #media-links a:after,

	22 #media-links a:after

	23 {

	24 content: "";

	25 background-image: url("/images/icon-external-link.svg");

	26 background-size: 20px;
	juliandoucette 2017/07/31 20:00:28 NIT: SVG background-image and background-size do n NIT: SVG background-image and background-size do not work in IE 8. Please avoid if easy (we don't have to support IE8 on eyeo.com, but it may not hurt - we can ask Jessica how many IE 8 visitors we get. I don't remember if we have done this before). ire 2017/08/01 10:04:05 Okay. I'm going to change this to using an actual Show quoted text On 2017/07/31 20:00:28, juliandoucette wrote: > NIT: SVG background-image and background-size do not work in IE 8. Please avoid > if easy (we don't have to support IE8 on http://eyeo.com, but it may not hurt - we can > ask Jessica how many IE 8 visitors we get. I don't remember if we have done this > before). Okay. I'm going to change this to using an actual image in the HTML.
	27 position: absolute;

	28 top: 50%;

	29 left: 50%;

	30 height: 20px;

	31 width: 20px;

	32 margin-left: -10px;

	33 margin-top: -12px;

	34 opacity: 0;

	35 }

	36

	37 #media-links a:hover:after,

	38 #media-links a:focus:after

	39 {

	40 opacity: 1;

	41 }

	42

	43 #media-links a:hover img,

	44 #media-links a:focus img

	45 {

	46 opacity: 0.2;

	47 }

	48

	49 </style>

OLD	NEW

« includes/index/media.html ('K') | « includes/index/media.md ('k') | pages/index.html » ('j') | no next file with comments »