Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Commit

Permalink
Root cause fix (reverted previous commit and fixed real issue); scrol…
Browse files Browse the repository at this point in the history
…lbars were being added (x and y) because entire container was capped at the size of the parent contextMenu object. By setting this to 100% w/h, it allows the menu to grow appropriately. You'll notice contextMenu already was restricting with calc(~'100% - @{usedUpChrome}')... I'm not sure why it needs width and max-width set.
  • Loading branch information
bsclifton committed May 1, 2016
1 parent 10ec0a4 commit 064c55d
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions less/contextMenu.less
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,11 @@
}
// This is a reasonable max height and also solves problems for bookmarks menu
// and bookmarks overflow menu reaching down too low.
height: 100%;
width: 100%;
@usedUpChrome: @navbarHeight + @bookmarksToolbarWithFaviconsHeight;
max-height: calc(~'100% - @{usedUpChrome}');
overflow: auto;
padding: 0px 0px;
position: absolute;
z-index: 350;
Expand Down

0 comments on commit 064c55d

Please sign in to comment.