jess
/
Graphite
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix bug where duplication with Ctrl+D doesn't properly duplicate (#423) 

* bug fix: duplication didn't properly duplicate

* cargo fmt

* changed the formatting slightly for readability
This commit is contained in:
caleb 2021-12-26 14:39:07 -07:00 committed by Keavon Chambers
parent 04c1b2ed03
commit ff39ebfdbb
1 changed files with 13 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
graphene/src/document.rs
View File

@ -525,9 +525,19 @@ impl Document {
let layer = self.layer(path)?.clone();
let (folder_path, _) = split_path(path.as_slice()).unwrap_or_else(|_| (&[], 0));
let folder = self.folder_mut(folder_path)?;
folder.add_layer(layer, None, -1).ok_or(DocumentError::IndexOutOfBounds)?;
self.mark_as_dirty(&path[..path.len() - 1])?;
Some(vec![DocumentChanged, FolderChanged { path: folder_path.to_vec() }])
if let Some(new_layer_id) = folder.add_layer(layer, None, -1) {
let new_path = [folder_path, &[new_layer_id]].concat();
self.mark_as_dirty(folder_path)?;
Some(
[
vec![DocumentChanged, CreatedLayer { path: new_path }, FolderChanged { path: folder_path.to_vec() }],
update_thumbnails_upstream(path.as_slice()),
]
.concat(),
)
} else {
return Err(DocumentError::IndexOutOfBounds);
}
}
Operation::RenameLayer { path, name } => {
self.layer_mut(path)?.name = Some(name.clone());