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

Unified Diff: chrome/browser/resources/settings/settings_page/settings_section.js

Issue 1508233002: Settings Rewrite: Update comments on related to routing. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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/settings/settings_page/settings_section.js
diff --git a/chrome/browser/resources/settings/settings_page/settings_section.js b/chrome/browser/resources/settings/settings_page/settings_section.js
index fd8da6df3bd5265ee85e28c25759d6dde9e770ac..67cb1d39a2471336db6236175d1c1e016d3e06d8 100644
--- a/chrome/browser/resources/settings/settings_page/settings_section.js
+++ b/chrome/browser/resources/settings/settings_page/settings_section.js
@@ -7,9 +7,12 @@
* 'settings-section' shows a paper material themed section with a header
* which shows its page title.
*
+ * The section name must match the name specified in settings_router.js.
+ * Otherwise, settings-section will not expand and contract correctly.
+ *
* Example:
*
- * <settings-section page-title="[[pageTitle]]">
+ * <settings-section page-title="[[pageTitle]]" section="privacy">
* <!-- Insert your section controls here -->
* </settings-section>
*

Powered by Google App Engine
This is Rietveld 408576698