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

Unified Diff: chrome/browser/resources/bookmark_manager/js/main.js

Issue 472933004: Change bookmark manager header to look more like settings/extensions/history. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: shave Created 6 years, 4 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/bookmark_manager/js/main.js
diff --git a/chrome/browser/resources/bookmark_manager/js/main.js b/chrome/browser/resources/bookmark_manager/js/main.js
index de23c3a95e92266e45df74f8fd8a2827f8ea263a..6716b9e750c29920377a9259d8b303b3c300a0cd 100644
--- a/chrome/browser/resources/bookmark_manager/js/main.js
+++ b/chrome/browser/resources/bookmark_manager/js/main.js
@@ -284,13 +284,6 @@ function setSearch(searchText) {
navigateTo(id);
}
-// Handle the logo button UI.
-// When the user clicks the button we should navigate "home" and focus the list.
-function handleClickOnLogoButton(e) {
- setSearch('');
- $('list').focus();
-}
-
/**
* This returns the user visible path to the folder where the bookmark is
* located.
@@ -1328,7 +1321,7 @@ function continueInitializeBookmarkManager(localizedStrings) {
// when // the user goes back and forward in the history.
window.addEventListener('hashchange', processHash);
- document.querySelector('.header form').onsubmit = function(e) {
+ document.querySelector('header form').onsubmit = function(e) {
setSearch($('term').value);
e.preventDefault();
};
@@ -1338,9 +1331,6 @@ function continueInitializeBookmarkManager(localizedStrings) {
document.querySelector('.summary > button').addEventListener(
'click', handleOrganizeButtonClick);
- document.querySelector('button.logo').addEventListener(
- 'click', handleClickOnLogoButton);
-
document.addEventListener('canExecute', handleCanExecuteForDocument);
document.addEventListener('command', handleCommand);
« no previous file with comments | « chrome/browser/resources/bookmark_manager/css/bmm.css ('k') | chrome/browser/resources/bookmark_manager/main.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698