refactor(ui): move scale property to subclasses and fix resize event handling

Remove scale from base Widget class and add per-type set_scale/scale methods to Button, Image, and Label. Dispatch separate WindowResizeEvent alongside existing FrameBufferResizeEvent. Correct centering calculations from `+` to `-` in main menu and world UI managers.

src/app.cpp

@@ -677,12 +677,21 @@ void App::run() {
 // static Gait player_gait = Gait::WALK;
 void App::update() {
     glfwPollEvents();
+    {
+        int w, h;
+        glfwGetFramebufferSize(m_window.get_glfw_window(), &w, &h);
 
-    int w, h;
-    glfwGetFramebufferSize(m_window.get_glfw_window(), &w, &h);
+        if (w != m_renderer.frame_width() || h != m_renderer.frame_height()) {
+            dispatch_event(FrameBufferResizeEvent{w, h});
+        }
+    }
+    {
+        int w, h;
+        glfwGetWindowSize(m_window.get_glfw_window(), &w, &h);
 
-    if (w != m_renderer.frame_width() || h != m_renderer.frame_height()) {
-        dispatch_event(FrameBufferResizeEvent{w, h});
+        if (w != m_renderer.window_width() || h != m_renderer.window_height()) {
+            dispatch_event(WindowResizeEvent{w, h});
+        }
     }
 
     current_time = glfwGetTime();

src/debug_collector.cpp

@@ -23,8 +23,8 @@ void DebugCollector::init_text() {
 #endif
     version_text.set_color(Color::WHITE)
         .set_text(version)
-        .set_position(0.0f, 100.0f)
-        .set_scale(0.8f);
+        .set_scale(0.8f)
+        .set_position(0.0f, 100.0f);
     m_component.try_emplace(version_text.id(), &version_text);
 
     // fps
@@ -35,8 +35,8 @@ void DebugCollector::init_text() {
     // player_pos
     auto& player_pos_text = m_widget.add_child<Label>("player_pos");
     player_pos_text.set_text("x: 0.00 y: 0.00 z: 0.00")
-        .set_position(0.0f, 150.0f)
-        .set_scale(0.8f);
+        .set_scale(0.8f)
+        .set_position(0.0f, 150.0f);
     m_component.try_emplace(player_pos_text.id(), &player_pos_text);
 
     // rendered_chunk

src/ui/button.cpp

@@ -59,14 +59,16 @@ Widget& Button::set_position(const glm::vec2& pos) {
 Widget& Button::set_position(float x, float y) {
     return set_position(glm::vec2{x, y});
 }
-Widget& Button::set_scale(float scale) {
-    m_scale = scale;
+Button& Button::set_scale(float scale) {
     m_background->set_scale(scale);
     m_max_pos.x = m_min_pos.x + width();
     m_max_pos.y = m_min_pos.y + height();
     return *this;
 }
-float Button::width() const { return m_background->width() * m_scale; }
-float Button::height() const { return m_background->height() * m_scale; }
+
+float Button::scale() const { return m_background->scale(); }
+
+float Button::width() const { return m_background->width(); }
+float Button::height() const { return m_background->height(); }
 
 } // namespace Cubed

src/ui/image.cpp

@@ -13,7 +13,11 @@ Image& Image::set_image(const std::string& path,
     m_texture = texture_manager.get_image_texture(path);
     return *this;
 }
-
+Image& Image::set_scale(float scale) {
+    m_scale = scale;
+    return *this;
+}
+float Image::scale() const { return m_scale; }
 float Image::height() const {
     if (!m_texture) {
         Logger::error("Image id {} not set image!", m_id);

src/ui/label.cpp

@@ -15,7 +15,10 @@ Label& Label::set_color(Color color) {
     m_text.color = color;
     return *this;
 }
-
+Label& Label::set_scale(float scale) {
+    m_scale = scale;
+    return *this;
+}
 void Label::update(float dt) { on_update(dt); }
 
 void Label::on_update(float dt) { (void)dt; }
@@ -43,5 +46,5 @@ float Label::width() const { return m_real_width * m_scale; }
 float Label::height() const { return m_real_height * m_scale; }
 float Label::offset_x() const { return m_offset_x; }
 float Label::offset_y() const { return m_offset_y; }
-
+float Label::scale() const { return m_scale; }
 } // namespace Cubed

src/ui/main_menu_ui_manager.cpp

@@ -22,19 +22,22 @@ void MainMenuUIManager::init() {
 
     start_game_button->set_clicked(
         [this]() { m_scene.scene_manager().request_push(SceneType::WORLD); });
-    start_game_button->set_position(
-        m_scene.scene_manager().app().renderer().window_width() / 2.0f +
-            back.width() / 2.0f,
-        m_scene.scene_manager().app().renderer().window_width() / 2.0f +
-            back.height() / 2.0f);
     start_game_button->set_scale(3.0f);
+    start_game_button->set_position(
+        m_scene.scene_manager().app().renderer().window_width() / 2.0f -
+            back.width() / 2.0f,
+        m_scene.scene_manager().app().renderer().window_height() / 2.0f -
+            back.height() / 2.0f);
+
     m_widgets.try_emplace("start game", std::move(start_game_button));
 }
+
 void MainMenuUIManager::update(float dt) {
     for (auto& w : m_widgets) {
         w.second->update(dt);
     }
 }
+
 void MainMenuUIManager::render(Renderer& renderer) {
     renderer.begin_render_ui();
 
@@ -101,8 +104,8 @@ bool MainMenuUIManager::handle_window_resize_event(const WindowResizeEvent& e) {
     if (it != m_widgets.end()) {
         auto* start_game = dynamic_cast<Button*>(it->second.get());
         if (start_game) {
-            start_game->set_position(e.width / 2 + start_game->width() / 2,
-                                     e.height / 2 + start_game->height() / 2);
+            start_game->set_position(e.width / 2 - start_game->width() / 2,
+                                     e.height / 2 - start_game->height() / 2);
         }
     }
     for (auto& w : m_widgets) {

src/ui/widget.cpp

@@ -31,16 +31,10 @@ Widget& Widget::set_position(float x, float y) {
     return *this;
 }
 
-Widget& Widget::set_scale(float scale) {
-    m_scale = scale;
-    return *this;
-}
-
 float Widget::width() const { return 0.0f; }
 float Widget::height() const { return 0.0f; }
 
 const glm::vec2& Widget::pos() const { return m_pos; }
-float Widget::scale() const { return m_scale; }
 const std::string& Widget::id() const { return m_id; }
 
 bool Widget::handle_key_event(const KeyEvent& e) {

src/ui/world_ui_manager.cpp

@@ -16,10 +16,11 @@ void WorldUIManager::init() {
     crosshair->set_image("texture/ui/0.png",
                          m_scene.scene_manager().app().texture_manager());
     auto& renderer = m_scene.scene_manager().app().renderer();
-    crosshair
-        ->set_position(renderer.window_width() / 2 + crosshair->width() / 2,
-                       renderer.window_height() / 2 + crosshair->height() / 2)
-        .set_scale(3.0f);
+
+    crosshair->set_scale(3.0f).set_position(
+        renderer.window_width() / 2 - crosshair->width() / 2,
+        renderer.window_height() / 2 - crosshair->height() / 2);
+
     m_widgets.try_emplace("crosshair", std::move(crosshair));
 }
 void WorldUIManager::update(float dt) {
@@ -96,8 +97,9 @@ bool WorldUIManager::handle_window_resize_event(const WindowResizeEvent& e) {
     if (it != m_widgets.end()) {
         auto* crosshair = dynamic_cast<Image*>(it->second.get());
         if (crosshair) {
-            crosshair->set_position(e.width / 2 + crosshair->width() / 2,
-                                    e.height / 2 + crosshair->height() / 2);
+            crosshair->set_position(e.width / 2.0f - crosshair->width() / 2.0f,
+                                    e.height / 2.0f -
+                                        crosshair->height() / 2.0f);
         }
     }
     for (auto& w : m_widgets) {