Skip to content

Commit

Permalink
Merge pull request #53300 from Calinou/viewport-tweak-configuration-w…
Browse files Browse the repository at this point in the history
…arning-3.x
  • Loading branch information
akien-mga authored Oct 4, 2021
2 parents 55c9cc2 + c6617e9 commit 72a922e
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion doc/classes/Viewport.xml
Original file line number Diff line number Diff line change
Expand Up @@ -272,7 +272,7 @@
If set to a value greater than [code]0.0[/code], contrast-adaptive sharpening will be applied to the 3D viewport. This has a low performance cost and can be used to recover some of the sharpness lost from using FXAA. Values around [code]0.5[/code] generally give the best results. See also [member fxaa].
</member>
<member name="size" type="Vector2" setter="set_size" getter="get_size" default="Vector2( 0, 0 )">
The width and height of viewport.
The width and height of viewport. Must be set to a value greater than or equal to 2 pixels on both dimensions. Otherwise, nothing will be displayed.
</member>
<member name="size_override_stretch" type="bool" setter="set_size_override_stretch" getter="is_size_override_stretch_enabled" default="false">
If [code]true[/code], the size override affects stretch as well.
Expand Down
5 changes: 3 additions & 2 deletions scene/main/viewport.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -720,6 +720,7 @@ void Viewport::set_size(const Size2 &p_size) {
VS::get_singleton()->viewport_set_size(viewport, size.width, size.height);

_update_stretch_transform();
update_configuration_warning();

emit_signal("size_changed");
}
Expand Down Expand Up @@ -2935,11 +2936,11 @@ String Viewport::get_configuration_warning() const {

String warning = Node::get_configuration_warning();

if (size.x == 0 || size.y == 0) {
if (size.x <= 1 || size.y <= 1) {
if (warning != String()) {
warning += "\n\n";
}
warning += TTR("Viewport size must be greater than 0 to render anything.");
warning += TTR("The Viewport size must be greater than or equal to 2 pixels on both dimensions to render anything.");
}
return warning;
}
Expand Down

0 comments on commit 72a922e

Please sign in to comment.