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

Unified Diff: chrome/browser/resources/chromeos/wallpaper_manager/main.html

Issue 12769017: Add delete context menu for custom wallpapers and some ui tweaks (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 years, 9 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/chromeos/wallpaper_manager/main.html
diff --git a/chrome/browser/resources/chromeos/wallpaper_manager/main.html b/chrome/browser/resources/chromeos/wallpaper_manager/main.html
index 11243d4c85e6a5c16eaeabed726864121ece4e46..dc7d4823314252ac74c036dfe812c83dac9630a1 100644
--- a/chrome/browser/resources/chromeos/wallpaper_manager/main.html
+++ b/chrome/browser/resources/chromeos/wallpaper_manager/main.html
@@ -9,6 +9,8 @@ found in the LICENSE file.
<head>
<title>Wallpaper Picker</title>
<link rel="stylesheet" href="../../../../../ui/webui/resources/css/widgets.css">
+ <link rel="stylesheet" href="../../../../../ui/webui/resources/css/menu.css"></link>
+
<link rel="stylesheet" href="css/wallpaper_manager.css">
<script src="js/main_scripts.js"></script>
<if expr="0">
@@ -29,6 +31,13 @@ found in the LICENSE file.
<script src="../../../../../ui/webui/resources/js/cr/ui/list_selection_controller.js"></script>
<script src="../../../../../ui/webui/resources/js/cr/ui/list.js"></script>
<script src="../../../../../ui/webui/resources/js/cr/ui/grid.js"></script>
+ <script src="../../../../../ui/webui/resources/js/cr/ui/command.js"></script>
+ <script src="../../../../../ui/webui/resources/js/cr/ui/position_util.js"></script>
+ <script src="../../../../../ui/webui/resources/js/cr/ui/menu_item.js"></script>
+ <script src="../../../../../ui/webui/resources/js/cr/ui/menu.js"></script>
+ <script src="../../../../../ui/webui/resources/js/cr/ui/menu_button.js"></script>
+ <script src="../../../../../ui/webui/resources/js/cr/ui/context_menu_handler.js"></script>
+
<script src="js/progress_manager.js"></script>
<script src="js/wallpaper_directories.js"></script>
<script src="js/wallpaper_images_grid.js"></script>
@@ -38,6 +47,15 @@ found in the LICENSE file.
</head>
<body i18n-values=".style.fontFamily:webFontFamily;
.style.fontSize:webFontSize">
+ <commands>
+ <command id="delete" i18n-values="label:deleteCommandLabel"
+ shortcut="U+007F"></command>
+ </commands>
+
+ <menu id="wallpaper-context-menu" class="chrome-menu" visibleif="custom">
+ <menuitem command="#delete"></menuitem>
+ </menu>
+
<div id="error-container" class="overlay-container" hidden>
<div class="page">
<div id="close-error" class="close-overlay"></div>
@@ -80,7 +98,7 @@ found in the LICENSE file.
<div class="progress-track"></div>
</div>
<div class="bottom-bar">
- <div id="online-wallpaper-attribute">
+ <div id="online-wallpaper-attribute" visibleif="online">
<img id="attribute-image" hidden>
<div id="wallpaper-attribute" hidden>
<div>

Powered by Google App Engine
This is Rietveld 408576698