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

Unified Diff: chrome/browser/resources/md_user_manager/user_manager_pages.js

Issue 1722843002: MD user manager (html/js/css for create profile flow) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/md_user_manager/user_manager_pages.js
diff --git a/chrome/browser/resources/settings/site_settings/all_sites.js b/chrome/browser/resources/md_user_manager/user_manager_pages.js
similarity index 31%
copy from chrome/browser/resources/settings/site_settings/all_sites.js
copy to chrome/browser/resources/md_user_manager/user_manager_pages.js
index 9d3513c7dc01c7778c626fe23a431414fa459165..f3659485344a907213d58ca4fbbaa35990620632 100644
--- a/chrome/browser/resources/settings/site_settings/all_sites.js
+++ b/chrome/browser/resources/md_user_manager/user_manager_pages.js
@@ -4,40 +4,35 @@
/**
* @fileoverview
- * 'all-sites' is the polymer element for showing the list of all sites under
- * Site Settings.
+ * 'user-manager-pages' is the element that controls paging in the user manager
+ * screen.
Dan Beam 2016/02/25 16:44:39 don't wrap after @fileoverview
Moe 2016/03/02 18:03:18 Done.
*
- * @group Chrome Settings Elements
- * @element all-sites
+ * Example:
+ *
+ * <user-manager-pages selected-page="sample-page-id">
+ * <sample-page id="sample-page-id"></sample-page>
+ * ... other pages ...
+ * </user-manager-pages>
+ *
+ * @element user-manager-pages
Dan Beam 2016/02/25 16:44:39 just remove Example: to the end of this comment
Moe 2016/03/02 18:03:18 Done.
*/
Polymer({
- is: 'all-sites',
-
- behaviors: [SiteSettingsBehavior],
+ is: 'user-manager-pages',
properties: {
/**
- * Preferences state.
- */
- prefs: {
- type: Object,
- notify: true,
- },
-
- /**
- * The current active route.
+ * id of the currently selected page.
Dan Beam 2016/02/25 16:44:39 ID
Moe 2016/03/02 18:03:18 Done.
+ * @type {boolean}
*/
- currentRoute: {
- type: Object,
- notify: true,
- },
-
- /**
- * The origin that was selected by the user in the dropdown list.
- */
- selectedOrigin: {
+ selectedPage: {
type: String,
- notify: true,
- },
+ }
},
+
+ /** @override */
+ attached: function() {
+ this.addEventListener('change-page', function(e) {
+ this.selectedPage = e.detail.page;
+ }.bind(this));
+ }
});

Powered by Google App Engine
This is Rietveld 408576698