Commit 77a5b2b4 authored by Gauthier Ambard's avatar Gauthier Ambard Committed by Commit Bot

Polish the popup menu

This CL adds some polish to the popup menu, increasing its max height
and swaping the BrowserActions and CollectionsActions items.

Bug: 845477
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: Ia7f05c29f0110bedb6f0ede7847ca3e39e420360
Reviewed-on: https://chromium-review.googlesource.com/1069148Reviewed-by: default avatarStepan Khapugin <stkhapugin@chromium.org>
Commit-Queue: Gauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#561009}
parent 4ebb367a
......@@ -593,7 +593,7 @@ PopupMenuToolsItem* CreateTableViewItem(int titleID,
NSArray* collectionActions = [self collectionItems];
self.items = @[ tabActions, browserActions, collectionActions ];
self.items = @[ tabActions, collectionActions, browserActions ];
}
- (NSArray<TableViewItem*>*)itemsForNewTab {
......
......@@ -19,7 +19,7 @@ namespace {
const CGFloat kMinHeight = 200;
const CGFloat kMinWidth = 200;
const CGFloat kMaxWidth = 300;
const CGFloat kMaxHeight = 400;
const CGFloat kMaxHeight = 435;
const CGFloat kMinWidthDifference = 50;
const CGFloat kMinHorizontalMargin = 5;
const CGFloat kMinVerticalMargin = 15;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment