IRCClient: Add a toolbar with some actions.

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");
 }

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;
 

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) {

Base/res/icons/16x16/.gitignore

@@ -0,0 +1 @@
+*.data

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,19 +46,33 @@ 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
+    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
 
@@ -72,6 +89,7 @@ void GBoxLayout::run(GWidget& widget)
     }
 
 #ifdef GBOXLAYOUT_DEBUG
+    if (should_log)
         printf("GBoxLayout:   automatic_size=%s\n", automatic_size.to_string().characters());
 #endif
 
@@ -106,6 +124,7 @@ void GBoxLayout::run(GWidget& widget)
         }
 
 #ifdef GBOXLAYOUT_DEBUG
+        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);

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);
 }

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()

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)

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(); }