diff --git a/src/BoxLayout.cpp b/src/BoxLayout.cpp index 3b3b31e..aaebf38 100644 --- a/src/BoxLayout.cpp +++ b/src/BoxLayout.cpp @@ -1,6 +1,7 @@ #include "BoxLayout.hpp" #include "Box.hpp" #include "Widget.hpp" +#include #include @@ -12,19 +13,39 @@ void BoxLayout::run() { } Box free_space = m_target->rect().max_geometry(); - Box total_space = m_target->rect().max_geometry(); Point current_position { 0.0, 0.0 }; - int unslotted_widgets = 0; + + // Loop through all children: + // - Children which do not have a corresponding slot are counted into the unslotted_widgets variable, which will be used to designate the amount of space per unslotted widget. + // - Slots with a matching child will have their percentages computed. The amount of remaining free space after slotted widgets are added is also calculated. for (unsigned int i = 0; i < m_target->children().size(); i++) { if (i >= m_slots.size()) { unslotted_widgets++; + } else { + Slot *slot = &m_slots[i]; + + if (m_direction == Direction::Horizontal) { + if (slot->type == SlotType::Percent) { + slot->pixel = (free_space.width() / 100.0) * slot->percent; + } + free_space.set_width(free_space.width() - slot->pixel); + } else { + if (slot->type == SlotType::Percent) { + slot->pixel = (free_space.height() / 100.0) * slot->percent; + } + free_space.set_height(free_space.height() - slot->pixel); + } } } - double space_per_unslotted_widget = m_direction == Direction::Horizontal ? total_space.width() / unslotted_widgets : total_space.height() / unslotted_widgets; + + double space_per_unslotted_widget = m_direction == Direction::Horizontal ? free_space.width() / unslotted_widgets : free_space.height() / unslotted_widgets; for (unsigned int i = 0; i < m_target->children().size(); i++) { auto child = m_target->children()[i]; + + // Determine the slot to use for this widget. + // Widgets which don't have a slot will get a default one based on space_per_unslotted_widget. Slot slot; if (i >= m_slots.size()) { slot = Slot { @@ -36,36 +57,20 @@ void BoxLayout::run() { slot = m_slots[i]; } - // resolve slot size - if (slot.type == SlotType::Percent) { - if (m_direction == Direction::Horizontal) { - slot.pixel = (free_space.width() / 100.0) * slot.percent; - } else { - slot.pixel = (free_space.height() / 100.0) * slot.percent; - } - } + // make sure pixel values are aligned to the pixel grid + slot.pixel = std::floor(slot.pixel); - // justify child geometry, stop looping if free space has been exhausted + // position all children according to the slots child->rect().set_x(current_position.x()); child->rect().set_y(current_position.y()); if (m_direction == Direction::Horizontal) { child->rect().set_width(slot.pixel); child->rect().set_height(free_space.height()); current_position.add(slot.pixel, 0); - free_space.set_width(free_space.width() - slot.pixel); - - if (free_space.width() <= 0) { - break; - } } else { child->rect().set_width(free_space.width()); child->rect().set_height(slot.pixel); current_position.add(0, slot.pixel); - free_space.set_height(free_space.height() - slot.pixel); - - if (free_space.height() <= 0) { - break; - } } } }; diff --git a/src/DocumentLayout.cpp b/src/DocumentLayout.cpp index ec4cd0c..8761b4c 100644 --- a/src/DocumentLayout.cpp +++ b/src/DocumentLayout.cpp @@ -9,7 +9,7 @@ void DocumentLayout::run() { if (!m_target) return; - Point bound { m_margin * 2, m_margin * 2 }; + Point bound { m_margin, m_margin }; Point current_position { m_margin, m_margin }; double largest_height_so_far = -1.0; diff --git a/src/Widget.cpp b/src/Widget.cpp index 2ca6073..279369f 100644 --- a/src/Widget.cpp +++ b/src/Widget.cpp @@ -224,7 +224,7 @@ void Widget::handle_mouse_button_event(MouseButtonEvent &event) { bool update_activation_to = event.was_left_button_pressed(); if (!m_rect.contains_point(event.point())) { - return; + update_activation_to = false; } on_mouse_button(event); diff --git a/src/main.cpp b/src/main.cpp index c5aeade..fed5f52 100644 --- a/src/main.cpp +++ b/src/main.cpp @@ -23,18 +23,29 @@ int main() { window.spawn_window(); auto main_widget = window.set_main_widget(); - main_widget->set_layout(Raven::Direction::Horizontal); + auto main_layout = main_widget->set_layout(Raven::Direction::Vertical); + main_layout->slot_pixel(32); // top bar - auto inner = main_widget->add(); - inner->resize(400, 200); - inner->set_style(&Raven::accent_widget_style); - auto inner_layout = inner->set_layout(Raven::Direction::Horizontal); - inner_layout->slot_pixel(20); - inner_layout->slot_percent(50); + auto top_bar = main_widget->add(); + top_bar->set_style(&Raven::accent_widget_style); + top_bar->set_layout(Raven::Direction::Horizontal); + + auto scroll_container = main_widget->add(); + auto container_widget = scroll_container->make_target(); + container_widget->set_layout(Raven::Direction::Vertical); + container_widget->resize(400, 400); - for (int i = 0; i < 4; i++) { - auto button = inner->add("h"); - } + auto new_button = top_bar->add("add"); + new_button->on_click = [container_widget]() { + auto button = container_widget->add("button"); + button->on_click = [button]() { + button->set_style(&Raven::accent_button_style); + }; + }; + auto remove_button = top_bar->add("remove"); + remove_button->on_click = [container_widget]() { + container_widget->remove_child(container_widget->children()[0]); + }; window.run(true); return 0;