Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: pages/press.html

Issue 29369432: Issue 4685 - Update images on eyeo.com (Closed)
Patch Set: See notes in comments Created Jan. 11, 2017, 5:34 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« pages/jobs/index.tmpl ('K') | « pages/jobs/index.tmpl ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pages/press.html
===================================================================
--- a/pages/press.html
+++ b/pages/press.html
@@ -1,19 +1,37 @@
title=Press
-<script src="/js/jquery.ui.core.min.js"></script>
-<script src="/js/jquery.ui.widget.min.js"></script>
-<script src="/js/jquery.ui.tabs.min.js"></script>
-<script>
- jQuery(function()
- {
- jQuery("#tabs").tabs();
- });
-</script>
+<head>
+ <style>
+ #media-gallery img
+ {
+ vertical-align: middle;
juliandoucette 2017/01/11 17:48:09 Note: a line-height and img vertical-align vertica
juliandoucette 2017/01/12 15:32:38 Acknowledged.
+ max-height: 150px;
+ max-width: 150px;
+ }
+
+ #media-gallery a
+ {
+ display: inline-block;
+ line-height: 150px;
+ width: 150px;
+ }
+ </style>
+ <script src="/js/jquery.ui.core.min.js"></script>
+ <script src="/js/jquery.ui.widget.min.js"></script>
+ <script src="/js/jquery.ui.tabs.min.js"></script>
+ <script>
juliandoucette 2017/01/11 17:48:09 Note: Moved from include into page head.
juliandoucette 2017/01/12 15:32:38 Acknowledged.
+ jQuery(function()
+ {
+ jQuery("#tabs").tabs();
+ jQuery("#media a").attr("download", "");
+ });
+ </script>
+</head>
<header class="page-title">
<h1>Press</h1>
</header>
<div id="press-info" class="content-block columns-container one-sidebar">
<div class="column sidebar-left">
<div class="contact-person">
@@ -40,17 +58,17 @@
<? include press/faq ?>
</div>
<div id="facts">
<? include press/facts ?>
</div>
<div id="media">
- <? include press/media ?>
+ <? include press/media/section ?>
</div>
<div id="releases">
<? include press/releases ?>
</div>
</div>
</div>
</div>
« pages/jobs/index.tmpl ('K') | « pages/jobs/index.tmpl ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld