diff --git a/Userland/Applications/PixelPaint/LayerListWidget.cpp b/Userland/Applications/PixelPaint/LayerListWidget.cpp index 3bd65ff67e..edb26969fd 100644 --- a/Userland/Applications/PixelPaint/LayerListWidget.cpp +++ b/Userland/Applications/PixelPaint/LayerListWidget.cpp @@ -277,9 +277,7 @@ size_t LayerListWidget::hole_index_during_move() const VERIFY(is_moving_gadget()); auto& moving_gadget = m_gadgets[m_moving_gadget_index.value()]; int center_y_of_moving_gadget = moving_gadget.rect.translated(0, moving_gadget.movement_delta.y()).center().y(); - - int top_of_gadgets = max(0, height() - m_total_gadget_height); - return (center_y_of_moving_gadget - top_of_gadgets) / vertical_step; + return center_y_of_moving_gadget / vertical_step; } void LayerListWidget::select_bottom_layer() @@ -318,9 +316,7 @@ void LayerListWidget::cycle_through_selection(int delta) void LayerListWidget::relayout_gadgets() { - m_total_gadget_height = static_cast(m_gadgets.size()) * vertical_step + 6; - int y = max(0, height() - m_total_gadget_height); - + int y = 0; Optional hole_index; if (is_moving_gadget()) hole_index = hole_index_during_move(); @@ -336,8 +332,9 @@ void LayerListWidget::relayout_gadgets() ++index; } - set_content_size({ widget_inner_rect().width(), m_total_gadget_height }); - vertical_scrollbar().set_range(0, max(m_total_gadget_height - height(), 0)); + auto total_gadget_height = static_cast(m_gadgets.size()) * vertical_step + 6; + set_content_size({ widget_inner_rect().width(), total_gadget_height }); + vertical_scrollbar().set_range(0, max(total_gadget_height - height(), 0)); update(); } diff --git a/Userland/Applications/PixelPaint/LayerListWidget.h b/Userland/Applications/PixelPaint/LayerListWidget.h index 458e923f5c..27144a3410 100644 --- a/Userland/Applications/PixelPaint/LayerListWidget.h +++ b/Userland/Applications/PixelPaint/LayerListWidget.h @@ -73,7 +73,6 @@ private: Gfx::IntPoint m_moving_event_origin; size_t m_selected_gadget_index { 0 }; - int m_total_gadget_height { 0 }; }; }