Disable snapping by default as a bug mitigation, and assorted cleanup

Author: Keavon

demo-artwork/isometric-fountain.graphite

@@ -72,7 +72,7 @@ pub struct SnappingState {
 impl Default for SnappingState {
 	fn default() -> Self {
 		Self {
-			snapping_enabled: true,
+			snapping_enabled: false, // TODO: Reenable this by default once <https://github.com/GraphiteEditor/Graphite/issues/1817> is fixed
 			grid_snapping: false,
 			bounds: Default::default(),
 			nodes: Default::default(),

demo-artwork/just-a-potted-cactus.graphite

@@ -292,28 +292,28 @@ mod test {
 	#[should_panic]
 	pub fn dyn_input_invalid_eval_panic() {
 		// let add = DynAnyNode::new(AddPairNode::new()).into_type_erased();
-		// add.eval(Box::new(&("32", 32u32)));
+		// add.eval(Box::new(&("32", 32_u32)));
 		let dyn_any = DynAnyNode::<(u32, u32), u32, _>::new(FutureWrapperNode { node: AddPairNode::new() });
 		let type_erased = Box::new(dyn_any) as TypeErasedBox;
 		let _ref_type_erased = type_erased.as_ref();
 		// let type_erased = Box::pin(dyn_any) as TypeErasedBox<'_>;
-		type_erased.eval(Box::new(&("32", 32u32)));
+		futures::executor::block_on(type_erased.eval(Box::new(&("32", 32_u32))));
 	}
 
 	#[test]
 	pub fn dyn_input_compose() {
 		// let add = DynAnyNode::new(AddPairNode::new()).into_type_erased();
-		// add.eval(Box::new(&("32", 32u32)));
+		// add.eval(Box::new(&("32", 32_u32)));
 		let dyn_any = DynAnyNode::<(u32, u32), u32, _>::new(FutureWrapperNode { node: AddPairNode::new() });
 		let type_erased = Box::new(dyn_any) as TypeErasedBox<'_>;
-		type_erased.eval(Box::new((4u32, 2u32)));
+		futures::executor::block_on(type_erased.eval(Box::new((4_u32, 2_u32))));
 		let id_node = FutureWrapperNode::new(IdentityNode::new());
 		let any_id = DynAnyNode::<u32, u32, _>::new(id_node);
 		let type_erased_id = Box::new(any_id) as TypeErasedBox;
 		let type_erased = ComposeTypeErased::new(NodeContainer::new(type_erased), NodeContainer::new(type_erased_id));
-		type_erased.eval(Box::new((4u32, 2u32)));
+		futures::executor::block_on(type_erased.eval(Box::new((4_u32, 2_u32))));
 		// let downcast: DowncastBothNode<(u32, u32), u32> = DowncastBothNode::new(type_erased.as_ref());
-		// downcast.eval((4u32, 2u32));
+		// downcast.eval((4_u32, 2_u32));
 	}
 
 	// TODO: Fix this test