Browse Source

Revert "Make sure that `data.last_minimum_size` is consistent with `get_combined_minimum_size()` at the same time"

This reverts commit 594852caf1353ba48ba40c19bc05977635b5b9e8.
Yuri Sizov 2 years ago
parent
commit
df26066ce9
1 changed files with 19 additions and 15 deletions
  1. 19 15
      scene/gui/control.cpp

+ 19 - 15
scene/gui/control.cpp

@@ -1582,6 +1582,22 @@ void Control::_update_minimum_size() {
 		return;
 	}
 
+	Size2 minsize = get_combined_minimum_size();
+	data.updating_last_minimum_size = false;
+
+	if (minsize != data.last_minimum_size) {
+		data.last_minimum_size = minsize;
+		_size_changed();
+		emit_signal(SceneStringNames::get_singleton()->minimum_size_changed);
+	}
+}
+
+void Control::update_minimum_size() {
+	ERR_MAIN_THREAD_GUARD;
+	if (!is_inside_tree() || data.block_minimum_size_adjust) {
+		return;
+	}
+
 	Control *invalidate = this;
 
 	// Invalidate cache upwards.
@@ -1604,21 +1620,6 @@ void Control::_update_minimum_size() {
 		return;
 	}
 
-	Size2 minsize = get_combined_minimum_size();
-	data.updating_last_minimum_size = false;
-	if (minsize != data.last_minimum_size) {
-		data.last_minimum_size = minsize;
-		_size_changed();
-		emit_signal(SceneStringNames::get_singleton()->minimum_size_changed);
-	}
-}
-
-void Control::update_minimum_size() {
-	ERR_MAIN_THREAD_GUARD;
-	if (!is_inside_tree() || data.block_minimum_size_adjust) {
-		return;
-	}
-
 	if (data.updating_last_minimum_size) {
 		return;
 	}
@@ -3139,6 +3140,7 @@ void Control::_notification(int p_notification) {
 		} break;
 
 		case NOTIFICATION_POST_ENTER_TREE: {
+			data.minimum_size_valid = false;
 			data.is_rtl_dirty = true;
 			_size_changed();
 		} break;
@@ -3267,7 +3269,9 @@ void Control::_notification(int p_notification) {
 					get_viewport()->_gui_hide_control(this);
 				}
 			} else {
+				data.minimum_size_valid = false;
 				_update_minimum_size();
+				_size_changed();
 			}
 		} break;