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

Unified Diff: chrome/browser/resources/welcome/win10/inline/combined.js

Issue 2401853005: Adding client code for new Windows 10 First Run Experience (Closed)
Patch Set: Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/welcome/win10/inline/combined.js
diff --git a/chrome/browser/resources/welcome/win10/inline/combined.js b/chrome/browser/resources/welcome/win10/inline/combined.js
new file mode 100644
index 0000000000000000000000000000000000000000..3b3a723ba154cb33098ce96a3a403e7bf2e05b97
--- /dev/null
+++ b/chrome/browser/resources/welcome/win10/inline/combined.js
@@ -0,0 +1,54 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
tmartino 2016/10/12 22:06:40 We have a lot of dupe code between these JS files
Patrick Monette 2016/10/13 21:30:05 I tried something. What do you think now?
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+cr.define('welcome', function() {
+ 'use strict';
+
+ function injectLocalizedStrings() {
+ $('section-steps--click-edge').innerHTML =
+ loadTimeData.getString('clickEdgeText');
+ $('section-steps--select-chrome').innerHTML =
+ loadTimeData.getString('clickSelectChrome');
+ $('section-steps--right-click').innerHTML =
+ loadTimeData.getString('rightClickText');
+ $('section-steps--pin-taskbar').innerHTML =
+ loadTimeData.getString('pinInstructionText');
+ }
+
+ function toggleSections(sections, screenshotImages) {
tmartino 2016/10/12 22:06:40 |screenshotImages| unused?
Patrick Monette 2016/10/13 21:30:05 Good catch. Removed.
+ sections.forEach(function(section) {
+ section.classList.toggle('section--expanded');
+ });
+ }
+
+ function onSetDefaultBrowser(e) {
+ chrome.send('handleSetDefaultBrowser');
+ }
+
+ function onContinue(e) {
+ chrome.send('handleContinue');
+ }
+
+ function initialize() {
+ injectLocalizedStrings();
+
+ var sections = document.querySelectorAll('.section');
+ var screenshotImages = document.querySelectorAll('.screenshot-image');
+ sections.forEach(function(section) {
+ var sectionHeading = section.querySelector('.section-heading');
+ sectionHeading.addEventListener('click', function() {
+ toggleSections(sections, screenshotImages);
+ });
+ });
+
+ $('set-default-button').addEventListener('click', onSetDefaultBrowser);
+ $('continue-button').addEventListener('click', onContinue);
+ }
+
+ return {
+ initialize: initialize
+ };
+});
+
+document.addEventListener('DOMContentLoaded', welcome.initialize);

Powered by Google App Engine
This is Rietveld 408576698