Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(8061)

Issue 6249060: Node Copy/Paste feature (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 10 months ago by lukas.toenne
Modified:
11 years, 10 months ago
Reviewers:
ideasman42, bf-codereview
Base URL:
https://svn.blender.org/svnroot/bf-blender/trunk/blender/
Visibility:
Public.

Description

Clipboard feature for nodes. With the Copy operator a copy of all selected nodes and links between them is stored in an offscreen list (not in the library). The Paste operator then in turn copies these into the active node tree in the editor. Limitation is that animated node values will lose all animation data. Fixing this would require copying a subset of the ID block's (node tree) fcurves to the clipboard and then insert a copy (with redirected paths) into the target node tree.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+258 lines, -21 lines) Patch
source/blender/blenkernel/BKE_node.h View 1 chunk +7 lines, -0 lines 0 comments Download
source/blender/blenkernel/intern/node.c View 10 chunks +88 lines, -21 lines 0 comments Download
source/blender/editors/space_node/node_edit.c View 1 chunk +153 lines, -0 lines 0 comments Download
source/blender/editors/space_node/node_intern.h View 1 chunk +4 lines, -0 lines 0 comments Download
source/blender/editors/space_node/node_ops.c View 2 chunks +6 lines, -0 lines 0 comments Download

Messages

Total messages: 2
lukas.toenne
11 years, 10 months ago (2012-05-28 17:28:23 UTC) #1
ideasman42
11 years, 10 months ago (2012-05-29 07:35:43 UTC) #2
On 2012/05/28 17:28:23, lukas.toenne wrote:

main concern is for eg - if you copy a node that uses a scene, then remove the
scene, then paste.

is the ID checked to still be valid? - couldn't see this from quick skim-review.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b