Prechádzať zdrojové kódy

Print a warning for Viewports that have 2D usage and HDR enabled

2D usage + HDR isn't supported, so the usage must be changed to 3D
to be able to use HDR.
Hugo Locurcio 4 rokov pred
rodič
commit
f64d5f4095
2 zmenil súbory, kde vykonal 15 pridanie a 5 odobranie
  1. 1 0
      doc/classes/Viewport.xml
  2. 14 5
      scene/main/viewport.cpp

+ 1 - 0
doc/classes/Viewport.xml

@@ -315,6 +315,7 @@
 		</member>
 		</member>
 		<member name="usage" type="int" setter="set_usage" getter="get_usage" enum="Viewport.Usage" default="2">
 		<member name="usage" type="int" setter="set_usage" getter="get_usage" enum="Viewport.Usage" default="2">
 			The rendering mode of viewport.
 			The rendering mode of viewport.
+			[b]Note:[/b] If set to [constant USAGE_2D] or [constant USAGE_2D_NO_SAMPLING], [member hdr] will have no effect when enabled since HDR is not supported for 2D.
 		</member>
 		</member>
 		<member name="world" type="World" setter="set_world" getter="get_world">
 		<member name="world" type="World" setter="set_world" getter="get_world">
 			The custom [World] which can be used as 3D environment source.
 			The custom [World] which can be used as 3D environment source.

+ 14 - 5
scene/main/viewport.cpp

@@ -2928,11 +2928,6 @@ Control *Viewport::get_modal_stack_top() const {
 }
 }
 
 
 String Viewport::get_configuration_warning() const {
 String Viewport::get_configuration_warning() const {
-	/*if (get_parent() && !Object::cast_to<Control>(get_parent()) && !render_target) {
-
-		return TTR("This viewport is not set as render target. If you intend for it to display its contents directly to the screen, make it a child of a Control so it can obtain a size. Otherwise, make it a RenderTarget and assign its internal texture to some node for display.");
-	}*/
-
 	String warning = Node::get_configuration_warning();
 	String warning = Node::get_configuration_warning();
 
 
 	if (size.x == 0 || size.y == 0) {
 	if (size.x == 0 || size.y == 0) {
@@ -2941,6 +2936,14 @@ String Viewport::get_configuration_warning() const {
 		}
 		}
 		warning += TTR("Viewport size must be greater than 0 to render anything.");
 		warning += TTR("Viewport size must be greater than 0 to render anything.");
 	}
 	}
+
+	if (hdr && (usage == USAGE_2D || usage == USAGE_2D_NO_SAMPLING)) {
+		if (warning != String()) {
+			warning += "\n\n";
+		}
+		warning += TTR("This Viewport has HDR enabled, but its Usage is set to 2D or 2D No-Sampling.\nHDR is only supported in Viewports that have their Usage set to 3D or 3D No-Effects.\nHDR will be disabled for this Viewport.");
+	}
+
 	return warning;
 	return warning;
 }
 }
 
 
@@ -2995,6 +2998,7 @@ void Viewport::set_hdr(bool p_hdr) {
 
 
 	hdr = p_hdr;
 	hdr = p_hdr;
 	VS::get_singleton()->viewport_set_hdr(viewport, p_hdr);
 	VS::get_singleton()->viewport_set_hdr(viewport, p_hdr);
+	update_configuration_warning();
 }
 }
 
 
 bool Viewport::get_hdr() const {
 bool Viewport::get_hdr() const {
@@ -3002,8 +3006,13 @@ bool Viewport::get_hdr() const {
 }
 }
 
 
 void Viewport::set_usage(Usage p_usage) {
 void Viewport::set_usage(Usage p_usage) {
+	if (usage == p_usage) {
+		return;
+	}
+
 	usage = p_usage;
 	usage = p_usage;
 	VS::get_singleton()->viewport_set_usage(viewport, VS::ViewportUsage(p_usage));
 	VS::get_singleton()->viewport_set_usage(viewport, VS::ViewportUsage(p_usage));
+	update_configuration_warning();
 }
 }
 
 
 Viewport::Usage Viewport::get_usage() const {
 Viewport::Usage Viewport::get_usage() const {