griefith/test
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Compositor: set crop node default size to 1920x1080

Browse Source 
The crop node default values produces an image of size 100x100. While
this is better than 0x0, it still makes the backdrop image look like it
disappeared after adding a crop node when the new image happens to be
behind a node.

Ideally, the default should be the render resolution, but we don't have
this information at node declaration time, so initialize width and
height to 1920 x 1080 such that the node has no apparent effect for the
default blend file.

<img width="216" alt="default.png" src="attachments/a904a92c-3c0c-4075-80ea-f6168bfb0966">

Pull Request: https://projects.blender.org/blender/blender/pulls/139493
This commit is contained in:
Habib Gahbiche
2025-05-27 18:55:06 +02:00
parent 9af8d8af21
commit 2c271e2442
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
source/blender/nodes/composite/nodes/node_composite_crop.cc
View File

@@ -39,12 +39,12 @@ static void cmp_node_crop_declare(NodeDeclarationBuilder &b)
.compositor_expects_single_value()
.description("The Y position of the lower left corner of the crop region");
b.add_input<decl::Int>("Width")
.default_value(100)
.default_value(1920)
.min(1)
.compositor_expects_single_value()
.description("The width of the crop region");
b.add_input<decl::Int>("Height")
.default_value(100)
.default_value(1080)
.min(1)
.compositor_expects_single_value()
.description("The width of the crop region");