bcarrella/cef
1
0
Fork 0
mirror of https://bitbucket.org/chromiumembedded/cef synced 2025-06-05 21:39:12 +02:00
Code Issues Packages Projects Releases Wiki Activity

views: cefclient: Fix top menu placement (issue #2102)

Browse Source
This commit is contained in:
Marshall Greenblatt
2017-02-17 21:42:15 -05:00
parent 6ed4fe96b8
commit c8c3ef4792
1 changed files with 9 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
tests/cefclient/browser/views_window.cc
View File

@@ -303,10 +303,12 @@ void ViewsWindow::OnMenuButtonPressed(CefRefPtr<CefMenuButton> menu_button,
DCHECK(with_controls_); DCHECK(with_controls_);
CefRefPtr<CefMenuModel> menu_model; CefRefPtr<CefMenuModel> menu_model;
cef_menu_anchor_position_t position = CEF_MENU_ANCHOR_TOPLEFT;
switch (menu_button->GetID()) { switch (menu_button->GetID()) {
case ID_MENU_BUTTON: case ID_MENU_BUTTON:
menu_model = button_menu_model_; menu_model = button_menu_model_;
position = CEF_MENU_ANCHOR_TOPRIGHT;
break; break;
case ID_TOP_FILE_MENU_BUTTON: case ID_TOP_FILE_MENU_BUTTON:
menu_model = file_menu_model_; menu_model = file_menu_model_;
@@ -316,8 +318,14 @@ void ViewsWindow::OnMenuButtonPressed(CefRefPtr<CefMenuButton> menu_button,
break; break;
} }
CefPoint point = screen_point;
if (position == CEF_MENU_ANCHOR_TOPLEFT) {
// Adjust menu position left by button width.
point.x -= menu_button->GetBounds().width - 4;
}
if (menu_model) if (menu_model)
menu_button->ShowMenu(menu_model, screen_point, CEF_MENU_ANCHOR_TOPRIGHT); menu_button->ShowMenu(menu_model, point, position);
} }
void ViewsWindow::ExecuteCommand(CefRefPtr<CefMenuModel> menu_model, void ViewsWindow::ExecuteCommand(CefRefPtr<CefMenuModel> menu_model,