diff options
author | satorux@chromium.org <satorux@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-05 09:17:20 +0000 |
---|---|---|
committer | satorux@chromium.org <satorux@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-05 09:17:20 +0000 |
commit | 80ee4c59ab2490b5008a9424558d949693411d8a (patch) | |
tree | 1bb1ab960dedd5aecee7105a2d3b9ee0839f2d23 /chrome/browser/resources/filebrowse.html | |
parent | 25caac1aa71125f55bdd694218c16d03668e5d71 (diff) | |
download | chromium_src-80ee4c59ab2490b5008a9424558d949693411d8a.zip chromium_src-80ee4c59ab2490b5008a9424558d949693411d8a.tar.gz chromium_src-80ee4c59ab2490b5008a9424558d949693411d8a.tar.bz2 |
Fix text layout of the delete menu in the file browser.
The layout of the delete menu is bad for Japanese, and probably
other languages as well.
TEST=confirmed the delete menu is nicely layouted for English and Japanese.
BUG=chromium-os:10589
Review URL: http://codereview.chromium.org/6000012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70494 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/resources/filebrowse.html')
-rw-r--r-- | chrome/browser/resources/filebrowse.html | 24 |
1 files changed, 14 insertions, 10 deletions
diff --git a/chrome/browser/resources/filebrowse.html b/chrome/browser/resources/filebrowse.html index f6a5050..e6fb44e 100644 --- a/chrome/browser/resources/filebrowse.html +++ b/chrome/browser/resources/filebrowse.html @@ -61,18 +61,19 @@ div.header { from(#D0DAF8), to(#A6BAF7)); } +.deleteYesNoContainer { + display: -webkit-box; + -webkit-box-align: center; + -webkit-box-pack: center; +} .deleteYes { - width: 40px; - height: 30px; - float: left; + margin: 4px; text-decoration: underline; color: blue; } .deleteNo { - width: 40px; - height: 30px; - float: left; + margin: 4px; text-decoration: underline; color: blue; } @@ -81,7 +82,7 @@ div.header { text-align: center; border: 1px solid rgba(0, 0, 0, 0.25); border-radius: 2px; - height: 36px; + min-height: 36px; padding: 4px; } @@ -541,7 +542,7 @@ div.fullcontainer { .menu { top: 14px; right: 2px; - width: 100px; + width: 180px; -webkit-box-shadow: rgba(0, 0, 0, 0.3) 0px 3px 3px; border-bottom-left-radius: 4px 4px; border-bottom-right-radius: 4px 4px; @@ -1111,6 +1112,8 @@ function deleteFileConfirm(path) { var askingDiv = document.createElement('div'); askingDiv.className = 'confirmdelete'; askingDiv.textContent = localStrings.getString('confirmdelete'); + yesNoDiv = document.createElement('div'); + yesNoDiv.className = 'deleteYesNoContainer'; var yes = document.createElement('div'); yes.className = 'deleteYes'; yes.textContent = localStrings.getString('confirmyes'); @@ -1119,8 +1122,9 @@ function deleteFileConfirm(path) { no.onclick = partial(removeDeleteConfirm, path); no.textContent = localStrings.getString('confirmcancel'); no.className = 'deleteNo'; - askingDiv.appendChild(yes); - askingDiv.appendChild(no); + yesNoDiv.appendChild(yes); + yesNoDiv.appendChild(no); + askingDiv.appendChild(yesNoDiv); element.firstChild.appendChild(askingDiv); askingDiv.scrollIntoView(); window.event.stopPropagation(); |