Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(59)

Side by Side Diff: components/dom_distiller/core/javascript/dom_distiller_viewer.js

Issue 1043263004: Basic hookup for distiller feedback (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@rebase-html-feedback
Patch Set: rebase Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 function addToPage(html) { 5 function addToPage(html) {
6 var div = document.createElement('div'); 6 var div = document.createElement('div');
7 div.innerHTML = html; 7 div.innerHTML = html;
8 document.getElementById('content').appendChild(div); 8 document.getElementById('content').appendChild(div);
9 } 9 }
10 10
(...skipping 62 matching lines...) Expand 10 before | Expand all | Expand 10 after
73 * @param noText The i18n text for the feedback answer 'NO'. 73 * @param noText The i18n text for the feedback answer 'NO'.
74 */ 74 */
75 function showFeedbackForm(questionText, yesText, noText) { 75 function showFeedbackForm(questionText, yesText, noText) {
76 document.getElementById('feedbackYes').innerText = yesText; 76 document.getElementById('feedbackYes').innerText = yesText;
77 document.getElementById('feedbackNo').innerText = noText; 77 document.getElementById('feedbackNo').innerText = noText;
78 document.getElementById('feedbackQuestion').innerText = questionText; 78 document.getElementById('feedbackQuestion').innerText = questionText;
79 79
80 document.getElementById('feedbackContainer').style.display = 'block'; 80 document.getElementById('feedbackContainer').style.display = 'block';
81 } 81 }
82 82
83 /**
84 * Send feedback about this distilled article.
85 * @param good True if the feedback was positive, false if negative.
86 */
87 function sendFeedback(good) {
88 var img = document.createElement('img');
89 if (good) {
90 img.src = '/feedbackgood';
91 } else {
92 img.src = '/feedbackbad';
93 }
94 img.style.display = "none";
95 document.body.appendChild(img);
96 }
97
83 // Add a listener to the "View Original" link to report opt-outs. 98 // Add a listener to the "View Original" link to report opt-outs.
84 document.getElementById('showOriginal').addEventListener('click', function(e) { 99 document.getElementById('showOriginal').addEventListener('click', function(e) {
85 var img = document.createElement('img'); 100 var img = document.createElement('img');
86 img.src = "/vieworiginal"; 101 img.src = "/vieworiginal";
87 img.style.display = "none"; 102 img.style.display = "none";
88 document.body.appendChild(img); 103 document.body.appendChild(img);
89 }, true); 104 }, true);
90 105
106 document.getElementById('feedbackYes').addEventListener('click', function(e) {
107 sendFeedback(true);
108 document.getElementById('feedbackContainer').className += " fadeOut";
109 }, true);
110
111 document.getElementById('feedbackNo').addEventListener('click', function(e) {
112 sendFeedback(false);
113 document.getElementById('feedbackContainer').className += " fadeOut";
114 }, true);
115
116 document.getElementById('feedbackContainer').addEventListener('animationend',
117 function(e) {
118 document.getElementById('feedbackContainer').style.display = "none";
119 }, true);
120
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698