From aef00021a9c8a51326d6ed6f9a4fc9ddeae175bd Mon Sep 17 00:00:00 2001
From: "Andrii Doroshenko (Xrayez)" <xrayez@gmail.com>
Date: Mon, 19 Oct 2020 20:41:25 +0300
Subject: [PATCH] Add `border_width` to `ReferenceRect`

Exposes a hidden parameter behind `CanvasItem.draw_rect()`.
---
 doc/classes/ReferenceRect.xml |  3 +++
 scene/gui/reference_rect.cpp  | 20 ++++++++++++++------
 scene/gui/reference_rect.h    | 11 +++++++----
 3 files changed, 24 insertions(+), 10 deletions(-)

diff --git a/doc/classes/ReferenceRect.xml b/doc/classes/ReferenceRect.xml
index 4f8f71b3dbc..ed2d397f7d3 100644
--- a/doc/classes/ReferenceRect.xml
+++ b/doc/classes/ReferenceRect.xml
@@ -14,6 +14,9 @@
 		<member name="border_color" type="Color" setter="set_border_color" getter="get_border_color" default="Color( 1, 0, 0, 1 )">
 			Sets the border [Color] of the [ReferenceRect].
 		</member>
+		<member name="border_width" type="float" setter="set_border_width" getter="get_border_width" default="1.0">
+			Sets the border width of the [ReferenceRect]. The border grows both inwards and outwards with respect to the rectangle box.
+		</member>
 		<member name="editor_only" type="bool" setter="set_editor_only" getter="get_editor_only" default="true">
 			If set to [code]true[/code], the [ReferenceRect] will only be visible while in editor. Otherwise, [ReferenceRect] will be visible in game.
 		</member>
diff --git a/scene/gui/reference_rect.cpp b/scene/gui/reference_rect.cpp
index 27c57c684a8..32ec0049f0d 100644
--- a/scene/gui/reference_rect.cpp
+++ b/scene/gui/reference_rect.cpp
@@ -38,7 +38,7 @@ void ReferenceRect::_notification(int p_what) {
 			return;
 		}
 		if (Engine::get_singleton()->is_editor_hint() || !editor_only) {
-			draw_rect(Rect2(Point2(), get_size()), border_color, false);
+			draw_rect(Rect2(Point2(), get_size()), border_color, false, border_width);
 		}
 	}
 }
@@ -52,6 +52,15 @@ Color ReferenceRect::get_border_color() const {
 	return border_color;
 }
 
+void ReferenceRect::set_border_width(float p_width) {
+	border_width = MAX(0.0, p_width);
+	update();
+}
+
+float ReferenceRect::get_border_width() const {
+	return border_width;
+}
+
 void ReferenceRect::set_editor_only(const bool &p_enabled) {
 	editor_only = p_enabled;
 	update();
@@ -65,14 +74,13 @@ void ReferenceRect::_bind_methods() {
 	ClassDB::bind_method(D_METHOD("get_border_color"), &ReferenceRect::get_border_color);
 	ClassDB::bind_method(D_METHOD("set_border_color", "color"), &ReferenceRect::set_border_color);
 
+	ClassDB::bind_method(D_METHOD("get_border_width"), &ReferenceRect::get_border_width);
+	ClassDB::bind_method(D_METHOD("set_border_width", "width"), &ReferenceRect::set_border_width);
+
 	ClassDB::bind_method(D_METHOD("get_editor_only"), &ReferenceRect::get_editor_only);
 	ClassDB::bind_method(D_METHOD("set_editor_only", "enabled"), &ReferenceRect::set_editor_only);
 
 	ADD_PROPERTY(PropertyInfo(Variant::COLOR, "border_color"), "set_border_color", "get_border_color");
+	ADD_PROPERTY(PropertyInfo(Variant::FLOAT, "border_width", PROPERTY_HINT_RANGE, "0.0,5.0,0.1,or_greater"), "set_border_width", "get_border_width");
 	ADD_PROPERTY(PropertyInfo(Variant::BOOL, "editor_only"), "set_editor_only", "get_editor_only");
 }
-
-ReferenceRect::ReferenceRect() {
-	border_color = Color(1, 0, 0);
-	editor_only = true;
-}
diff --git a/scene/gui/reference_rect.h b/scene/gui/reference_rect.h
index db2f4269f33..becbbf47c54 100644
--- a/scene/gui/reference_rect.h
+++ b/scene/gui/reference_rect.h
@@ -35,19 +35,22 @@
 
 class ReferenceRect : public Control {
 	GDCLASS(ReferenceRect, Control);
-	Color border_color;
-	bool editor_only;
+
+	Color border_color = Color(1, 0, 0);
+	float border_width = 1.0;
+	bool editor_only = true;
 
 protected:
 	void _notification(int p_what);
 	static void _bind_methods();
 
 public:
-	ReferenceRect();
-
 	void set_border_color(const Color &p_color);
 	Color get_border_color() const;
 
+	void set_border_width(float p_width);
+	float get_border_width() const;
+
 	void set_editor_only(const bool &p_enabled);
 	bool get_editor_only() const;
 };