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

Side by Side Diff: chrome/browser/resources/md_user_manager/control_bar.js

Issue 1630903002: material design user manager with create profile flow (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed Dan's comments Created 4 years, 10 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.
2 * Use of this source code is governed by a BSD-style license that can be
3 * found in the LICENSE file.
4 */
5
6 /**
7 * @fileoverview
8 * 'control-bar' is the horizontal bar at the bottom of the user manager screen.
9 *
10 * @element control-bar
11 */
12 Polymer({
13 is: 'control-bar',
14
15 properties: {
16
Dan Beam 2016/02/12 23:42:24 nit: remove \n
Moe 2016/02/20 00:20:59 Done.
17 /**
18 * True if |Browse as Guest| button is displayed.
19 * @type {boolean}
20 */
21 showGuest: {
22 type: Boolean,
23 value: false
24 },
25
26 /**
27 * True if |Add Person| button is displayed.
28 * @type {boolean}
29 */
30 showAddPerson: {
31 type: Boolean,
32 value: false
33 }
34 },
35
36 /**
37 * Handler for |Browse as Guest| button click event.
38 * @param {!Event} event
39 * @private
40 */
41 handleGuestClick_: function(event) {
42 chrome.send('launchGuest');
Dan Beam 2016/02/12 23:42:24 i would recommend abstracting this away so you can
Moe 2016/02/20 00:20:59 Makes sense. Done!
43 },
44
45 /**
46 * Handler for |Add Person| button click event.
47 * @param {!Event} event
48 * @private
49 */
50 handleAddUserClick_: function(event) {
51 // Event is caught by user-manager-pages.
52 this.fire('change-page', {page: 'create-user-page'});
53 }
54 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698