Browse Source

Merge pull request #61686 from YeldhamDev/weird_bugs

Fix specific bug related to submenus in `PopupMenu`
Rémi Verschelde 3 năm trước cách đây
mục cha
commit
ef8401edf5
1 tập tin đã thay đổi với 2 bổ sung2 xóa
  1. 2 2
      scene/gui/popup_menu.cpp

+ 2 - 2
scene/gui/popup_menu.cpp

@@ -167,6 +167,7 @@ void PopupMenu::_activate_submenu(int over) {
 
 	pm->set_position(pos);
 	pm->set_scale(get_global_transform().get_scale());
+	pm->popup();
 
 	PopupMenu *pum = Object::cast_to<PopupMenu>(pm);
 	if (pum) {
@@ -175,6 +176,7 @@ void PopupMenu::_activate_submenu(int over) {
 			pum->set_current_index(0);
 		}
 
+		// Setting autohide areas *must* be done after `popup()` which can move the popup (to fit it into the viewport).
 		pr.position -= pum->get_global_position();
 		pum->clear_autohide_areas();
 		pum->add_autohide_area(Rect2(pr.position.x, pr.position.y, pr.size.x, items[over]._ofs_cache));
@@ -183,8 +185,6 @@ void PopupMenu::_activate_submenu(int over) {
 			pum->add_autohide_area(Rect2(pr.position.x, pr.position.y + from, pr.size.x, pr.size.y - from));
 		}
 	}
-
-	pm->popup();
 }
 
 void PopupMenu::_submenu_timeout() {