From e73aeb4cea77d7cc38931771178d1e7a489498e9 Mon Sep 17 00:00:00 2001 From: Timothy Slater Date: Wed, 26 Oct 2022 07:53:39 -0500 Subject: [PATCH] PixelPaint: Make scopes hideable Vectorscope and Histogram are now hidden by default. New menu "Scopes" allows for them to be toggled on/off. --- Userland/Applications/PixelPaint/MainWidget.cpp | 11 +++++++++++ Userland/Applications/PixelPaint/PixelPaintWindow.gml | 2 ++ 2 files changed, 13 insertions(+) diff --git a/Userland/Applications/PixelPaint/MainWidget.cpp b/Userland/Applications/PixelPaint/MainWidget.cpp index 9b21e9d1c3..d4dff77470 100644 --- a/Userland/Applications/PixelPaint/MainWidget.cpp +++ b/Userland/Applications/PixelPaint/MainWidget.cpp @@ -509,6 +509,17 @@ void MainWidget::initialize_menubar(GUI::Window& window) m_show_active_layer_boundary_action->set_checked(Config::read_bool("PixelPaint"sv, "ImageEditor"sv, "ShowActiveLayerBoundary"sv, true)); m_view_menu->add_action(*m_show_active_layer_boundary_action); + m_view_menu->add_separator(); + auto& scopes_menu = m_view_menu->add_submenu("&Scopes"); + + scopes_menu.add_action(GUI::Action::create_checkable("&Histogram", [&](auto& action) { + m_histogram_widget->parent_widget()->set_visible(action.is_checked()); + })); + + scopes_menu.add_action(GUI::Action::create_checkable("&Vectorscope", [&](auto& action) { + m_vectorscope_widget->parent_widget()->set_visible(action.is_checked()); + })); + m_tool_menu = window.add_menu("&Tool"); m_toolbox->for_each_tool([&](auto& tool) { if (tool.action()) diff --git a/Userland/Applications/PixelPaint/PixelPaintWindow.gml b/Userland/Applications/PixelPaint/PixelPaintWindow.gml index 668a3e0a60..8fe3166c22 100644 --- a/Userland/Applications/PixelPaint/PixelPaintWindow.gml +++ b/Userland/Applications/PixelPaint/PixelPaintWindow.gml @@ -67,6 +67,7 @@ @GUI::GroupBox { title: "Histogram" preferred_height: "shrink" + visible: false layout: @GUI::VerticalBoxLayout { margins: [6] } @@ -80,6 +81,7 @@ @GUI::GroupBox { title: "Vectorscope" min_height: 80 + visible: false layout: @GUI::VerticalBoxLayout { margins: [6] }