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

PixelPaint: Remove context menu for SprayTool

Browse source 
Remove the context menu for SprayTool and just use the tool properties
widget for options.
This commit is contained in:
Marcus Nilsson 2021-08-02 21:16:46 +02:00 • committed by Andreas Kling
parent b257d8d284
commit 15e9d0b4d8
2 changed files with 0 additions and 24 deletions
Download patch file Download diff file Expand all files Collapse all files

21
Userland/Applications/PixelPaint/SprayTool.cpp
View file

@ -88,27 +88,6 @@ void SprayTool::on_mouseup(Layer&, GUI::MouseEvent&, GUI::MouseEvent&)
} }
} }
void SprayTool::on_tool_button_contextmenu(GUI::ContextMenuEvent& event)
{
if (!m_context_menu) {
m_context_menu = GUI::Menu::construct();
m_thickness_actions.set_exclusive(true);
auto insert_action = [&](int size, bool checked = false) {
auto action = GUI::Action::create_checkable(String::number(size), [this, size](auto&) {
m_thickness = size;
});
action->set_checked(checked);
m_thickness_actions.add_action(*action);
m_context_menu->add_action(move(action));
};
insert_action(1, true);
insert_action(2);
insert_action(3);
insert_action(4);
}
m_context_menu->popup(event.screen_position());
}
GUI::Widget* SprayTool::get_properties_widget() GUI::Widget* SprayTool::get_properties_widget()
{ {
if (!m_properties_widget) { if (!m_properties_widget) {

3
Userland/Applications/PixelPaint/SprayTool.h
View file

@ -21,7 +21,6 @@ public:
virtual void on_mousedown(Layer&, GUI::MouseEvent& layer_event, GUI::MouseEvent& image_event) override; virtual void on_mousedown(Layer&, GUI::MouseEvent& layer_event, GUI::MouseEvent& image_event) override;
virtual void on_mouseup(Layer&, GUI::MouseEvent& layer_event, GUI::MouseEvent& image_event) override; virtual void on_mouseup(Layer&, GUI::MouseEvent& layer_event, GUI::MouseEvent& image_event) override;
virtual void on_mousemove(Layer&, GUI::MouseEvent& layer_event, GUI::MouseEvent& image_event) override; virtual void on_mousemove(Layer&, GUI::MouseEvent& layer_event, GUI::MouseEvent& image_event) override;
virtual void on_tool_button_contextmenu(GUI::ContextMenuEvent&) override;
virtual GUI::Widget* get_properties_widget() override; virtual GUI::Widget* get_properties_widget() override;
private: private:
@ -31,8 +30,6 @@ private:
RefPtr<Core::Timer> m_timer; RefPtr<Core::Timer> m_timer;
Gfx::IntPoint m_last_pos; Gfx::IntPoint m_last_pos;
Color m_color; Color m_color;
RefPtr<GUI::Menu> m_context_menu;
GUI::ActionGroup m_thickness_actions;
int m_thickness { 10 }; int m_thickness { 10 };
int m_density { 40 }; int m_density { 40 };
}; };