diff options
| author | stuartmorgan@chromium.org <stuartmorgan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-11 00:22:45 +0000 |
|---|---|---|
| committer | stuartmorgan@chromium.org <stuartmorgan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-11 00:22:45 +0000 |
| commit | f3a08e286d0719a388a56eae5e396fc86c223ddc (patch) | |
| tree | 249fba44ef84b29c11880350bfe0bccea43c0e9a | |
| parent | 9c3ed7e5bdcb33d4c8a80d5c91c750452c5a057a (diff) | |
| download | chromium_src-f3a08e286d0719a388a56eae5e396fc86c223ddc.zip chromium_src-f3a08e286d0719a388a56eae5e396fc86c223ddc.tar.gz chromium_src-f3a08e286d0719a388a56eae5e396fc86c223ddc.tar.bz2 | |
DOMUI Prefs: Fix the add page overlay
Add an explicit list height now that we don't inherit one by default.
Clear list selection when clearing state, so re-opening will give a fresh dialog.
BUG=60842
TEST=Open the Add dialog for startup pages, select something, cancel, and then re-open. The seection should be gone.
Review URL: http://codereview.chromium.org/6155004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70968 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | chrome/browser/resources/options/add_startup_page_overlay.css | 4 | ||||
| -rw-r--r-- | chrome/browser/resources/options/add_startup_page_overlay.js | 1 |
2 files changed, 5 insertions, 0 deletions
diff --git a/chrome/browser/resources/options/add_startup_page_overlay.css b/chrome/browser/resources/options/add_startup_page_overlay.css index a863eb6..7929ba1 100644 --- a/chrome/browser/resources/options/add_startup_page_overlay.css +++ b/chrome/browser/resources/options/add_startup_page_overlay.css @@ -19,6 +19,10 @@ -webkit-box-flex: 1; } +#addStartupRecentPageList { + height: 320px; +} + #addStartupRecentPageList .url { display: none; color: hsl(0, 0%, 70%); diff --git a/chrome/browser/resources/options/add_startup_page_overlay.js b/chrome/browser/resources/options/add_startup_page_overlay.js index 659ef3b..d27f266 100644 --- a/chrome/browser/resources/options/add_startup_page_overlay.js +++ b/chrome/browser/resources/options/add_startup_page_overlay.js @@ -65,6 +65,7 @@ cr.define('options', function() { */ dismissOverlay_: function() { $('addStartupPageURL').value = ''; + $('addStartupRecentPageList').selectionModel.unselectAll(); this.updateAddButtonState_(); OptionsPage.clearOverlays(); }, |