diff --git a/.vscode/extensions.json b/.vscode/extensions.json index 13a358fb61..4ca6063754 100644 --- a/.vscode/extensions.json +++ b/.vscode/extensions.json @@ -3,6 +3,7 @@ "matklad.rust-analyzer", "dbaeumer.vscode-eslint", "octref.vetur", - "formulahendry.auto-close-tag" + "formulahendry.auto-close-tag", + "aaron-bond.better-comments" ] -} \ No newline at end of file +} diff --git a/charcoal/src/lib.rs b/charcoal/src/lib.rs index 909562f6d5..78b81e0011 100644 --- a/charcoal/src/lib.rs +++ b/charcoal/src/lib.rs @@ -1,7 +1,7 @@ #[cfg(test)] mod tests { - #[test] - fn it_works() { - assert_eq!(2 + 2, 4); - } + // #[test] + // fn it_works() { + // assert_eq!(2 + 2, 4); + // } } diff --git a/editor/src/communication/dispatcher.rs b/editor/src/communication/dispatcher.rs index abe519d9dd..cc7331f6c4 100644 --- a/editor/src/communication/dispatcher.rs +++ b/editor/src/communication/dispatcher.rs @@ -7,6 +7,7 @@ pub use crate::tool::ToolMessageHandler; use crate::global::GlobalMessageHandler; use std::collections::VecDeque; +#[derive(Debug, Default)] pub struct Dispatcher { input_preprocessor: InputPreprocessor, input_mapper: InputMapper, @@ -30,6 +31,10 @@ const SIDE_EFFECT_FREE_MESSAGES: &[MessageDiscriminant] = &[ ]; impl Dispatcher { + pub fn new() -> Self { + Self::default() + } + pub fn handle_message>(&mut self, message: T) { self.messages.push_back(message.into()); @@ -68,18 +73,6 @@ impl Dispatcher { list } - pub fn new() -> Dispatcher { - Dispatcher { - input_preprocessor: InputPreprocessor::default(), - global_message_handler: GlobalMessageHandler::new(), - input_mapper: InputMapper::default(), - documents_message_handler: DocumentsMessageHandler::default(), - tool_message_handler: ToolMessageHandler::default(), - messages: VecDeque::new(), - responses: vec![], - } - } - fn log_message(&self, message: &Message) { use Message::*; if log::max_level() == log::LevelFilter::Trace diff --git a/editor/src/document/document_file.rs b/editor/src/document/document_file.rs index 2abae6c1a5..20d8fb2784 100644 --- a/editor/src/document/document_file.rs +++ b/editor/src/document/document_file.rs @@ -5,7 +5,7 @@ pub use super::layer_panel::*; use super::movement_handler::{MovementMessage, MovementMessageHandler}; use super::overlay_message_handler::OverlayMessageHandler; use super::transform_layer_handler::{TransformLayerMessage, TransformLayerMessageHandler}; -use super::vectorize_layerdata; +use super::vectorize_layer_metadata; use crate::consts::DEFAULT_DOCUMENT_NAME; use crate::consts::{ASYMPTOTIC_EFFECT, FILE_EXPORT_SUFFIX, FILE_SAVE_SUFFIX, SCALE_EFFECT, SCROLLBAR_SPACING}; @@ -24,7 +24,7 @@ use kurbo::PathSeg; use log::warn; use serde::{Deserialize, Serialize}; -type DocumentSave = (GrapheneDocument, HashMap, LayerData>); +type DocumentSave = (GrapheneDocument, HashMap, LayerMetadata>); #[derive(PartialEq, Clone, Debug, Serialize, Deserialize, Hash)] pub enum FlipAxis { @@ -69,8 +69,8 @@ pub struct DocumentMessageHandler { pub document_redo_history: Vec, pub saved_document_identifier: u64, pub name: String, - #[serde(with = "vectorize_layerdata")] - pub layer_data: HashMap, LayerData>, + #[serde(with = "vectorize_layer_metadata")] + pub layer_metadata: HashMap, LayerMetadata>, layer_range_selection_reference: Vec, #[serde(skip)] movement_handler: MovementMessageHandler, @@ -90,7 +90,7 @@ impl Default for DocumentMessageHandler { document_redo_history: Vec::new(), name: String::from("Untitled Document"), saved_document_identifier: 0, - layer_data: vec![(vec![], LayerData::new(true))].into_iter().collect(), + layer_metadata: vec![(vec![], LayerMetadata::new(true))].into_iter().collect(), layer_range_selection_reference: Vec::new(), movement_handler: MovementMessageHandler::default(), overlay_message_handler: OverlayMessageHandler::default(), @@ -111,9 +111,9 @@ pub enum DocumentMessage { DispatchOperation(Box), #[child] Overlay(OverlayMessage), - UpdateLayerData { - path: Vec, - layer_data_entry: LayerData, + UpdateLayerMetadata { + layer_path: Vec, + layer_metadata: LayerMetadata, }, SetSelectedLayers(Vec>), AddSelectedLayers(Vec>), @@ -186,7 +186,7 @@ impl DocumentMessageHandler { } pub fn deserialize_document(serialized_content: &str) -> Result { - log::info!("Deserialising: {:?}", serialized_content); + log::info!("Deserializing: {:?}", serialized_content); serde_json::from_str(serialized_content).map_err(|e| DocumentError::InvalidFile(e.to_string())) } @@ -209,15 +209,15 @@ impl DocumentMessageHandler { pub fn is_unmodified_default(&self) -> bool { self.serialize_root().len() == Self::default().serialize_root().len() - && self.document_undo_history.len() == 0 - && self.document_redo_history.len() == 0 + && self.document_undo_history.is_empty() + && self.document_redo_history.is_empty() && self.name.starts_with(DEFAULT_DOCUMENT_NAME) } fn select_layer(&mut self, path: &[LayerId]) -> Option { println!("Select_layer fail: {:?}", self.all_layers_sorted()); - self.layer_data_mut(path).selected = true; + self.layer_metadata_mut(path).selected = true; let data = self.layer_panel_entry(path.to_vec()).ok()?; (!path.is_empty()).then(|| FrontendMessage::UpdateLayer { data }.into()) } @@ -269,12 +269,8 @@ impl DocumentMessageHandler { shapes.collect::>() } - pub fn create_layer_data(&mut self, path: &[LayerId]) { - self.layer_data.insert(path.to_vec(), LayerData::new(true)); - } - pub fn selected_layers(&self) -> impl Iterator { - self.layer_data.iter().filter_map(|(path, data)| data.selected.then(|| path.as_slice())) + self.layer_metadata.iter().filter_map(|(path, data)| data.selected.then(|| path.as_slice())) } pub fn selected_layers_without_children(&self) -> Vec> { @@ -305,7 +301,7 @@ impl DocumentMessageHandler { } pub fn selected_layers_contains(&self, path: &[LayerId]) -> bool { - self.layer_data.get(path).map(|layer| layer.selected).unwrap_or(false) + self.layer_metadata.get(path).map(|layer| layer.selected).unwrap_or(false) } pub fn selected_visible_layers(&self) -> impl Iterator { @@ -324,7 +320,7 @@ impl DocumentMessageHandler { LayerDataType::Shape(_) => (), LayerDataType::Folder(ref folder) => { path.push(*id); - if self.layer_data(path).expanded { + if self.layer_metadata(path).expanded { structure.push(space); self.serialize_structure(folder, structure, data, path); space = 0; @@ -370,7 +366,7 @@ impl DocumentMessageHandler { /// Returns an unsorted list of all layer paths including folders at all levels, except the document's top-level root folder itself pub fn all_layers(&self) -> Vec> { - self.layer_data.keys().filter(|path| !path.is_empty()).cloned().collect() + self.layer_metadata.keys().filter(|path| !path.is_empty()).cloned().collect() } /// Returns the paths to all layers in order, optionally including only selected or non-selected layers. @@ -378,14 +374,13 @@ impl DocumentMessageHandler { // Compute the indices for each layer to be able to sort them let mut layers_with_indices: Vec<(Vec, Vec)> = self - .layer_data + .layer_metadata .iter() // 'path.len() > 0' filters out root layer since it has no indices .filter_map(|(path, data)| (!path.is_empty() && (data.selected == selected.unwrap_or(data.selected))).then(|| path.clone())) .filter_map(|path| { - // Currently it is possible that layer_data contains layers that are don't actually exist (has been partially fixed in #281) - // and thus indices_for_path can return an error. We currently skip these layers and log a warning. - // Once this problem is solved this code can be simplified + // TODO: Currently it is possible that `layer_metadata` contains layers that are don't actually exist (has been partially fixed in #281) and thus + // TODO: `indices_for_path` can return an error. We currently skip these layers and log a warning. Once this problem is solved this code can be simplified. match self.graphene_document.indices_for_path(&path) { Err(err) => { warn!("layers_sorted: Could not get indices for the layer {:?}: {:?}", path, err); @@ -416,23 +411,23 @@ impl DocumentMessageHandler { self.layers_sorted(Some(false)) } - pub fn layer_data(&self, path: &[LayerId]) -> &LayerData { - self.layer_data.get(path).expect("Layerdata does not exist") + pub fn layer_metadata(&self, path: &[LayerId]) -> &LayerMetadata { + self.layer_metadata.get(path).unwrap_or_else(|| panic!("Editor's layer metadata for {:?} does not exist", path)) } - pub fn layer_data_mut(&mut self, path: &[LayerId]) -> &mut LayerData { - Self::layer_data_mut_no_borrow_self(&mut self.layer_data, path) + pub fn layer_metadata_mut(&mut self, path: &[LayerId]) -> &mut LayerMetadata { + Self::layer_metadata_mut_no_borrow_self(&mut self.layer_metadata, path) } - pub fn layer_data_mut_no_borrow_self<'a>(layer_data: &'a mut HashMap, LayerData>, path: &[LayerId]) -> &'a mut LayerData { - layer_data + pub fn layer_metadata_mut_no_borrow_self<'a>(layer_metadata: &'a mut HashMap, LayerMetadata>, path: &[LayerId]) -> &'a mut LayerMetadata { + layer_metadata .get_mut(path) .unwrap_or_else(|| panic!("Layer data cannot be found because the path {:?} does not exist", path)) } pub fn backup(&mut self, responses: &mut VecDeque) { self.document_redo_history.clear(); - self.document_undo_history.push((self.graphene_document.clone(), self.layer_data.clone())); + self.document_undo_history.push((self.graphene_document.clone(), self.layer_metadata.clone())); // Push the UpdateOpenDocumentsList message to the bus in order to update the save status of the open documents responses.push_back(DocumentsMessage::UpdateOpenDocumentsList.into()); @@ -449,10 +444,10 @@ impl DocumentMessageHandler { responses.push_back(DocumentsMessage::UpdateOpenDocumentsList.into()); match self.document_undo_history.pop() { - Some((document, layer_data)) => { + Some((document, layer_metadata)) => { let document = std::mem::replace(&mut self.graphene_document, document); - let layer_data = std::mem::replace(&mut self.layer_data, layer_data); - self.document_redo_history.push((document, layer_data)); + let layer_metadata = std::mem::replace(&mut self.layer_metadata, layer_metadata); + self.document_redo_history.push((document, layer_metadata)); Ok(()) } None => Err(EditorError::NoTransactionInProgress), @@ -464,10 +459,10 @@ impl DocumentMessageHandler { responses.push_back(DocumentsMessage::UpdateOpenDocumentsList.into()); match self.document_redo_history.pop() { - Some((document, layer_data)) => { + Some((document, layer_metadata)) => { let document = std::mem::replace(&mut self.graphene_document, document); - let layer_data = std::mem::replace(&mut self.layer_data, layer_data); - self.document_undo_history.push((document.clone(), layer_data.clone())); + let layer_metadata = std::mem::replace(&mut self.layer_metadata, layer_metadata); + self.document_undo_history.push((document, layer_metadata)); Ok(()) } None => Err(EditorError::NoTransactionInProgress), @@ -496,7 +491,7 @@ impl DocumentMessageHandler { } pub fn layer_panel_entry(&mut self, path: Vec) -> Result { - let data: LayerData = *self.layer_data_mut(&path); + let data: LayerMetadata = *self.layer_metadata_mut(&path); let layer = self.graphene_document.layer(&path)?; let entry = layer_panel_entry(&data, self.graphene_document.multiply_transforms(&path)?, layer, path); Ok(entry) @@ -512,14 +507,14 @@ impl DocumentMessageHandler { } pub fn layer_panel_entry_from_path(&self, path: &[LayerId]) -> Option { - let layer_data = self.layer_data(path); + let layer_metadata = self.layer_metadata(path); let transform = self .graphene_document .generate_transform_across_scope(path, Some(self.graphene_document.root.transform.inverse())) .ok()?; let layer = self.graphene_document.layer(path).ok()?; - Some(layer_panel_entry(layer_data, transform, layer, path.to_vec())) + Some(layer_panel_entry(layer_metadata, transform, layer, path.to_vec())) } } @@ -530,7 +525,7 @@ impl MessageHandler for DocumentMessageHand Movement(message) => self.movement_handler.process_action(message, (&self.graphene_document, ipp), responses), TransformLayers(message) => self .transform_layer_handler - .process_action(message, (&mut self.layer_data, &mut self.graphene_document, ipp), responses), + .process_action(message, (&mut self.layer_metadata, &mut self.graphene_document, ipp), responses), DeleteLayer(path) => responses.push_back(DocumentOperation::DeleteLayer { path }.into()), StartTransaction => self.backup(responses), RollbackTransaction => { @@ -543,8 +538,11 @@ impl MessageHandler for DocumentMessageHand } CommitTransaction => (), Overlay(message) => { - self.overlay_message_handler - .process_action(message, (Self::layer_data_mut_no_borrow_self(&mut self.layer_data, &[]), &self.graphene_document, ipp), responses); + self.overlay_message_handler.process_action( + message, + (Self::layer_metadata_mut_no_borrow_self(&mut self.layer_metadata, &[]), &self.graphene_document, ipp), + responses, + ); // responses.push_back(OverlayMessage::RenderOverlays.into()); } ExportDocument => { @@ -621,7 +619,7 @@ impl MessageHandler for DocumentMessageHand } SetBlendModeForSelectedLayers(blend_mode) => { self.backup(responses); - for path in self.layer_data.iter().filter_map(|(path, data)| data.selected.then(|| path.clone())) { + for path in self.layer_metadata.iter().filter_map(|(path, data)| data.selected.then(|| path.clone())) { responses.push_back(DocumentOperation::SetLayerBlendMode { path, blend_mode }.into()); } } @@ -638,12 +636,12 @@ impl MessageHandler for DocumentMessageHand responses.push_back(ToolMessage::DocumentIsDirty.into()); } ToggleLayerExpansion(path) => { - self.layer_data_mut(&path).expanded ^= true; + self.layer_metadata_mut(&path).expanded ^= true; responses.push_back(DocumentStructureChanged.into()); responses.push_back(LayerChanged(path).into()) } SetLayerExpansion(path, is_expanded) => { - self.layer_data_mut(&path).expanded = is_expanded; + self.layer_metadata_mut(&path).expanded = is_expanded; responses.push_back(DocumentStructureChanged.into()); responses.push_back(LayerChanged(path).into()) } @@ -677,7 +675,7 @@ impl MessageHandler for DocumentMessageHand // If we have shift pressed and a layer already selected then fill the range if shift && last_selection_exists { // Fill the selection range - self.layer_data + self.layer_metadata .iter() .filter(|(target, _)| self.graphene_document.layer_is_between(target, &selected, &self.layer_range_selection_reference)) .for_each(|(layer_path, _)| { @@ -686,7 +684,7 @@ impl MessageHandler for DocumentMessageHand } else { if ctrl { // Toggle selection when holding ctrl - let layer = self.layer_data_mut(&selected); + let layer = self.layer_metadata_mut(&selected); layer.selected = !layer.selected; responses.push_back(LayerChanged(selected.clone()).into()); responses.push_back(ToolMessage::DocumentIsDirty.into()); @@ -708,12 +706,13 @@ impl MessageHandler for DocumentMessageHand } } } - UpdateLayerData { path, layer_data_entry } => { - self.layer_data.insert(path, layer_data_entry); + UpdateLayerMetadata { layer_path: path, layer_metadata } => { + self.layer_metadata.insert(path, layer_metadata); } SetSelectedLayers(paths) => { - self.layer_data.iter_mut().filter(|(_, layer_data)| layer_data.selected).for_each(|(path, layer_data)| { - layer_data.selected = false; + let selected = self.layer_metadata.iter_mut().filter(|(_, layer_metadata)| layer_metadata.selected); + selected.for_each(|(path, layer_metadata)| { + layer_metadata.selected = false; responses.push_back(LayerChanged(path.clone()).into()) }); @@ -728,7 +727,7 @@ impl MessageHandler for DocumentMessageHand responses.push_back(ToolMessage::DocumentIsDirty.into()); } DebugPrintDocument => { - log::debug!("{:#?}\n{:#?}", self.graphene_document, self.layer_data); + log::debug!("{:#?}\n{:#?}", self.graphene_document, self.layer_metadata); } SelectAllLayers => { let all_layer_paths = self.all_layers(); @@ -773,11 +772,11 @@ impl MessageHandler for DocumentMessageHand match &response { DocumentResponse::FolderChanged { path } => responses.push_back(FolderChanged(path.clone()).into()), DocumentResponse::DeletedLayer { path } => { - self.layer_data.remove(path); + self.layer_metadata.remove(path); } DocumentResponse::LayerChanged { path } => responses.push_back(LayerChanged(path.clone()).into()), DocumentResponse::CreatedLayer { path } => { - self.layer_data.insert(path.clone(), LayerData::new(false)); + self.layer_metadata.insert(path.clone(), LayerMetadata::new(false)); responses.push_back(LayerChanged(path.clone()).into()); self.layer_range_selection_reference = path.clone(); responses.push_back(SetSelectedLayers(vec![path.clone()]).into()); @@ -989,9 +988,9 @@ impl MessageHandler for DocumentMessageHand .into(), ); responses.push_back( - DocumentMessage::UpdateLayerData { - path: destination_path, - layer_data_entry: *self.layer_data(&target_layer), + DocumentMessage::UpdateLayerMetadata { + layer_path: destination_path, + layer_metadata: *self.layer_metadata(&target_layer), } .into(), ); @@ -1019,7 +1018,7 @@ impl MessageHandler for DocumentMessageHand MoveLayerInTree, ); - if self.layer_data.values().any(|data| data.selected) { + if self.layer_metadata.values().any(|data| data.selected) { let select = actions!(DocumentMessageDiscriminant; DeleteSelectedLayers, DuplicateSelectedLayers, diff --git a/editor/src/document/document_message_handler.rs b/editor/src/document/document_message_handler.rs index 2190b96cc6..56e1adae01 100644 --- a/editor/src/document/document_message_handler.rs +++ b/editor/src/document/document_message_handler.rs @@ -1,4 +1,4 @@ -use super::{DocumentMessageHandler, LayerData}; +use super::{DocumentMessageHandler, LayerMetadata}; use crate::consts::DEFAULT_DOCUMENT_NAME; use crate::frontend::frontend_message_handler::FrontendDocumentDetails; use crate::input::InputPreprocessor; @@ -68,7 +68,7 @@ pub struct DocumentsMessageHandler { #[derive(Debug, Clone, PartialEq, Serialize, Deserialize)] pub struct CopyBufferEntry { layer: Layer, - layer_data: LayerData, + layer_metadata: LayerMetadata, } impl DocumentsMessageHandler { @@ -121,7 +121,7 @@ impl DocumentsMessageHandler { responses.extend( new_document - .layer_data + .layer_metadata .keys() .filter_map(|path| new_document.layer_panel_entry_from_path(path)) .map(|entry| FrontendMessage::UpdateLayer { data: entry }.into()) @@ -193,7 +193,7 @@ impl MessageHandler for DocumentsMessageHa responses.push_back(FrontendMessage::SetActiveDocument { document_id: id }.into()); responses.push_back(RenderDocument.into()); responses.push_back(DocumentMessage::DocumentStructureChanged.into()); - for layer in self.active_document().layer_data.keys() { + for layer in self.active_document().layer_metadata.keys() { responses.push_back(DocumentMessage::LayerChanged(layer.clone()).into()); } } @@ -249,7 +249,7 @@ impl MessageHandler for DocumentsMessageHa .document_ids .iter() .filter_map(|id| { - self.documents.get(&id).map(|doc| FrontendDocumentDetails { + self.documents.get(id).map(|doc| FrontendDocumentDetails { is_saved: doc.is_saved(), id: *id, name: doc.name.clone(), @@ -262,7 +262,7 @@ impl MessageHandler for DocumentsMessageHa responses.push_back(FrontendMessage::RemoveAutoSaveDocument { document_id: id }.into()); responses.push_back(RenderDocument.into()); responses.push_back(DocumentMessage::DocumentStructureChanged.into()); - for layer in self.active_document().layer_data.keys() { + for layer in self.active_document().layer_metadata.keys() { responses.push_back(DocumentMessage::LayerChanged(layer.clone()).into()); } } @@ -314,7 +314,7 @@ impl MessageHandler for DocumentsMessageHa .document_ids .iter() .filter_map(|id| { - self.documents.get(&id).map(|doc| FrontendDocumentDetails { + self.documents.get(id).map(|doc| FrontendDocumentDetails { is_saved: doc.is_saved(), id: *id, name: doc.name.clone(), @@ -359,9 +359,9 @@ impl MessageHandler for DocumentsMessageHa copy_buffer[clipboard as usize].clear(); for layer_path in active_document.selected_layers_without_children() { - match (active_document.graphene_document.layer(&layer_path).map(|t| t.clone()), *active_document.layer_data(&layer_path)) { - (Ok(layer), layer_data) => { - copy_buffer[clipboard as usize].push(CopyBufferEntry { layer, layer_data }); + match (active_document.graphene_document.layer(&layer_path).map(|t| t.clone()), *active_document.layer_metadata(&layer_path)) { + (Ok(layer), layer_metadata) => { + copy_buffer[clipboard as usize].push(CopyBufferEntry { layer, layer_metadata }); } (Err(e), _) => warn!("Could not access selected layer {:?}: {:?}", layer_path, e), } @@ -407,9 +407,9 @@ impl MessageHandler for DocumentsMessageHa .into(), ); responses.push_back( - DocumentMessage::UpdateLayerData { - path: destination_path, - layer_data_entry: entry.layer_data, + DocumentMessage::UpdateLayerMetadata { + layer_path: destination_path, + layer_metadata: entry.layer_metadata, } .into(), ); @@ -439,7 +439,7 @@ impl MessageHandler for DocumentsMessageHa Paste, ); - if self.active_document().layer_data.values().any(|data| data.selected) { + if self.active_document().layer_metadata.values().any(|data| data.selected) { let select = actions!(DocumentsMessageDiscriminant; Copy, Cut, diff --git a/editor/src/document/layer_panel.rs b/editor/src/document/layer_panel.rs index ad094380b3..8bc9841cd4 100644 --- a/editor/src/document/layer_panel.rs +++ b/editor/src/document/layer_panel.rs @@ -1,22 +1,24 @@ -use glam::{DAffine2, DVec2}; -use graphene::layers::{style::ViewMode, BlendMode, Layer, LayerData as DocumentLayerData, LayerDataType}; +use graphene::layers::{style::ViewMode, BlendMode, Layer, LayerData, LayerDataType}; use graphene::LayerId; -use serde::{ser::SerializeStruct, Deserialize, Serialize}; + use std::fmt; +use glam::{DAffine2, DVec2}; +use serde::{ser::SerializeStruct, Deserialize, Serialize}; + #[derive(Debug, Clone, Serialize, Deserialize, PartialEq, Copy)] -pub struct LayerData { +pub struct LayerMetadata { pub selected: bool, pub expanded: bool, } -impl LayerData { - pub fn new(expanded: bool) -> LayerData { - LayerData { selected: false, expanded } +impl LayerMetadata { + pub fn new(expanded: bool) -> LayerMetadata { + LayerMetadata { selected: false, expanded } } } -pub fn layer_panel_entry(layer_data: &LayerData, transform: DAffine2, layer: &Layer, path: Vec) -> LayerPanelEntry { +pub fn layer_panel_entry(layer_metadata: &LayerMetadata, transform: DAffine2, layer: &Layer, path: Vec) -> LayerPanelEntry { let layer_type: LayerDataTypeDiscriminant = (&layer.data).into(); let name = layer.name.clone().unwrap_or_else(|| format!("Unnamed {}", layer_type)); let arr = layer.data.bounding_box(transform).unwrap_or([DVec2::ZERO, DVec2::ZERO]); @@ -45,7 +47,7 @@ pub fn layer_panel_entry(layer_data: &LayerData, transform: DAffine2, layer: &La blend_mode: layer.blend_mode, opacity: layer.opacity, layer_type: (&layer.data).into(), - layer_data: *layer_data, + layer_metadata: *layer_metadata, path, thumbnail, } @@ -85,7 +87,7 @@ pub struct LayerPanelEntry { pub blend_mode: BlendMode, pub opacity: f64, pub layer_type: LayerDataTypeDiscriminant, - pub layer_data: LayerData, + pub layer_metadata: LayerMetadata, pub path: Vec, pub thumbnail: String, } diff --git a/editor/src/document/mod.rs b/editor/src/document/mod.rs index 1cc14d9849..35cc531e00 100644 --- a/editor/src/document/mod.rs +++ b/editor/src/document/mod.rs @@ -4,10 +4,10 @@ pub mod layer_panel; mod movement_handler; mod overlay_message_handler; mod transform_layer_handler; -mod vectorize_layerdata; +mod vectorize_layer_metadata; #[doc(inline)] -pub use document_file::LayerData; +pub use document_file::LayerMetadata; #[doc(inline)] pub use document_file::{AlignAggregate, AlignAxis, DocumentMessage, DocumentMessageDiscriminant, DocumentMessageHandler, FlipAxis, VectorManipulatorSegment, VectorManipulatorShape}; diff --git a/editor/src/document/overlay_message_handler.rs b/editor/src/document/overlay_message_handler.rs index 6bd3dbaed5..77f9d2d7ca 100644 --- a/editor/src/document/overlay_message_handler.rs +++ b/editor/src/document/overlay_message_handler.rs @@ -1,5 +1,5 @@ pub use crate::document::layer_panel::*; -use crate::document::{DocumentMessage, LayerData}; +use crate::document::{DocumentMessage, LayerMetadata}; use crate::input::InputPreprocessor; use crate::message_prelude::*; use graphene::document::Document; @@ -29,9 +29,9 @@ pub struct OverlayMessageHandler { overlay_path_mapping: HashMap, Vec>, } -impl MessageHandler for OverlayMessageHandler { - fn process_action(&mut self, message: OverlayMessage, data: (&mut LayerData, &Document, &InputPreprocessor), responses: &mut VecDeque) { - let (layer_data, document, ipp) = data; +impl MessageHandler for OverlayMessageHandler { + fn process_action(&mut self, message: OverlayMessage, _data: (&mut LayerMetadata, &Document, &InputPreprocessor), responses: &mut VecDeque) { + // let (layer_metadata, document, ipp) = data; use OverlayMessage::*; match message { DispatchOperation(operation) => match self.overlays_graphene_document.handle_operation(&operation) { diff --git a/editor/src/document/transform_layer_handler.rs b/editor/src/document/transform_layer_handler.rs index 30af14854d..f2b38902a9 100644 --- a/editor/src/document/transform_layer_handler.rs +++ b/editor/src/document/transform_layer_handler.rs @@ -1,6 +1,6 @@ pub use super::layer_panel::*; -use super::LayerData; +use super::LayerMetadata; use crate::consts::{ROTATE_SNAP_ANGLE, SCALE_SNAP_INTERVAL, SLOWING_DIVISOR}; use crate::input::keyboard::Key; @@ -25,11 +25,11 @@ impl<'a> Selected<'a> { pub fn new( original_transforms: &'a mut OriginalTransforms, pivot: &'a mut DVec2, - layer_data: &'a mut HashMap, LayerData>, + layer_metadata: &'a mut HashMap, LayerMetadata>, responses: &'a mut VecDeque, document: &'a mut Document, ) -> Self { - let selected = layer_data.iter().filter_map(|(layer_path, data)| data.selected.then(|| layer_path.to_owned())).collect(); + let selected = layer_metadata.iter().filter_map(|(layer_path, data)| data.selected.then(|| layer_path.to_owned())).collect(); for path in &selected { if !original_transforms.contains_key::>(path) { original_transforms.insert(path.clone(), document.layer(path).unwrap().transform); @@ -394,12 +394,12 @@ pub struct TransformLayerMessageHandler { pivot: DVec2, } -impl MessageHandler, LayerData>, &mut Document, &InputPreprocessor)> for TransformLayerMessageHandler { - fn process_action(&mut self, message: TransformLayerMessage, data: (&mut HashMap, LayerData>, &mut Document, &InputPreprocessor), responses: &mut VecDeque) { +impl MessageHandler, LayerMetadata>, &mut Document, &InputPreprocessor)> for TransformLayerMessageHandler { + fn process_action(&mut self, message: TransformLayerMessage, data: (&mut HashMap, LayerMetadata>, &mut Document, &InputPreprocessor), responses: &mut VecDeque) { use TransformLayerMessage::*; - let (layer_data, document, ipp) = data; - let mut selected = Selected::new(&mut self.original_transforms, &mut self.pivot, layer_data, responses, document); + let (layer_metadata, document, ipp) = data; + let mut selected = Selected::new(&mut self.original_transforms, &mut self.pivot, layer_metadata, responses, document); let mut begin_operation = |operation: Operation, typing: &mut Typing, mouse_position: &mut DVec2, start_mouse: &mut DVec2| { if !(operation == Operation::None) { diff --git a/editor/src/document/vectorize_layerdata.rs b/editor/src/document/vectorize_layer_metadata.rs similarity index 100% rename from editor/src/document/vectorize_layerdata.rs rename to editor/src/document/vectorize_layer_metadata.rs diff --git a/editor/src/global/global_message_handler.rs b/editor/src/global/global_message_handler.rs index f447a43e94..6fe1fb1e54 100644 --- a/editor/src/global/global_message_handler.rs +++ b/editor/src/global/global_message_handler.rs @@ -13,12 +13,6 @@ pub enum GlobalMessage { #[derive(Debug, Default)] pub struct GlobalMessageHandler {} -impl GlobalMessageHandler { - pub fn new() -> Self { - Self::default() - } -} - impl MessageHandler for GlobalMessageHandler { fn process_action(&mut self, message: GlobalMessage, _data: (), _responses: &mut VecDeque) { use GlobalMessage::*; diff --git a/editor/src/input/keyboard.rs b/editor/src/input/keyboard.rs index f0ec9b99a1..3198b8cdb3 100644 --- a/editor/src/input/keyboard.rs +++ b/editor/src/input/keyboard.rs @@ -197,7 +197,7 @@ macro_rules! bit_ops { macro_rules! bit_ops_assign { ($(($op:ident, $func:ident)),* $(,)?) => { $(impl $op for BitVector { - fn $func(&mut self, right: Self) { + fn $func(&mut self, right: Self) { for (left, right) in self.0.iter_mut().zip(right.0.iter()) { $op::$func(left, right); } diff --git a/editor/src/input/mouse.rs b/editor/src/input/mouse.rs index ca1d514b83..023a9a3560 100644 --- a/editor/src/input/mouse.rs +++ b/editor/src/input/mouse.rs @@ -31,6 +31,7 @@ impl ViewportBounds { #[derive(Debug, Copy, Clone, Default, Eq, PartialEq, Hash, Serialize, Deserialize)] pub struct ScrollDelta { + // TODO: Switch these to `f64` values (not trivial because floats don't provide PartialEq, Eq, and Hash) pub x: i32, pub y: i32, pub z: i32, diff --git a/editor/src/tool/snapping.rs b/editor/src/tool/snapping.rs index 3f4792faa5..1597361bc9 100644 --- a/editor/src/tool/snapping.rs +++ b/editor/src/tool/snapping.rs @@ -58,13 +58,11 @@ impl SnapHandler { .unwrap_or(0.), ); - // Do not move if over snap tolerance - let clamped_closest_move = DVec2::new( + // Clamp, do not move if over snap tolerance + DVec2::new( if closest_move.x.abs() > SNAP_TOLERANCE { 0. } else { closest_move.x }, if closest_move.y.abs() > SNAP_TOLERANCE { 0. } else { closest_move.y }, - ); - - clamped_closest_move + ) } else { DVec2::ZERO } diff --git a/editor/src/tool/tools/line.rs b/editor/src/tool/tools/line.rs index 8053ce1976..c348a8d6b6 100644 --- a/editor/src/tool/tools/line.rs +++ b/editor/src/tool/tools/line.rs @@ -42,7 +42,7 @@ impl<'a> MessageHandler> for Line { fn actions(&self) -> ActionList { use LineToolFsmState::*; match self.fsm_state { - Ready => actions!(LineMessageDiscriminant; DragStart), + Ready => actions!(LineMessageDiscriminant; DragStart), Drawing => actions!(LineMessageDiscriminant; DragStop, Redraw, Abort), } } diff --git a/editor/src/tool/tools/path.rs b/editor/src/tool/tools/path.rs index 24fafc225a..713acb539d 100644 --- a/editor/src/tool/tools/path.rs +++ b/editor/src/tool/tools/path.rs @@ -303,7 +303,7 @@ impl Fsm for PathToolFsmState { } } -fn calculate_total_overlays_per_type(shapes_to_draw: &Vec) -> (usize, usize, usize) { +fn calculate_total_overlays_per_type(shapes_to_draw: &[VectorManipulatorShape]) -> (usize, usize, usize) { let (mut total_anchors, mut total_handles, mut total_anchor_handle_lines) = (0, 0, 0); for shape_to_draw in shapes_to_draw { diff --git a/frontend/.eslintrc.js b/frontend/.eslintrc.js index ddb7572817..358fe5ae78 100644 --- a/frontend/.eslintrc.js +++ b/frontend/.eslintrc.js @@ -73,9 +73,54 @@ module.exports = { "@typescript-eslint/no-use-before-define": "off", "@typescript-eslint/no-unused-vars": ["error", { argsIgnorePattern: "^_" }], "@typescript-eslint/no-loss-of-precision": "off", // TODO: Remove this line after upgrading to eslint 7.1 or greater + "@typescript-eslint/explicit-function-return-type": ["error"], // Import plugin config (used to intelligently validate module import statements) "import/prefer-default-export": "off", + "import/no-relative-packages": "error", + "import/order": [ + "error", + { + alphabetize: { + order: "asc", + caseInsensitive: true, + }, + warnOnUnassignedImports: true, + "newlines-between": "always-and-inside-groups", + pathGroups: [ + { + pattern: "**/*.vue", + group: "unknown", + position: "after", + }, + { + pattern: "**/assets/12px-solid/*.svg", + group: "unknown", + position: "after", + }, + { + pattern: "**/assets/16px-solid/*.svg", + group: "unknown", + position: "after", + }, + { + pattern: "**/assets/16px-two-tone/*.svg", + group: "unknown", + position: "after", + }, + { + pattern: "**/assets/24px-full-color/*.svg", + group: "unknown", + position: "after", + }, + { + pattern: "**/assets/24px-two-tone/*.svg", + group: "unknown", + position: "after", + }, + ], + }, + ], // Prettier plugin config (used to enforce HTML, CSS, and JS formatting styles as an ESLint plugin, where fixes are reported to ESLint to be applied when linting) "prettier-vue/prettier": [ @@ -90,4 +135,12 @@ module.exports = { // Vue plugin config (used to validate Vue single-file components) "vue/multi-word-component-names": "off", }, + overrides: [ + { + files: ["*.js"], + rules: { + "@typescript-eslint/explicit-function-return-type": ["off"], + }, + }, + ], }; diff --git a/frontend/src/App.vue b/frontend/src/App.vue index 9ee7b9df98..77d472ad97 100644 --- a/frontend/src/App.vue +++ b/frontend/src/App.vue @@ -221,16 +221,16 @@ img { diff --git a/frontend/src/components/panels/Properties.vue b/frontend/src/components/panels/Properties.vue index fdd88d3382..0763d62cd8 100644 --- a/frontend/src/components/panels/Properties.vue +++ b/frontend/src/components/panels/Properties.vue @@ -7,8 +7,5 @@ diff --git a/frontend/src/components/widgets/buttons/IconButton.vue b/frontend/src/components/widgets/buttons/IconButton.vue index d5f174f743..4a0eec3f74 100644 --- a/frontend/src/components/widgets/buttons/IconButton.vue +++ b/frontend/src/components/widgets/buttons/IconButton.vue @@ -1,5 +1,5 @@ @@ -57,16 +57,16 @@ diff --git a/frontend/src/components/widgets/buttons/TextButton.vue b/frontend/src/components/widgets/buttons/TextButton.vue index 7da0e453a9..257e4ef02a 100644 --- a/frontend/src/components/widgets/buttons/TextButton.vue +++ b/frontend/src/components/widgets/buttons/TextButton.vue @@ -1,5 +1,5 @@ @@ -49,18 +49,18 @@ diff --git a/frontend/src/components/widgets/floating-menus/FloatingMenu.vue b/frontend/src/components/widgets/floating-menus/FloatingMenu.vue index 5d1fe02e5c..67166d49c6 100644 --- a/frontend/src/components/widgets/floating-menus/FloatingMenu.vue +++ b/frontend/src/components/widgets/floating-menus/FloatingMenu.vue @@ -1,6 +1,6 @@ - + @@ -95,28 +95,17 @@ diff --git a/frontend/src/components/widgets/options/ToolOptions.vue b/frontend/src/components/widgets/options/ToolOptions.vue index 07c98dfd73..0c8ee88d56 100644 --- a/frontend/src/components/widgets/options/ToolOptions.vue +++ b/frontend/src/components/widgets/options/ToolOptions.vue @@ -31,17 +31,19 @@ diff --git a/frontend/src/components/window/MainWindow.vue b/frontend/src/components/window/MainWindow.vue index 54b2664df4..aa7b5e749a 100644 --- a/frontend/src/components/window/MainWindow.vue +++ b/frontend/src/components/window/MainWindow.vue @@ -39,18 +39,13 @@ diff --git a/frontend/src/components/window/title-bar/WindowButtonsWeb.vue b/frontend/src/components/window/title-bar/WindowButtonsWeb.vue index 45a601bb20..2c5334c848 100644 --- a/frontend/src/components/window/title-bar/WindowButtonsWeb.vue +++ b/frontend/src/components/window/title-bar/WindowButtonsWeb.vue @@ -1,5 +1,5 @@