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

Unified Diff: chrome/browser/resources/settings/settings_menu/settings_menu.html

Issue 1136273002: Convert cr-settings-menu to Polymer 0.8 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@AddPaperItem
Patch Set: rebase Created 5 years, 7 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/settings/settings_menu/settings_menu.html
diff --git a/chrome/browser/resources/settings/settings_menu/settings_menu.html b/chrome/browser/resources/settings/settings_menu/settings_menu.html
index bbf3aad7a93089cc7a498e2a9b4d8204b4a90ec4..fd502b275b75cc731d8217e9be64e95e2275b865 100644
--- a/chrome/browser/resources/settings/settings_menu/settings_menu.html
+++ b/chrome/browser/resources/settings/settings_menu/settings_menu.html
@@ -1,22 +1,20 @@
-<link rel="import" href="chrome://resources/polymer/core-menu/core-menu.html">
-<link rel="import" href="chrome://resources/polymer/paper-item/paper-item.html">
-<link rel="import" href="settings_menu_style.html">
+<link rel="import" href="chrome://resources/polymer/v0_8/polymer/polymer.html">
+<link rel="import" href="chrome://resources/polymer/v0_8/iron-icon/iron-icon.html">
+<link rel="import" href="chrome://resources/polymer/v0_8/iron-icons/iron-icons.html">
+<link rel="import" href="chrome://resources/polymer/v0_8/paper-menu/paper-menu.html">
+<link rel="import" href="chrome://resources/polymer/v0_8/paper-item/paper-icon-item.html">
-<polymer-element name="cr-settings-menu">
+<dom-module id="cr-settings-menu">
+ <link rel="import" type="css" href="settings_menu.css">
<template>
- <core-style ref="settingsMenuStyle"></core-style>
- <!-- TODO(jlklein): selectedAttribute here is a workaround for
- https://github.com/Polymer/polymer/issues/946. Remove when core-menu
- is updated. -->
- <core-menu selected="{{selectedId}}" valueattr="data-id"
- selectedAttribute="">
- <template repeat="{{pages}}">
- <paper-item data-id="{{PAGE_ID}}">
- <core-icon icon="{{icon}}"></core-icon>
- {{pageTitle}}
- </paper-item>
+ <paper-menu selected="{{selectedId}}" attr-for-selected="data-id">
+ <template is="x-repeat" items="[[pages]]">
+ <paper-icon-item data-id$="[[item.PAGE_ID]]" icon-width="34px">
+ <iron-icon icon="[[item.icon]]" item-icon></iron-icon>
+ <span>[[item.pageTitle]]</span>
+ </paper-icon-item>
</template>
- </core-menu>
+ </paper-menu>
</template>
<script src="settings_menu.js"></script>
-</polymer-element>
+</dom-module>

Powered by Google App Engine
This is Rietveld 408576698