Chromium Code Reviews| Index: chrome/browser/resources/sync_internals/events.js |
| diff --git a/chrome/browser/resources/sync_internals/events.js b/chrome/browser/resources/sync_internals/events.js |
| index 0cda0860a6c521b6e6e31fc79c06bab09677f89f..8ff80bd66eae39b536fc241c2fb906c33c012d74 100644 |
| --- a/chrome/browser/resources/sync_internals/events.js |
| +++ b/chrome/browser/resources/sync_internals/events.js |
| @@ -3,7 +3,17 @@ |
| // found in the LICENSE file. |
| (function() { |
| -function makeLogEntryNode(entry) { |
| +'use strict'; |
| +var toggleDisplayDetails = function() { |
| + var detailsNode = this.parentNode.getElementsByClassName('details')[0]; |
| + if (detailsNode.classList.contains('hidden')) { |
| + detailsNode.classList.remove('hidden'); |
| + } else { |
| + detailsNode.classList.add('hidden'); |
| + } |
| +} |
|
kjiwa
2013/11/05 00:21:56
As a minor note this function can be generalized i
|
| + |
| +var makeLogEntryNode = function(entry) { |
| var timeNode = document.createElement('td'); |
| timeNode.textContent = entry.date; |
| @@ -15,7 +25,13 @@ function makeLogEntryNode(entry) { |
| var details = document.createElement('pre'); |
| details.textContent = JSON.stringify(entry.details, null, 2); |
| + details.className = 'details'; |
| + details.classList.add('hidden'); |
|
kjiwa
2013/11/05 00:21:56
For consistency you may want to use:
details.cl
|
| + var detailsToggleButton = document.createElement('button'); |
| + detailsToggleButton.addEventListener('click', toggleDisplayDetails, false); |
| + detailsToggleButton.textContent = 'Show/Hide Details'; |
| var detailsNode = document.createElement('td'); |
| + detailsNode.appendChild(detailsToggleButton); |
| detailsNode.appendChild(details); |
| var node = document.createElement('tr'); |