mirror/godot
2
0
Fork 0
mirror of https://github.com/godotengine/godot.git synced 2025-04-13 01:00:35 +08:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #102079 from bruvzg/emb_nmenu

Browse Source 
Fix native popup menus position in embedded windows.
This commit is contained in:
Thaddeus Crews 2025-02-04 09:04:09 -06:00
commit 92ebcd2343
No known key found for this signature in database
GPG Key ID: 62181B86FE9E5D84
3 changed files with 19 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
platform/macos/native_menu_macos.mm
View File

@ -293,7 +293,7 @@ void NativeMenuMacOS::popup(const RID &p_rid, const Vector2i &p_position) {
position += ds->_get_screens_origin();
position /= ds->screen_get_max_scale();
[md->menu popUpMenuPositioningItem:nil atLocation:NSMakePoint(position.x, position.y) inView:nil];
[md->menu popUpMenuPositioningItem:nil atLocation:NSMakePoint(position.x, position.y - 5) inView:nil]; // Menu vertical position doesn't include rounded corners, add `5` display pixels to better align it with Godot buttons.
}
}

19
scene/gui/popup_menu.cpp
View File

@ -2958,6 +2958,21 @@ void PopupMenu::_bind_methods() {
PropertyListHelper::register_base_helper(&base_property_helper);
}
void PopupMenu::_native_popup(const Rect2i &p_rect) {
Point2i popup_pos = p_rect.position;
if (is_embedded()) {
popup_pos = get_embedder()->get_screen_transform().xform(popup_pos); // Note: for embedded windows "screen transform" is transform relative to embedder not the actual screen.
DisplayServer::WindowID wid = get_window_id();
if (wid == DisplayServer::INVALID_WINDOW_ID) {
wid = DisplayServer::MAIN_WINDOW_ID;
}
popup_pos += DisplayServer::get_singleton()->window_get_position(wid);
}
float win_scale = get_parent_visible_window()->get_content_scale_factor();
NativeMenu::get_singleton()->set_minimum_width(global_menu, p_rect.size.x * win_scale);
NativeMenu::get_singleton()->popup(global_menu, popup_pos);
}
void PopupMenu::popup(const Rect2i &p_bounds) {
bool native = global_menu.is_valid();
#ifdef TOOLS_ENABLED
@ -2967,7 +2982,7 @@ void PopupMenu::popup(const Rect2i &p_bounds) {
#endif
if (native) {
NativeMenu::get_singleton()->popup(global_menu, (p_bounds != Rect2i()) ? p_bounds.position : get_position());
_native_popup(p_bounds != Rect2i() ? p_bounds : Rect2i(get_position(), Size2i()));
} else {
set_flag(FLAG_NO_FOCUS, !is_embedded());
@ -2995,7 +3010,7 @@ void PopupMenu::set_visible(bool p_visible) {
if (native) {
if (p_visible) {
NativeMenu::get_singleton()->popup(global_menu, get_position());
_native_popup(Rect2i(get_position(), get_size()));
}
} else {
set_flag(FLAG_NO_FOCUS, !is_embedded());

1
scene/gui/popup_menu.h
View File

@ -215,6 +215,7 @@ class PopupMenu : public Popup {
bool _set_item_accelerator(int p_index, const Ref<InputEventKey> &p_ie);
void _set_item_checkable_type(int p_index, int p_checkable_type);
int _get_item_checkable_type(int p_index) const;
void _native_popup(const Rect2i &p_rect);
protected:
virtual Rect2i _popup_adjust_rect() const override;