Page MenuHome
Create Task
Maniphest T70508

Wrong snapping default in Node Editor
Closed, ResolvedPublic
Actions

Description

System Information
Operating system: Windows-10-10.0.17134 64 Bits
Graphics card: GeForce GTX 950M/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 436.48

Blender Version
Broken: version: 2.81 (sub 13), branch: master, commit date: 2019-10-03 21:03, hash: rBfbc096cf075b
Worked: 2.80

Short description of error
Nodes don't snap to the grid when snapping is turned on.

Exact steps for others to reproduce the error
Open Blender.
Switch to any node editor.
Move some nodes around.
Turn on snapping, and move some nodes around.
Notice how they don't snap to the grid.

Edit @Julian Eisel (Severin): Issue is that values of the RNA enum were changed in rBa2443848646c, without versioning to account for existing values/defaults.

Revisions and Commits

rB Blender
rB4761028fc75a Fix T70508: Wrong snapping default in Node Editor

Related Objects

Event Timeline

Andy Sellers (andrewm90) created this task.Oct 4 2019, 6:32 AM
Charlie Jolly (charlie) added a subscriber: Charlie Jolly (charlie).Oct 4 2019, 10:56 AM

Can't reproduce. This works for me in the latest builds.

Julian Eisel (Severin) lowered the priority of this task from 90 to 30.Oct 4 2019, 1:04 PM
Julian Eisel (Severin) added a subscriber: Julian Eisel (Severin).

The default for the Snapping mode seems to have changed, it now snaps to other nodes by default, not to the grid. Can you confirm that that is why it's not working?

However, I don't see a commit that intentionally did that change, it's likely due to missing versioning in rBa2443848646c.
Note that Reset to Default Value sets to the intentioned default, that is grid snapping.

Andy Sellers (andrewm90) added a comment.Oct 4 2019, 6:14 PM

Yeah, it looks like it's just a changed default.

Julian Eisel (Severin) renamed this task from Nodes Don't Snap To Grid to Wrong default in Node Editor.Oct 7 2019, 11:31 AM
Julian Eisel (Severin) renamed this task from Wrong default in Node Editor to Wrong snapping default in Node Editor.
Julian Eisel (Severin) assigned this task to Germano Cavalcante (mano-wii).
Julian Eisel (Severin) raised the priority of this task from 30 to 50.
Julian Eisel (Severin) updated the task description. (Show Details)
Julian Eisel (Severin) edited projects, added BF Blender (2.81); removed BF Blender.
Germano Cavalcante (mano-wii) changed the task status from Unknown Status to Resolved by committing rB4761028fc75a: Fix T70508: Wrong snapping default in Node Editor.Oct 7 2019, 3:04 PM
Germano Cavalcante (mano-wii) added a commit: rB4761028fc75a: Fix T70508: Wrong snapping default in Node Editor.