RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-27 11:07:45 +00:00
Code Issues Activity Actions

PixelPaint: Correctly set default layer name

Browse source 
Previously, if you confirmed the "new layer" dialog without any change
to the layer name, the layer would end up with an empty string for its
name.
This commit is contained in:
Jelle Raaijmakers 2023-03-26 00:51:21 +01:00 • committed by Sam Atkins
parent f17bee0cb5
commit 256030da4e
2 changed files with 4 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

2
Userland/Applications/PixelPaint/CreateNewLayerDialog.cpp
View file

@ -28,7 +28,7 @@ CreateNewLayerDialog::CreateNewLayerDialog(Gfx::IntSize suggested_size, GUI::Win
name_label.set_text_alignment(Gfx::TextAlignment::CenterLeft); name_label.set_text_alignment(Gfx::TextAlignment::CenterLeft);
m_name_textbox = main_widget->add<GUI::TextBox>(); m_name_textbox = main_widget->add<GUI::TextBox>();
m_name_textbox->set_text("Layer"sv); m_name_textbox->set_text(default_layer_name);
m_name_textbox->select_all(); m_name_textbox->select_all();
m_name_textbox->on_change = [this] { m_name_textbox->on_change = [this] {
m_layer_name = m_name_textbox->text(); m_layer_name = m_name_textbox->text();

4
Userland/Applications/PixelPaint/CreateNewLayerDialog.h
View file

@ -18,10 +18,12 @@ public:
DeprecatedString const& layer_name() const { return m_layer_name; } DeprecatedString const& layer_name() const { return m_layer_name; }
private: private:
static constexpr StringView default_layer_name = "Layer"sv;
CreateNewLayerDialog(Gfx::IntSize suggested_size, GUI::Window* parent_window); CreateNewLayerDialog(Gfx::IntSize suggested_size, GUI::Window* parent_window);
Gfx::IntSize m_layer_size; Gfx::IntSize m_layer_size;
DeprecatedString m_layer_name; DeprecatedString m_layer_name { default_layer_name };
RefPtr<GUI::TextBox> m_name_textbox; RefPtr<GUI::TextBox> m_name_textbox;
}; };