RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-25 16:17:45 +00:00
Code Issues Activity Actions

PixelPaint: Ensure a layer is selected when restoring a snapshot

Browse source 
Previously when restoring the starting snapshot in the UndoStack we
would end up with no layer selected, which was kinda annoying
This commit is contained in:
Marco Cutecchia 2021-09-25 19:39:11 +02:00 • committed by Andreas Kling
parent 6108bac606
commit c7431da3f2
1 changed files with 8 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

9
Userland/Applications/PixelPaint/Image.cpp
View file

@ -259,14 +259,21 @@ void Image::restore_snapshot(Image const& snapshot)
{ {
m_layers.clear(); m_layers.clear();
select_layer(nullptr); select_layer(nullptr);
bool layer_selected = false;
for (const auto& snapshot_layer : snapshot.m_layers) { for (const auto& snapshot_layer : snapshot.m_layers) {
auto layer = Layer::try_create_snapshot(*this, snapshot_layer); auto layer = Layer::try_create_snapshot(*this, snapshot_layer);
VERIFY(layer); VERIFY(layer);
if (layer->is_selected()) if (layer->is_selected()) {
select_layer(layer.ptr()); select_layer(layer.ptr());
layer_selected = true;
}
add_layer(*layer); add_layer(*layer);
} }
if (!layer_selected)
select_layer(&layer(0));
did_modify_layer_stack(); did_modify_layer_stack();
} }