Explorar el Código

PixelPaint: Add luminosity masking for editing masks

This adds a function where editing masks can be refined by selecting
a luminosity range that is applied to the content image and mapped to
the editing mask. This function allows the editing of image regions
that match only certain luminosity values.
Torstennator hace 2 años
padre
commit
dbbf54df2c

+ 3 - 0
Userland/Applications/PixelPaint/CMakeLists.txt

@@ -10,6 +10,7 @@ compile_gml(EditGuideDialog.gml EditGuideDialogGML.h edit_guide_dialog_gml)
 compile_gml(FilterGallery.gml FilterGalleryGML.h filter_gallery_gml)
 compile_gml(ResizeImageDialog.gml ResizeImageDialogGML.h resize_image_dialog_gml)
 compile_gml(LevelsDialog.gml LevelsDialogGML.h levels_dialog_gml)
+compile_gml(LuminosityMasking.gml LuminosityMaskingGML.h luminosity_masking_gml)
 compile_gml(Filters/MedianSettings.gml Filters/MedianSettingsGML.h median_settings_gml)
 
 set(SOURCES
@@ -38,6 +39,7 @@ set(SOURCES
     IconBag.cpp
     Image.cpp
     ImageEditor.cpp
+    ImageMasking.cpp
     ImageProcessor.cpp
     Layer.cpp
     LayerListWidget.cpp
@@ -81,6 +83,7 @@ set(GENERATED_SOURCES
     FilterGalleryGML.h
     Filters/MedianSettingsGML.h
     LevelsDialogGML.h
+    LuminosityMaskingGML.h
     PixelPaintWindowGML.h
     ResizeImageDialogGML.h
 )

+ 177 - 0
Userland/Applications/PixelPaint/ImageMasking.cpp

@@ -0,0 +1,177 @@
+/*
+ * Copyright (c) 2023, Torsten Engelmann <engelTorsten@gmx.de>
+ *
+ * SPDX-License-Identifier: BSD-2-Clause
+ */
+
+#include "ImageMasking.h"
+#include <Applications/PixelPaint/LuminosityMaskingGML.h>
+#include <LibGUI/Button.h>
+#include <LibGUI/CheckBox.h>
+#include <LibGUI/Label.h>
+#include <LibGUI/Painter.h>
+#include <LibGUI/RangeSlider.h>
+#include <LibGfx/Palette.h>
+#include <LibGfx/Path.h>
+
+namespace PixelPaint {
+
+ImageMasking::ImageMasking(GUI::Window* parent_window, ImageEditor* editor)
+    : GUI::Dialog(parent_window)
+{
+    set_title("Luminosity Mask");
+    set_icon(parent_window->icon());
+
+    auto main_widget = set_main_widget<GUI::Widget>().release_value_but_fixme_should_propagate_errors();
+    main_widget->load_from_gml(luminosity_masking_gml).release_value_but_fixme_should_propagate_errors();
+
+    resize(300, 170);
+    set_resizable(false);
+
+    m_editor = editor;
+
+    m_full_masking_slider = main_widget->find_descendant_of_type_named<GUI::RangeSlider>("full_masking");
+    m_edge_masking_slider = main_widget->find_descendant_of_type_named<GUI::RangeSlider>("edge_masking");
+    auto range_illustration_container = main_widget->find_descendant_of_type_named<GUI::Widget>("range_illustration");
+    auto mask_visibility = main_widget->find_descendant_of_type_named<GUI::CheckBox>("mask_visibility");
+    auto apply_button = main_widget->find_descendant_of_type_named<GUI::Button>("apply_button");
+    auto cancel_button = main_widget->find_descendant_of_type_named<GUI::Button>("cancel_button");
+
+    VERIFY(m_full_masking_slider);
+    VERIFY(m_edge_masking_slider);
+    VERIFY(range_illustration_container);
+    VERIFY(mask_visibility);
+    VERIFY(apply_button);
+    VERIFY(cancel_button);
+    VERIFY(m_editor->active_layer());
+
+    m_full_masking_slider->set_gradient_color(Color(0, 0, 0, 255), Color(255, 255, 255, 255));
+    m_edge_masking_slider->set_gradient_color(Color(0, 0, 0, 255), Color(255, 255, 255, 255));
+
+    auto illustration_widget = range_illustration_container->try_add<RangeIllustrationWidget>(m_edge_masking_slider, m_full_masking_slider).release_value();
+    illustration_widget->set_width(range_illustration_container->width());
+    illustration_widget->set_height(range_illustration_container->height());
+
+    // check that edges of full and edge masking are not intersecting, and refine the mask with the updated values
+    m_full_masking_slider->on_range_change = [this, illustration_widget](int lower, int upper) {
+        if (lower < m_edge_masking_slider->lower_range())
+            m_full_masking_slider->set_lower_range(AK::max(lower, m_edge_masking_slider->lower_range()));
+        if (upper > m_edge_masking_slider->upper_range())
+            m_full_masking_slider->set_upper_range(AK::min(upper, m_edge_masking_slider->upper_range()));
+
+        illustration_widget->update();
+        generate_new_mask();
+    };
+    m_edge_masking_slider->on_range_change = [this, illustration_widget](int lower, int upper) {
+        if (lower > m_full_masking_slider->lower_range())
+            m_edge_masking_slider->set_lower_range(AK::min(lower, m_full_masking_slider->lower_range()));
+        if (upper < m_full_masking_slider->upper_range())
+            m_edge_masking_slider->set_upper_range(AK::max(upper, m_full_masking_slider->upper_range()));
+
+        illustration_widget->update();
+        generate_new_mask();
+    };
+
+    mask_visibility->set_checked(m_editor->active_layer()->mask_visibility());
+    mask_visibility->on_checked = [this](auto checked) {
+        m_editor->active_layer()->set_mask_visibility(checked);
+        m_editor->update();
+    };
+
+    apply_button->on_click = [this](auto) {
+        if (m_did_change)
+            m_editor->did_complete_action("Luminosity Masking"sv);
+
+        cleanup_resources();
+        done(ExecResult::OK);
+    };
+
+    cancel_button->on_click = [this](auto) {
+        done(ExecResult::Cancel);
+    };
+
+    generate_new_mask();
+}
+
+void ImageMasking::revert_possible_changes()
+{
+    if (m_did_change && m_reference_mask) {
+        MUST(m_editor->active_layer()->set_bitmaps(m_editor->active_layer()->content_bitmap(), m_reference_mask.release_nonnull()));
+        m_editor->layers_did_change();
+    }
+    cleanup_resources();
+}
+
+void ImageMasking::generate_new_mask()
+{
+    ensure_reference_mask().release_value_but_fixme_should_propagate_errors();
+
+    if (m_reference_mask.is_null())
+        return;
+
+    int min_luminosity_start = m_edge_masking_slider->lower_range();
+    int min_luminosity_full = m_full_masking_slider->lower_range();
+    int max_luminosity_full = m_full_masking_slider->upper_range();
+    int max_luminosity_end = m_edge_masking_slider->upper_range();
+    int current_content_luminosity, approximation_alpha;
+    bool has_start_range = min_luminosity_start != min_luminosity_full;
+    bool has_end_range = max_luminosity_end != max_luminosity_full;
+    Gfx::Color reference_mask_pixel, content_pixel;
+
+    for (int y = 0; y < m_reference_mask->height(); y++) {
+        for (int x = 0; x < m_reference_mask->width(); x++) {
+            reference_mask_pixel = m_reference_mask->get_pixel(x, y);
+            if (!reference_mask_pixel.alpha())
+                continue;
+
+            content_pixel = m_editor->active_layer()->content_bitmap().get_pixel(x, y);
+            current_content_luminosity = content_pixel.luminosity();
+
+            if (!content_pixel.alpha() || current_content_luminosity < min_luminosity_start || current_content_luminosity > max_luminosity_end) {
+                reference_mask_pixel.set_alpha(0);
+            } else if (current_content_luminosity >= min_luminosity_start && current_content_luminosity < min_luminosity_full && has_start_range) {
+                approximation_alpha = reference_mask_pixel.alpha() * static_cast<float>((current_content_luminosity - min_luminosity_start)) / (min_luminosity_full - min_luminosity_start);
+                reference_mask_pixel.set_alpha(approximation_alpha);
+            } else if (current_content_luminosity > max_luminosity_full && current_content_luminosity <= max_luminosity_end && has_end_range) {
+                approximation_alpha = reference_mask_pixel.alpha() * (1 - static_cast<float>((current_content_luminosity - max_luminosity_full)) / (max_luminosity_end - max_luminosity_full));
+                reference_mask_pixel.set_alpha(approximation_alpha);
+            }
+
+            m_editor->active_layer()->mask_bitmap()->set_pixel(x, y, reference_mask_pixel);
+        }
+    }
+
+    m_editor->active_layer()->did_modify_bitmap();
+    m_did_change = true;
+}
+
+ErrorOr<void> ImageMasking::ensure_reference_mask()
+{
+    if (m_reference_mask.is_null())
+        m_reference_mask = TRY(m_editor->active_layer()->mask_bitmap()->clone());
+
+    return {};
+}
+
+void ImageMasking::cleanup_resources()
+{
+    if (m_reference_mask)
+        m_reference_mask = nullptr;
+}
+
+void RangeIllustrationWidget::paint_event(GUI::PaintEvent&)
+{
+    GUI::Painter painter(*this);
+    painter.fill_rect(Gfx::IntRect(0, 0, width(), height()), palette().color(background_role()));
+    float fraction = width() / 255.0f;
+
+    Gfx::Path illustration;
+    illustration.move_to({ fraction * m_edge_mask_values->lower_range(), static_cast<float>(height()) });
+    illustration.line_to({ fraction * m_full_mask_values->lower_range(), 0 });
+    illustration.line_to({ fraction * m_full_mask_values->upper_range(), 0 });
+    illustration.line_to({ fraction * m_edge_mask_values->upper_range(), static_cast<float>(height()) });
+    illustration.close();
+
+    painter.fill_path(illustration, Color::MidGray);
+}
+}

+ 56 - 0
Userland/Applications/PixelPaint/ImageMasking.h

@@ -0,0 +1,56 @@
+/*
+ * Copyright (c) 2023, Torsten Engelmann <engelTorsten@gmx.de>
+ *
+ * SPDX-License-Identifier: BSD-2-Clause
+ */
+
+#pragma once
+
+#include "ImageEditor.h"
+#include "Layer.h"
+#include <LibGUI/Dialog.h>
+#include <LibGUI/RangeSlider.h>
+#include <LibGUI/Widget.h>
+
+namespace PixelPaint {
+
+class ImageMasking final : public GUI::Dialog {
+    C_OBJECT(ImageMasking);
+
+public:
+    void revert_possible_changes();
+
+private:
+    ImageMasking(GUI::Window* parent_window, ImageEditor*);
+
+    ImageEditor* m_editor { nullptr };
+    RefPtr<Gfx::Bitmap> m_reference_mask { nullptr };
+    bool m_did_change = false;
+
+    RefPtr<GUI::RangeSlider> m_full_masking_slider = { nullptr };
+    RefPtr<GUI::RangeSlider> m_edge_masking_slider = { nullptr };
+
+    ErrorOr<void> ensure_reference_mask();
+    void generate_new_mask();
+    void cleanup_resources();
+};
+
+class RangeIllustrationWidget final : public GUI::Widget {
+    C_OBJECT(RangeIllustrationWidget)
+public:
+    virtual ~RangeIllustrationWidget() override = default;
+
+protected:
+    virtual void paint_event(GUI::PaintEvent&) override;
+
+private:
+    RangeIllustrationWidget(RefPtr<GUI::RangeSlider> edge_mask_values, RefPtr<GUI::RangeSlider> full_mask_values)
+    {
+        m_edge_mask_values = edge_mask_values;
+        m_full_mask_values = full_mask_values;
+    }
+    RefPtr<GUI::RangeSlider> m_edge_mask_values;
+    RefPtr<GUI::RangeSlider> m_full_mask_values;
+};
+
+}

+ 65 - 0
Userland/Applications/PixelPaint/LuminosityMasking.gml

@@ -0,0 +1,65 @@
+@GUI::Frame {
+    fill_with_background_color: true
+    layout: @GUI::VerticalBoxLayout {
+        margins: [4]
+    }
+
+    @GUI::Widget {
+        layout: @GUI::VerticalBoxLayout {}
+
+        @GUI::Label {
+            name: "hint_label"
+            enabled: true
+            fixed_height: 20
+            visible: true
+            text: "Restrict mask to luminosity values:"
+            text_alignment: "CenterLeft"
+        }
+
+        @GUI::HorizontalRangeSlider {
+            name: "full_masking"
+            max: 255
+            min: 0
+            lower_range: 25
+            upper_range: 230
+            page_step: 10
+        }
+
+        @GUI::Widget {
+            name: "range_illustration"
+        }
+
+        @GUI::HorizontalRangeSlider {
+            name: "edge_masking"
+            max: 255
+            min: 0
+            lower_range: 0
+            upper_range: 255
+            page_step: 10
+        }
+
+        @GUI::CheckBox {
+            name: "mask_visibility"
+            text: "Show layer mask"
+        }
+
+        @GUI::HorizontalSeparator {}
+    }
+
+    @GUI::Widget {
+        layout: @GUI::HorizontalBoxLayout {}
+        fixed_height: 22
+
+        @GUI::Layout::Spacer {}
+
+        @GUI::DialogButton {
+            name: "apply_button"
+            text: "OK"
+        }
+
+        @GUI::DialogButton {
+            name: "cancel_button"
+            text: "Cancel"
+        }
+    }
+}

+ 15 - 0
Userland/Applications/PixelPaint/MainWidget.cpp

@@ -13,6 +13,7 @@
 #include "EditGuideDialog.h"
 #include "FilterGallery.h"
 #include "FilterParams.h"
+#include "ImageMasking.h"
 #include "LevelsDialog.h"
 #include "ResizeImageDialog.h"
 #include <AK/String.h>
@@ -867,6 +868,19 @@ ErrorOr<void> MainWidget::initialize_menubar(GUI::Window& window)
 
     TRY(m_layer_menu->try_add_action(*m_toggle_mask_visibility_action));
 
+    m_open_luminosity_masking_action = GUI::Action::create(
+        "Luminosity Masking", create_layer_mask_callback("Luminosity Masking", [&](Layer* active_layer) {
+            VERIFY(active_layer->mask_type() == Layer::MaskType::EditingMask);
+
+            auto* editor = current_image_editor();
+            VERIFY(editor);
+            auto dialog = PixelPaint::ImageMasking::construct(&window, editor);
+            if (dialog->exec() != GUI::Dialog::ExecResult::OK)
+                dialog->revert_possible_changes();
+        }));
+
+    TRY(m_layer_menu->try_add_action(*m_open_luminosity_masking_action));
+
     TRY(m_layer_menu->try_add_separator());
 
     TRY(m_layer_menu->try_add_action(GUI::Action::create(
@@ -1248,6 +1262,7 @@ void MainWidget::set_mask_actions_for_layer(Layer* layer)
     m_apply_mask_action->set_visible(layer->mask_type() == Layer::MaskType::BasicMask);
     m_toggle_mask_visibility_action->set_visible(layer->mask_type() == Layer::MaskType::EditingMask);
     m_toggle_mask_visibility_action->set_checked(layer->mask_visibility());
+    m_open_luminosity_masking_action->set_visible(layer->mask_type() == Layer::MaskType::EditingMask);
 }
 
 void MainWidget::open_image(FileSystemAccessClient::File file)

+ 1 - 0
Userland/Applications/PixelPaint/MainWidget.h

@@ -118,6 +118,7 @@ private:
     RefPtr<GUI::Action> m_invert_mask_action;
     RefPtr<GUI::Action> m_clear_mask_action;
     RefPtr<GUI::Action> m_toggle_mask_visibility_action;
+    RefPtr<GUI::Action> m_open_luminosity_masking_action;
 
     Gfx::IntPoint m_last_image_editor_mouse_position;
 };