diff options
-rw-r--r-- | Applications/IRCClient/IRCAppWindow.cpp | 29 | ||||
-rw-r--r-- | Applications/IRCClient/IRCClient.cpp | 7 | ||||
-rw-r--r-- | Applications/TextEditor/main.cpp | 2 | ||||
-rw-r--r-- | Base/res/icons/16x16/.gitignore | 1 | ||||
-rw-r--r-- | Base/res/icons/16x16/irc-join.png | bin | 0 -> 424 bytes | |||
-rw-r--r-- | Base/res/icons/16x16/irc-join.rgb | bin | 0 -> 1024 bytes | |||
-rw-r--r-- | Base/res/icons/16x16/irc-part.png | bin | 0 -> 468 bytes | |||
-rw-r--r-- | Base/res/icons/16x16/irc-part.rgb | bin | 0 -> 1024 bytes | |||
-rw-r--r-- | LibGUI/GBoxLayout.cpp | 33 | ||||
-rw-r--r-- | LibGUI/GStackWidget.cpp | 3 | ||||
-rw-r--r-- | LibGUI/GToolBar.cpp | 5 | ||||
-rw-r--r-- | LibGUI/GWidget.cpp | 12 | ||||
-rw-r--r-- | LibGUI/GWidget.h | 1 |
13 files changed, 76 insertions, 17 deletions
diff --git a/Applications/IRCClient/IRCAppWindow.cpp b/Applications/IRCClient/IRCAppWindow.cpp index c026bd074f..0595e7de45 100644 --- a/Applications/IRCClient/IRCAppWindow.cpp +++ b/Applications/IRCClient/IRCAppWindow.cpp @@ -4,6 +4,9 @@ #include <LibGUI/GStackWidget.h> #include <LibGUI/GTableView.h> #include <LibGUI/GBoxLayout.h> +#include <LibGUI/GToolBar.h> +#include <LibGUI/GAction.h> +#include <stdio.h> IRCAppWindow::IRCAppWindow() : GWindow() @@ -43,11 +46,28 @@ void IRCAppWindow::setup_client() void IRCAppWindow::setup_widgets() { auto* widget = new GWidget(nullptr); - widget->set_fill_with_background_color(true); set_main_widget(widget); - widget->set_layout(make<GBoxLayout>(Orientation::Horizontal)); + widget->set_layout(make<GBoxLayout>(Orientation::Vertical)); - auto* window_list = new GTableView(widget); + printf("main_widget: %s{%p}\n", widget->class_name(), widget); + + auto join_action = GAction::create("Join channel", GraphicsBitmap::load_from_file(GraphicsBitmap::Format::RGBA32, "/res/icons/16x16/irc-join.rgb", { 16, 16 }), [] (auto&) { + printf("FIXME: Implement join action\n"); + }); + + auto part_action = GAction::create("Part from channel", GraphicsBitmap::load_from_file(GraphicsBitmap::Format::RGBA32, "/res/icons/16x16/irc-part.rgb", { 16, 16 }), [] (auto&) { + printf("FIXME: Implement part action\n"); + }); + + auto* toolbar = new GToolBar(widget); + toolbar->add_action(join_action.copy_ref()); + toolbar->add_action(part_action.copy_ref()); + + auto* horizontal_container = new GWidget(widget); + printf("horizontal_widget: %s{%p}\n", horizontal_container->class_name(), horizontal_container); + horizontal_container->set_layout(make<GBoxLayout>(Orientation::Horizontal)); + + auto* window_list = new GTableView(horizontal_container); window_list->set_headers_visible(false); window_list->set_alternating_row_colors(false); window_list->set_model(OwnPtr<IRCClientWindowListModel>(m_client.client_window_list_model())); @@ -57,7 +77,8 @@ void IRCAppWindow::setup_widgets() m_container->set_active_widget(&window); }; - m_container = new GStackWidget(widget); + m_container = new GStackWidget(horizontal_container); + printf("m_container: %s{%p}\n", ((GWidget*)m_container)->class_name(), m_container); create_subwindow(IRCClientWindow::Server, "Server"); } diff --git a/Applications/IRCClient/IRCClient.cpp b/Applications/IRCClient/IRCClient.cpp index f27770cc6c..fd4ea8fe22 100644 --- a/Applications/IRCClient/IRCClient.cpp +++ b/Applications/IRCClient/IRCClient.cpp @@ -11,6 +11,8 @@ #include <unistd.h> #include <stdio.h> +//#define IRC_DEBUG + enum IRCNumeric { RPL_NAMREPLY = 353, RPL_ENDOFNAMES = 366, @@ -201,6 +203,7 @@ void IRCClient::join_channel(const String& channel_name) void IRCClient::handle(const Message& msg, const String&) { +#ifdef IRC_DEBUG printf("IRCClient::execute: prefix='%s', command='%s', arguments=%d\n", msg.prefix.characters(), msg.command.characters(), @@ -212,6 +215,7 @@ void IRCClient::handle(const Message& msg, const String&) printf(" [%d]: %s\n", i, arg.characters()); ++i; } +#endif bool is_numeric; int numeric = msg.command.to_uint(is_numeric); @@ -285,7 +289,9 @@ void IRCClient::handle_privmsg(const Message& msg) auto sender_nick = parts[0]; auto target = msg.arguments[0]; +#ifdef IRC_DEBUG printf("handle_privmsg: sender_nick='%s', target='%s'\n", sender_nick.characters(), target.characters()); +#endif if (sender_nick.is_empty()) return; @@ -353,7 +359,6 @@ void IRCClient::handle_join(const Message& msg) void IRCClient::handle_namreply(const Message& msg) { - printf("NAMREPLY:\n"); if (msg.arguments.size() < 4) return; diff --git a/Applications/TextEditor/main.cpp b/Applications/TextEditor/main.cpp index 78d9f16993..c7bddbe69d 100644 --- a/Applications/TextEditor/main.cpp +++ b/Applications/TextEditor/main.cpp @@ -22,7 +22,7 @@ int main(int argc, char** argv) widget->set_layout(make<GBoxLayout>(Orientation::Vertical)); auto* toolbar = new GToolBar(widget); - auto* text_editor = new GTextEditor(widget); + auto* text_editor = new GTextEditor(GTextEditor::MultiLine, widget); auto* statusbar = new GStatusBar(widget); text_editor->on_cursor_change = [statusbar] (GTextEditor& editor) { diff --git a/Base/res/icons/16x16/.gitignore b/Base/res/icons/16x16/.gitignore new file mode 100644 index 0000000000..f9e8176423 --- /dev/null +++ b/Base/res/icons/16x16/.gitignore @@ -0,0 +1 @@ +*.data diff --git a/Base/res/icons/16x16/irc-join.png b/Base/res/icons/16x16/irc-join.png Binary files differnew file mode 100644 index 0000000000..3a86927805 --- /dev/null +++ b/Base/res/icons/16x16/irc-join.png diff --git a/Base/res/icons/16x16/irc-join.rgb b/Base/res/icons/16x16/irc-join.rgb Binary files differnew file mode 100644 index 0000000000..10773d09d3 --- /dev/null +++ b/Base/res/icons/16x16/irc-join.rgb diff --git a/Base/res/icons/16x16/irc-part.png b/Base/res/icons/16x16/irc-part.png Binary files differnew file mode 100644 index 0000000000..7a9358113a --- /dev/null +++ b/Base/res/icons/16x16/irc-part.png diff --git a/Base/res/icons/16x16/irc-part.rgb b/Base/res/icons/16x16/irc-part.rgb Binary files differnew file mode 100644 index 0000000000..6440380486 --- /dev/null +++ b/Base/res/icons/16x16/irc-part.rgb diff --git a/LibGUI/GBoxLayout.cpp b/LibGUI/GBoxLayout.cpp index 75d7a05c4c..9abea5ed62 100644 --- a/LibGUI/GBoxLayout.cpp +++ b/LibGUI/GBoxLayout.cpp @@ -1,12 +1,9 @@ #include <LibGUI/GBoxLayout.h> #include <LibGUI/GWidget.h> +#include <stdio.h> //#define GBOXLAYOUT_DEBUG -#ifdef GBOXLAYOUT_DEBUG -#include <stdio.h> -#endif - GBoxLayout::GBoxLayout(Orientation orientation) : m_orientation(orientation) { @@ -35,6 +32,12 @@ static Size compute_preferred_size(GLayout::Entry& entry) void GBoxLayout::run(GWidget& widget) { + bool should_log = false; +#ifdef GBOXLAYOUT_DEBUG + should_log = true; +#endif + if (should_log) + printf("GBoxLayout: running layout on %s{%p}\n", widget.class_name(), &widget); if (m_entries.is_empty()) return; @@ -43,20 +46,34 @@ void GBoxLayout::run(GWidget& widget) int number_of_visible_entries = 0; + if (should_log) + printf("GBoxLayout: Starting with available size: %s\n", available_size.to_string().characters()); + for (auto& entry : m_entries) { if (!entry.widget->is_visible()) continue; ++number_of_visible_entries; if (entry.widget && entry.widget->size_policy(orientation()) == SizePolicy::Fixed) { + if (should_log) { + printf("GBoxLayout: Subtracting for fixed %s{%p}, size: %s\n", entry.widget->class_name(), entry.widget.ptr(), entry.widget->preferred_size().to_string().characters()); + printf("GBoxLayout: Available size before: %s\n", available_size.to_string().characters()); + } + available_size -= entry.widget->preferred_size(); + if (should_log) + printf("GBoxLayout: Available size after: %s\n", available_size.to_string().characters()); ++number_of_entries_with_fixed_size; } } + if (should_log) + printf("GBoxLayout: Number of visible: %d/%d\n", number_of_visible_entries, m_entries.size()); + int number_of_entries_with_automatic_size = number_of_visible_entries - number_of_entries_with_fixed_size; #ifdef GBOXLAYOUT_DEBUG - printf("GBoxLayout: available_size=%s, fixed=%d, fill=%d\n", available_size.to_string().characters(), number_of_entries_with_fixed_size, number_of_entries_with_automatic_size); + if (should_log) + printf("GBoxLayout: available_size=%s, fixed=%d, fill=%d\n", available_size.to_string().characters(), number_of_entries_with_fixed_size, number_of_entries_with_automatic_size); #endif Size automatic_size; @@ -72,7 +89,8 @@ void GBoxLayout::run(GWidget& widget) } #ifdef GBOXLAYOUT_DEBUG - printf("GBoxLayout: automatic_size=%s\n", automatic_size.to_string().characters()); + if (should_log) + printf("GBoxLayout: automatic_size=%s\n", automatic_size.to_string().characters()); #endif int current_x = margins().left(); @@ -106,7 +124,8 @@ void GBoxLayout::run(GWidget& widget) } #ifdef GBOXLAYOUT_DEBUG - printf("GBoxLayout: apply, %s{%p} <- %s\n", entry.widget->class_name(), entry.widget.ptr(), rect.to_string().characters()); + if (should_log) + printf("GBoxLayout: apply, %s{%p} <- %s\n", entry.widget->class_name(), entry.widget.ptr(), rect.to_string().characters()); #endif entry.widget->set_relative_rect(rect); diff --git a/LibGUI/GStackWidget.cpp b/LibGUI/GStackWidget.cpp index fea58ab676..ba225a7547 100644 --- a/LibGUI/GStackWidget.cpp +++ b/LibGUI/GStackWidget.cpp @@ -34,7 +34,7 @@ void GStackWidget::resize_event(GResizeEvent& event) void GStackWidget::child_event(GChildEvent& event) { if (!event.child() || !event.child()->is_widget()) - return; + return GWidget::child_event(event); auto& child = static_cast<GWidget&>(*event.child()); if (event.type() == GEvent::ChildAdded) { if (!m_active_widget) @@ -53,4 +53,5 @@ void GStackWidget::child_event(GChildEvent& event) set_active_widget(new_active_widget); } } + GWidget::child_event(event); } diff --git a/LibGUI/GToolBar.cpp b/LibGUI/GToolBar.cpp index 3024e218f8..992b7f867e 100644 --- a/LibGUI/GToolBar.cpp +++ b/LibGUI/GToolBar.cpp @@ -36,6 +36,8 @@ void GToolBar::add_action(Retained<GAction>&& action) button->set_button_style(GButtonStyle::CoolBar); button->set_size_policy(SizePolicy::Fixed, SizePolicy::Fixed); + ASSERT(button->size_policy(Orientation::Horizontal) == SizePolicy::Fixed); + ASSERT(button->size_policy(Orientation::Vertical) == SizePolicy::Fixed); button->set_preferred_size({ 24, 24 }); m_items.append(move(item)); @@ -60,6 +62,9 @@ public: painter.draw_line({ 0, 0 }, { 0, rect().bottom() }, Color::DarkGray); painter.draw_line({ 1, 0 }, { 1, rect().bottom() }, Color::White); } + +private: + virtual const char* class_name() const override { return "SeparatorWidget"; } }; void GToolBar::add_separator() diff --git a/LibGUI/GWidget.cpp b/LibGUI/GWidget.cpp index 15ee45de32..8303aad9c8 100644 --- a/LibGUI/GWidget.cpp +++ b/LibGUI/GWidget.cpp @@ -15,15 +15,21 @@ GWidget::GWidget(GWidget* parent) set_font(nullptr); m_background_color = Color::LightGray; m_foreground_color = Color::Black; - - if (parent && parent->layout()) - parent->layout()->add_widget(*this); } GWidget::~GWidget() { } +void GWidget::child_event(GChildEvent& event) +{ + if (event.type() == GEvent::ChildAdded) { + if (event.child() && event.child()->is_widget() && layout()) + layout()->add_widget(static_cast<GWidget&>(*event.child())); + } + return GObject::child_event(event); +} + void GWidget::set_relative_rect(const Rect& rect) { if (rect == m_relative_rect) diff --git a/LibGUI/GWidget.h b/LibGUI/GWidget.h index b9a65598b2..d856e66420 100644 --- a/LibGUI/GWidget.h +++ b/LibGUI/GWidget.h @@ -47,6 +47,7 @@ public: virtual void focusout_event(GEvent&); virtual void enter_event(GEvent&); virtual void leave_event(GEvent&); + virtual void child_event(GChildEvent&) override; Rect relative_rect() const { return m_relative_rect; } Point relative_position() const { return m_relative_rect.location(); } |