From b57c53a4273711d7eb0294433940dc49d3fd02bd Mon Sep 17 00:00:00 2001 From: Utsav Singh Date: Tue, 24 Sep 2024 02:06:28 +0530 Subject: [PATCH] Fix broken Opacity slider in Layers panel (#2004) Fix Opacity slider in Layers panel adding Transactions on_update and add Transactions when on_commit --- .../messages/portfolio/document/document_message_handler.rs | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/editor/src/messages/portfolio/document/document_message_handler.rs b/editor/src/messages/portfolio/document/document_message_handler.rs index 9a817036..d7da13f1 100644 --- a/editor/src/messages/portfolio/document/document_message_handler.rs +++ b/editor/src/messages/portfolio/document/document_message_handler.rs @@ -974,12 +974,7 @@ impl MessageHandler> for DocumentMessag } DocumentMessage::SetOpacityForSelectedLayers { opacity } => { let opacity = opacity.clamp(0., 1.); - let mut added_transaction = false; for layer in self.network_interface.selected_nodes(&[]).unwrap().selected_layers_except_artboards(&self.network_interface) { - if !added_transaction { - responses.add(DocumentMessage::AddTransaction); - added_transaction = true; - } responses.add(GraphOperationMessage::OpacitySet { layer, opacity }); } } @@ -1942,6 +1937,7 @@ impl DocumentMessageHandler { Message::NoOp } }) + .on_commit(|_| DocumentMessage::AddTransaction.into()) .widget_holder(), // Separator::new(SeparatorType::Unrelated).widget_holder(),