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

Delta Between Two Patch Sets: share.html

Issue 10990029: Made share page compatible with IE9 and IE10 (Closed)
Left Patch Set: Created June 17, 2013, 1:43 p.m.
Right Patch Set: Created June 17, 2013, 2:58 p.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
« no previous file with change/comment | « no previous file | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 1
2 <!DOCTYPE html> 2 <!DOCTYPE html>
3 <html> 3 <html>
4 <head> 4 <head>
5 <title>Teilen</title> 5 <title>Teilen</title>
6 <style> 6 <style>
7 body 7 body
8 { 8 {
9 width: 650px; 9 width: 650px;
10 margin: auto; 10 margin: auto;
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
76 } 76 }
77 </style> 77 </style>
78 <script><!-- 78 <script><!--
79 (function() 79 (function()
80 { 80 {
81 function initFacebook(locale) 81 function initFacebook(locale)
82 { 82 {
83 var map = { 83 var map = {
84 "en": "en_US", 84 "en": "en_US",
85 "he": "he_IL", 85 "he": "he_IL",
86 "ko": "ko_KR" 86 "ko": "ko_KR",
Felix Dahlke 2013/06/17 14:03:11 I think the trailing comma was actually intended b
Thomas Greiner 2013/06/17 14:59:57 Done.
87 }; 87 };
88 if (locale in map) 88 if (locale in map)
89 locale = map[locale]; 89 locale = map[locale];
90 90
91 // HACK AHEAD 91 // HACK AHEAD
92 if (locale.length == 2) 92 if (locale.length == 2)
93 locale = locale + "_" + locale.toUpperCase(); 93 locale = locale + "_" + locale.toUpperCase();
94 94
95 (function(d, s, id) { 95 (function(d, s, id) {
96 var js, fjs = d.getElementsByTagName(s)[0]; 96 var js, fjs = d.getElementsByTagName(s)[0];
(...skipping 20 matching lines...) Expand all
117 function sendDimensions() 117 function sendDimensions()
118 { 118 {
119 if (window.parent === window) 119 if (window.parent === window)
120 return; 120 return;
121 121
122 var dimensions = { 122 var dimensions = {
123 width: document.body.offsetWidth, 123 width: document.body.offsetWidth,
124 height: document.body.offsetHeight 124 height: document.body.offsetHeight
125 }; 125 };
126 126
127 // IE doesn't support data other than strings yet
127 if (/MSIE/.test(navigator.appVersion)) 128 if (/MSIE/.test(navigator.appVersion))
Felix Dahlke 2013/06/17 14:03:11 This definitely deserves a comment, we're hacking
Thomas Greiner 2013/06/17 14:59:57 Done.
128 dimensions = JSON.stringify(dimensions); 129 dimensions = JSON.stringify(dimensions);
129 130
130 window.parent.postMessage(dimensions, "*"); 131 window.parent.postMessage(dimensions, "*");
131 } 132 }
132 133
133 window.addEventListener("DOMContentLoaded", function() 134 window.addEventListener("DOMContentLoaded", function()
134 { 135 {
135 var locale = window.location.pathname.replace(/^\/+/, "").replace(/\/.*/, "" ); 136 var locale = window.location.pathname.replace(/^\/+/, "").replace(/\/.*/, "" );
136 if (/^\w\w(_\w\w)?/.test(locale)) 137 if (/^\w\w(_\w\w)?/.test(locale))
137 { 138 {
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
172 </div> 173 </div>
173 </div> 174 </div>
174 175
175 <div id="gplus"> 176 <div id="gplus">
176 <div class="widget-container"> 177 <div class="widget-container">
177 <div class="g-plus" data-href="https://plus.google.com/110020691898167279887 " data-width="500" data-rel="publisher"></div> 178 <div class="g-plus" data-href="https://plus.google.com/110020691898167279887 " data-width="500" data-rel="publisher"></div>
178 </div> 179 </div>
179 </div> 180 </div>
180 </body> 181 </body>
181 </html> 182 </html>
LEFTRIGHT
« no previous file | no next file » | Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Toggle Comments ('s')

Powered by Google App Engine
This is Rietveld