diff options
author | dhg@chromium.org <dhg@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-02-16 20:00:33 +0000 |
---|---|---|
committer | dhg@chromium.org <dhg@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-02-16 20:00:33 +0000 |
commit | 784eb2c88f163d8d0c28e5da6c69ba703d662568 (patch) | |
tree | 88c6c638b5c9f0248f2428c6f776b6dbccfb8cf1 /chrome | |
parent | 9f71041881af6f09015b84402a1f2986b88b572a (diff) | |
download | chromium_src-784eb2c88f163d8d0c28e5da6c69ba703d662568.zip chromium_src-784eb2c88f163d8d0c28e5da6c69ba703d662568.tar.gz chromium_src-784eb2c88f163d8d0c28e5da6c69ba703d662568.tar.bz2 |
Adding a confirm before deleting a file.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/600107
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39125 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome')
-rw-r--r-- | chrome/browser/browser_resources.grd | 2 | ||||
-rw-r--r-- | chrome/browser/resources/filebrowse.html | 52 |
2 files changed, 52 insertions, 2 deletions
diff --git a/chrome/browser/browser_resources.grd b/chrome/browser/browser_resources.grd index 7da7632..dfb9438 100644 --- a/chrome/browser/browser_resources.grd +++ b/chrome/browser/browser_resources.grd @@ -1,6 +1,6 @@ <?xml version="1.0" encoding="UTF-8"?> <!-- This comment is only here because changes to resources are not picked up -without changes to the corresponding grd file. dopeeqd --> +without changes to the corresponding grd file. dhg --> <grit latest_public_release="0" current_release="1"> <outputs> <output filename="grit/browser_resources.h" type="rc_header"> diff --git a/chrome/browser/resources/filebrowse.html b/chrome/browser/resources/filebrowse.html index d6595a2..4edc7f2 100644 --- a/chrome/browser/resources/filebrowse.html +++ b/chrome/browser/resources/filebrowse.html @@ -35,6 +35,30 @@ div.header { cursor: pointer; } +.deleteYes { + width: 40px; + height: 30px; + float: left; + text-decoration: underline; + color: blue; +} + +.deleteNo { + width: 40px; + height: 30px; + float: left; + text-decoration: underline; + color: blue; +} + +.confirmdelete { + text-align: center; + border: 1px solid rgba(0, 0, 0, 0.25); + border-radius: 2px; + height: 36px; + padding: 4px; +} + a.iconlink { display: block; /* font-family: helvetica; */ @@ -800,6 +824,32 @@ function deleteFile(path) { chrome.send('deleteFile', [path]); }; +function removeDeleteConfirm(path) { + var element = menus[path]; + element.firstChild.removeChild(element.firstChild.lastChild); +}; + +function deleteFileConfirm(path) { + var askingDiv = document.createElement('div'); + askingDiv.className = 'confirmdelete'; + askingDiv.textContent = 'Are you sure?'; + var yes = document.createElement('div'); + yes.className = 'deleteYes'; + yes.textContent = 'Yes'; + yes.onclick = new Function('deleteFile("' + path + '")'); + var no = document.createElement('div'); + no.onclick = new Function('removeDeleteConfirm("' + path + '")'); + no.textContent = 'Cancel'; + no.className = 'deleteNo'; + askingDiv.appendChild(yes); + askingDiv.appendChild(no); + var element = menus[path]; + if (element) { + element.firstChild.appendChild(askingDiv); + } + window.event.stopPropagation(); +}; + function createFolderTyping(elementId) { if (window.event.keyCode == 13) { createFolder(elementId); @@ -1238,7 +1288,7 @@ function createNewItem(title, path, isDirectory) { var deleteitem = document.createElement('div'); deleteitem.textContent = 'Delete'; deleteitem.className = 'menuitem'; - deleteitem.onclick = new Function('deleteFile("' + path +'")'); + deleteitem.onclick = new Function('deleteFileConfirm("' + path +'")'); menu.appendChild(deleteitem); menuicon.align = 'right'; menuicon.className = 'menuicon'; |