[ProjectList] Fix Wrap Long Project Title

Co-authored-by: unit-tick <official.prince.john@gmail.com>

Author: CarrotOfPower

editor/project_manager/project_list.cpp

@@ -34,6 +34,7 @@
 #include "core/input/input.h"
 #include "core/io/dir_access.h"
 #include "core/object/callable_mp.h"
+#include "core/object/class_db.h"
 #include "core/os/os.h"
 #include "core/os/time.h"
 #include "core/version.h"
@@ -175,7 +176,7 @@ void ProjectListItemControl::_notification(int p_what) {
 		} break;
 
 		case NOTIFICATION_READY: {
-			set_project_title_autowrap();
+			call_deferred("set_project_title_autowrap");
 		} break;
 	}
 }
@@ -424,80 +425,90 @@ void ProjectListItemControl::set_is_grayed(bool p_grayed) {
 	}
 }
 
-void ProjectListItemControl::set_project_title_index(int p_title_index) {
-	project_title_index = p_title_index;
-}
-
 void ProjectListItemControl::set_project_title_autowrap() {
 	title_fullsize_cache = project_title->get_size().x;
-	window_size_cache = get_window()->get_size().x;
+	ProjectList *pl = get_list();
+	pl->window_size_cache = get_window()->get_size().x;
 
 	int tag_size = 0;
-	int tag_maxsize = 0;
 	for (Node *child : tag_container->iterate_children()) {
 		ProjectTag *tag = Object::cast_to<ProjectTag>(child);
 		tag_size += tag->get_size().x;
-
-		if (tag_maxsize == 0) {
-			tag_maxsize = tag->get_custom_maximum_size().x;
-		}
-	}
-	tag_size_cache = MIN(tag_size, tag_maxsize);
-	int &title_size_cache = get_list()->title_size_cache[project_title_index];
-
-	int size_check = 800 * EDSCALE;
-	if (title_size_cache == 0) {
-		title_size_cache = size_check;
 	}
+	tag_size_cache = tag_size;
 
-	if (title_fullsize_cache > size_check - tag_size_cache) {
+	if (title_fullsize_cache > pl->title_size_cache - tag_size) {
 		resize_project_title();
 	}
 }
 
 void ProjectListItemControl::resize_project_title() {
-	if (get_window() == nullptr) {
-		return;
-	}
-
-	int window_size = get_window()->get_size().x;
-	int difference = window_size - window_size_cache;
-	window_size_cache = window_size;
-
-	int &title_size_cache = get_list()->title_size_cache[project_title_index];
-	title_size_cache += difference;
-
-	if (title_size_cache > title_fullsize_cache + tag_size_cache) {
+	const int title_available_space = get_list()->title_size_cache - tag_size_cache;
+	if (title_fullsize_cache <= title_available_space) {
 		project_title->set_custom_maximum_size(Vector2(-1, -1));
 		project_title->set_custom_minimum_size(Vector2(0, 0));
 		project_title->set_autowrap_mode(TextServer::AUTOWRAP_OFF);
-
 		return;
 	}
-	ProjectTag tag = ProjectTag("dummy");
-	int tag_maxsize = tag.get_custom_maximum_size().x;
-	int title_maxsize = title_size_cache - tag_size_cache;
-	int title_minsize = title_size_cache - tag_maxsize;
 
-	int abs_minsize = (200 * EDSCALE);
+	const int abs_minsize = (200 * EDSCALE);
 	if (title_fullsize_cache > abs_minsize) {
-		if (title_minsize < abs_minsize) {
-			title_minsize = abs_minsize + tag_maxsize - tag_size_cache;
-		}
-		if (title_maxsize < title_minsize) {
-			project_title->set_custom_maximum_size(Vector2(title_minsize, -1));
-		} else {
-			project_title->set_custom_maximum_size(Vector2(title_maxsize, -1));
-		}
+		const int title_maxsize = title_available_space;
+		const int title_minsize = MAX(title_maxsize, abs_minsize);
+		project_title->set_custom_maximum_size(Vector2(MAX(title_minsize, title_maxsize), -1));
 		project_title->set_custom_minimum_size(Vector2(title_minsize, 0));
 		project_title->set_autowrap_mode(TextServer::AUTOWRAP_WORD_SMART);
 	}
 }
 
+int ProjectListItemControl::get_edge_elements_width() const {
+	// Returns the combined width of the left spacer, favorite button, project icon,
+	// right spacer, and the separation constant (10 * EDSCALE)
+	int width = 0;
+	int child_count = get_child_count();
+	// Left spacer (first child)
+	if (child_count > 0) {
+		Control *spacer = Object::cast_to<Control>(get_child(0));
+		if (spacer) {
+			width += spacer->get_size().x;
+		}
+	}
+	// Favorite button
+	if (favorite_button) {
+		width += favorite_button->get_size().x;
+	}
+	// Project icon
+	if (project_icon) {
+		width += project_icon->get_size().x;
+	}
+	// Right spacer (last child)
+	if (child_count > 0) {
+		Control *right_spacer = Object::cast_to<Control>(get_child(child_count - 1));
+		if (right_spacer) {
+			width += right_spacer->get_size().x;
+		}
+	}
+	// Separation constant
+	width += 10 * EDSCALE;
+	return width;
+}
+
+void ProjectList::_set_title_size_cache() {
+	if (!_projects.is_empty()) {
+		Item &item = _projects.write[0];
+		// Approx. size of project_title + tag_size
+		title_size_cache = get_size().x - item.control->get_edge_elements_width() - 100 * EDSCALE;
+	} else {
+		title_size_cache = -1;
+	}
+}
+
 void ProjectListItemControl::_bind_methods() {
 	ADD_SIGNAL(MethodInfo("favorite_pressed"));
 	ADD_SIGNAL(MethodInfo("explore_pressed"));
 	ADD_SIGNAL(MethodInfo("request_menu"));
+
+	ClassDB::bind_method(D_METHOD("set_project_title_autowrap"), &ProjectListItemControl::set_project_title_autowrap);
 }
 
 ProjectListItemControl::ProjectListItemControl() {
@@ -661,6 +672,10 @@ bool ProjectList::project_feature_looks_like_version(const String &p_feature) {
 
 void ProjectList::_notification(int p_what) {
 	switch (p_what) {
+		case NOTIFICATION_READY: {
+			_set_title_size_cache();
+		} break;
+
 		case NOTIFICATION_TRANSLATION_CHANGED: {
 			if (is_ready()) {
 				for (const Item &item : _projects) {
@@ -936,14 +951,12 @@ void ProjectList::update_project_list() {
 	// If you have 150 projects, it may read through 150 files on your disk at once + load 150 icons.
 	// FIXME: Does it really have to be a full, hard reload? Runtime updates should be made much cheaper.
 
-	int temp_title_index = -1;
 	if (ProjectManager::get_singleton()->is_initialized()) {
 		// Clear whole list
 		for (int i = 0; i < _projects.size(); ++i) {
 			Item &project = _projects.write[i];
 			CRASH_COND(project.control == nullptr);
 
-			temp_title_index = project.project_title_index;
 			memdelete(project.control); // Why not queue_free()?
 		}
 
@@ -956,9 +969,6 @@ void ProjectList::update_project_list() {
 
 	// Create controls
 	for (int i = 0; i < _projects.size(); ++i) {
-		Item &item = _projects.write[i];
-		item.project_title_index = temp_title_index;
-
 		_create_project_item_control(i);
 	}
 
@@ -1146,14 +1156,10 @@ int ProjectList::refresh_project(const String &dir_path) {
 
 	bool was_selected = _selected_project_paths.has(dir_path);
 
-	int temp_title_index = -1;
-
 	// Remove item in any case
 	for (int i = 0; i < _projects.size(); ++i) {
 		const Item &existing_item = _projects[i];
 		if (existing_item.path == dir_path) {
-			temp_title_index = existing_item.project_title_index;
-
 			_remove_project(i, false);
 			break;
 		}
@@ -1165,7 +1171,6 @@ int ProjectList::refresh_project(const String &dir_path) {
 
 		Item item = load_project_data(dir_path, is_favorite);
 
-		item.project_title_index = temp_title_index;
 		_projects.push_back(item);
 		_create_project_item_control(_projects.size() - 1);
 
@@ -1233,16 +1238,11 @@ void ProjectList::_create_project_item_control(int p_index) {
 #endif
 	hb->connect("request_menu", callable_mp(this, &ProjectList::_open_menu).bind(hb));
 
-	if (item.project_title_index == -1) {
-		project_title_index_count++;
-		title_size_cache[project_title_index_count] = 0;
-		item.project_title_index = project_title_index_count;
-		hb->set_project_title_index(project_title_index_count);
-	} else {
-		hb->set_project_title_index(item.project_title_index);
-	}
-
 	project_list_vbox->add_child(hb);
+
+	if (title_size_cache == -1) {
+		_set_title_size_cache();
+	}
 }
 
 void ProjectList::_update_project_control_translatable_fields(const Item &item) {
@@ -1597,6 +1597,16 @@ void ProjectList::erase_selected_projects(bool p_delete_project_contents) {
 // Resize project titles.
 
 void ProjectList::resize_project_titles() {
+	Window *win = get_window();
+	if (win == nullptr) {
+		return;
+	}
+
+	const int window_size = win->get_size().x;
+	const int difference = window_size - window_size_cache;
+	window_size_cache = window_size;
+	title_size_cache += difference;
+
 	for (Item &item : _projects) {
 		item.control->resize_project_title();
 	}

editor/project_manager/project_list.h

@@ -62,8 +62,6 @@ class ProjectListItemControl : public HBoxContainer {
 
 	Color favorite_focus_color;
 
-	int project_title_index = -1;
-
 	bool project_is_missing = false;
 	bool icon_needs_reload = true;
 	bool is_selected = false;
@@ -98,7 +96,6 @@ class ProjectListItemControl : public HBoxContainer {
 
 	int title_fullsize_cache = 0;
 	int tag_size_cache = 0;
-	int window_size_cache = 0;
 
 protected:
 	void _notification(int p_what);
@@ -120,11 +117,12 @@ class ProjectListItemControl : public HBoxContainer {
 	void set_is_favorite(bool p_favorite);
 	void set_is_missing(bool p_missing);
 	void set_is_grayed(bool p_grayed);
-	void set_project_title_index(int p_title_index);
 	void set_project_title_autowrap();
 
 	void resize_project_title();
 
+	int get_edge_elements_width() const;
+
 	ProjectListItemControl();
 };
 
@@ -172,7 +170,6 @@ class ProjectList : public ScrollContainer {
 		bool missing = false;
 		bool recovery_mode = false;
 		int version = 0;
-		int project_title_index = -1;
 
 		ProjectListItemControl *control = nullptr;
 
@@ -221,8 +218,8 @@ class ProjectList : public ScrollContainer {
 		String get_last_edited_string() const;
 	};
 
-	HashMap<int, int> title_size_cache;
-	int project_title_index_count = -1;
+	int window_size_cache = 0;
+	int title_size_cache = 0;
 
 private:
 	String _config_path;
@@ -290,6 +287,9 @@ class ProjectList : public ScrollContainer {
 	void _deselect_project_nocheck(int p_index);
 	void _select_project_range(int p_begin, int p_end);
 
+	// Title size cache.
+	void _set_title_size_cache();
+
 	// Global menu integration.
 
 	void _global_menu_new_window(const Variant &p_tag);