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

PixelPaint: Don't select the same layer twice in LayerListWidget

Browse source 
Without this check we would do an unnecessary partial second round
trip because of the call chain:
LayerListWidget::set_selected_layer() ->
LayerListWidget::on_layer_select() ->
ImageEditor::set_active_layer() ->
ImageEditor::on_active_layer_change() ->
LayerListWidget::set_selected_layer()
This commit is contained in:
Marcus Nilsson 2021-08-15 18:22:12 +02:00 • committed by Andreas Kling
parent eaa99968fb
commit b76fe6357a
1 changed files with 4 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

4
Userland/Applications/PixelPaint/LayerListWidget.cpp
View file

@ -342,6 +342,10 @@ void LayerListWidget::set_selected_layer(Layer* layer)
{ {
if (!m_image) if (!m_image)
return; return;
if (layer->is_selected())
return;
for (size_t i = 0; i < m_image->layer_count(); ++i) { for (size_t i = 0; i < m_image->layer_count(); ++i) {
if (layer == &m_image->layer(i)) { if (layer == &m_image->layer(i)) {
m_image->layer(i).set_selected(true); m_image->layer(i).set_selected(true);