Delta Between Two Patch Sets: pages/donate.html
Issue 29334044:
Issue 219 - Donation page is broken in IE8 (Closed)
Left Patch Set: Added polyfills and fixed spacing
Created Jan. 28, 2016, 1:58 p.m.
Right Patch Set: Uploading diff against tip instead of revision
Created Jan. 28, 2016, 3:53 p.m.
| Left: | ||
| Right: |
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with change/comment
|
« no previous file
|
static/js/vendor/ie8.js » ('j')
|
no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| LEFT | RIGHT |
|---|---|
| 1 title=Donate to Adblock Plus | 1 title=Donate to Adblock Plus |
| 2 | 2 |
| 3 <head> | 3 <head> |
| 4 <style type="text/css"> | 4 <style type="text/css"> |
| 5 h2 | 5 h2 |
| 6 { | 6 { |
| 7 padding-top: 20px; | 7 padding-top: 20px; |
| 8 border-top: 1px solid #ccc; | 8 border-top: 1px solid #ccc; |
| 9 } | 9 } |
| 10 | 10 |
| (...skipping 424 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 435 var currencyLabel = selectedCurrency.parentNode[textAttr]; | 435 var currencyLabel = selectedCurrency.parentNode[textAttr]; |
| 436 var currencyLabels = document.querySelectorAll(".currency-label"); | 436 var currencyLabels = document.querySelectorAll(".currency-label"); |
| 437 for (var i = 0; i < currencyLabels.length; i++) | 437 for (var i = 0; i < currencyLabels.length; i++) |
| 438 currencyLabels[i][textAttr] = currencyLabel; | 438 currencyLabels[i][textAttr] = currencyLabel; |
| 439 ensureMinPrice(); | 439 ensureMinPrice(); |
| 440 } | 440 } |
| 441 | 441 |
| 442 function initializeDonationForm() | 442 function initializeDonationForm() |
| 443 { | 443 { |
| 444 formValues.name = getText("i18n_name"); | 444 formValues.name = getText("i18n_name"); |
| 445 document.getElementById('form-payment').addEventListener("click", onSelectPa yment); | 445 document.getElementById("form-payment").addEventListener("click", onSelectPa yment, true); |
|
Thomas Greiner
2016/01/28 15:23:37
Detail: Please use double quotes instead of single
juliandoucette
2016/01/28 15:50:44
Done.
| |
| 446 document.getElementById("form-currency").addEventListener("click", updateCur rency); | 446 document.getElementById("form-currency").addEventListener("click", updateCur rency, true); |
| 447 document.getElementById("form").addEventListener("submit", onSubmitDonation) ; | 447 document.getElementById("form").addEventListener("submit", onSubmitDonation, false); |
| 448 updateCurrency(); | 448 updateCurrency(); |
| 449 } | 449 } |
| 450 | 450 |
| 451 function onSelectPayment(event) | 451 function onSelectPayment(event) |
| 452 { | 452 { |
| 453 if (event.target.tagName != "BUTTON") | 453 if (event.target.tagName != "BUTTON") |
| 454 return; | 454 return; |
| 455 event.preventDefault(); | 455 event.preventDefault(); |
| 456 document.getElementById("form").className = (event.target || event.srcElemen t).className; | 456 document.getElementById("form").className = event.target.className; |
|
Thomas Greiner
2016/01/28 15:23:37
According to https://github.com/WebReflection/ie8
juliandoucette
2016/01/28 15:50:44
Done.
| |
| 457 ensureMinPrice(); | 457 ensureMinPrice(); |
| 458 } | 458 } |
| 459 | 459 |
| 460 function onSubmitDonation(event) | 460 function onSubmitDonation(event) |
| 461 { | 461 { |
| 462 event.preventDefault(); | 462 event.preventDefault(); |
| 463 var provider = providers[document.getElementById("form").className]; | 463 var provider = providers[document.getElementById("form").className]; |
| 464 if (provider) | 464 if (provider) |
| 465 provider(); | 465 provider(); |
| 466 } | 466 } |
| 467 | 467 |
| 468 document.addEventListener("DOMContentLoaded", initializeDonationForm); | 468 document.addEventListener("DOMContentLoaded", initializeDonationForm, false); |
|
Thomas Greiner
2016/01/28 15:23:37
Usually, we explicitly specify the last parameter
juliandoucette
2016/01/28 15:50:44
Done.
| |
| 469 | 469 |
| 470 })(); | 470 })(); |
| 471 | 471 |
| 472 // --> | 472 // --> |
| 473 </script> | 473 </script> |
| 474 </head> | 474 </head> |
| 475 <span class="hidden" id="i18n_name">{{s1 Adblock Plus Contribution}}</span> | 475 <span class="hidden" id="i18n_name">{{s1 Adblock Plus Contribution}}</span> |
| 476 | 476 |
| 477 <p> | 477 <p> |
| 478 {{s2 Adblock Plus is - and will always be - free. Our mission is to provide us ers with tools that allow them to gain control over their Internet browsing and protect them from unwanted and malicious elements. | 478 {{s2 Adblock Plus is - and will always be - free. Our mission is to provide us ers with tools that allow them to gain control over their Internet browsing and protect them from unwanted and malicious elements. |
| (...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 545 </div> | 545 </div> |
| 546 <div id="recurrent-cancellation"> | 546 <div id="recurrent-cancellation"> |
| 547 * {{s12 To cancel your monthly payment, click the link below that corres ponds to how you donated:}} | 547 * {{s12 To cancel your monthly payment, click the link below that corres ponds to how you donated:}} |
| 548 <ul> | 548 <ul> |
| 549 <li><a href="https://www.paypal.com/cgi-bin/webscr?cmd=_manage-paylist " target="_blank">{{s13 I donated with PayPal}}</a></li> | 549 <li><a href="https://www.paypal.com/cgi-bin/webscr?cmd=_manage-paylist " target="_blank">{{s13 I donated with PayPal}}</a></li> |
| 550 </ul> | 550 </ul> |
| 551 </div> | 551 </div> |
| 552 </div> | 552 </div> |
| 553 </div> | 553 </div> |
| 554 </form> | 554 </form> |
| LEFT | RIGHT |
« no previous file
|
static/js/vendor/ie8.js » ('j')
|
Toggle Intra-line Diffs ('i')
|
Expand Comments ('e')
|
Collapse Comments ('c')
|
Toggle Comments ('s')
