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

Side by Side 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 unified diff | Download patch
OLDNEW
(Empty)
1 // 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?
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 cr.define('welcome', function() {
6 'use strict';
7
8 function injectLocalizedStrings() {
9 $('section-steps--click-edge').innerHTML =
10 loadTimeData.getString('clickEdgeText');
11 $('section-steps--select-chrome').innerHTML =
12 loadTimeData.getString('clickSelectChrome');
13 $('section-steps--right-click').innerHTML =
14 loadTimeData.getString('rightClickText');
15 $('section-steps--pin-taskbar').innerHTML =
16 loadTimeData.getString('pinInstructionText');
17 }
18
19 function toggleSections(sections, screenshotImages) {
tmartino 2016/10/12 22:06:40 |screenshotImages| unused?
Patrick Monette 2016/10/13 21:30:05 Good catch. Removed.
20 sections.forEach(function(section) {
21 section.classList.toggle('section--expanded');
22 });
23 }
24
25 function onSetDefaultBrowser(e) {
26 chrome.send('handleSetDefaultBrowser');
27 }
28
29 function onContinue(e) {
30 chrome.send('handleContinue');
31 }
32
33 function initialize() {
34 injectLocalizedStrings();
35
36 var sections = document.querySelectorAll('.section');
37 var screenshotImages = document.querySelectorAll('.screenshot-image');
38 sections.forEach(function(section) {
39 var sectionHeading = section.querySelector('.section-heading');
40 sectionHeading.addEventListener('click', function() {
41 toggleSections(sections, screenshotImages);
42 });
43 });
44
45 $('set-default-button').addEventListener('click', onSetDefaultBrowser);
46 $('continue-button').addEventListener('click', onContinue);
47 }
48
49 return {
50 initialize: initialize
51 };
52 });
53
54 document.addEventListener('DOMContentLoaded', welcome.initialize);
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698