diff options
author | mrossetti@chromium.org <mrossetti@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-20 00:47:35 +0000 |
---|---|---|
committer | mrossetti@chromium.org <mrossetti@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-20 00:47:35 +0000 |
commit | 5110a0f6e1e6f16891a16320e74bbd5c400c3c38 (patch) | |
tree | 169cd4a4de6ec62e22ae6ce68d0a2a3a923f333e /chrome/browser/status_icons | |
parent | 358b346689684f1563accc499879e605e9463953 (diff) | |
download | chromium_src-5110a0f6e1e6f16891a16320e74bbd5c400c3c38.zip chromium_src-5110a0f6e1e6f16891a16320e74bbd5c400c3c38.tar.gz chromium_src-5110a0f6e1e6f16891a16320e74bbd5c400c3c38.tar.bz2 |
Adjustments to bookmark bar folder menu placement issues.
1) Sub-menus should expose as much as possible without needing to be scrolled up when the dock is showing at the bottom.
2) Fix how menu bottom is misplaced and window is foreshortened when that last scroll-up is performed (for a menu which fits on the screen after being completely scrolled up).
3) Align the right edge of the menu with the right edge of the bookmark bar button from which it springs.
4) Tighten up the menu item spacing.
BUG=54701,69418,69346,59057
TEST=1) Place window near bottom of screen with dock showing. Pop up a folder menu (one which will completely fit on the screen after being fully scrolled up) and scroll fully up. Verify that the bottom it properly placed and all folder contents are shown. Verify that the menu right edge is aligned with the right edge of the button.
2) Have a bookmark folder menu with a submenu near the bottom. Pop up the folder menu and then the submenu. Verify all of the submenu appears.
3) All BookmarkBarFolderControllerTests menu unit tests pass.
4) All BookmarkBarFolderControllerMenuTests menu unit tests pass.
Review URL: http://codereview.chromium.org/6257005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71889 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/status_icons')
0 files changed, 0 insertions, 0 deletions