0ct0pu5/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-12-04 13:30:31 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 7

WindowServer+LibGfx: Move title bar button layout to WindowTheme

Browse source
This commit is contained in:
thankyouverycool 2020-08-25 18:13:32 -04:00 committed by Andreas Kling
parent 1ab8939077
commit 208cb995ba
Notes: sideshowbarker 2024-07-19 03:08:13 +09:00 
Author: https://github.com/thankyouverycool
Commit: https://github.com/SerenityOS/serenity/commit/208cb995bab
Pull-request: https://github.com/SerenityOS/serenity/pull/3303
Reviewed-by: https://github.com/awesomekling
4 changed files with 36 additions and 24 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

28
Libraries/LibGfx/ClassicWindowTheme.cpp
View file

@ -177,4 +177,32 @@ IntRect ClassicWindowTheme::frame_rect_for_window(WindowType window_type, const
}
}
Vector<IntRect> ClassicWindowTheme::layout_buttons(WindowType window_type, const IntRect& window_rect, const Palette& palette, size_t buttons) const
{
int window_button_width = palette.window_title_button_width();
int window_button_height = palette.window_title_button_height();
int pos;
Vector<IntRect> button_rects;
if (window_type == WindowType::Notification)
pos = title_bar_rect(window_type, window_rect, palette).top() + 2;
else
pos = title_bar_text_rect(window_type, window_rect, palette).right() + 1;
for (size_t i = 0; i < buttons; i++) {
if (window_type == WindowType::Notification) {
// The button height & width have to be equal or it leaks out of its area
Gfx::IntRect rect { 0, pos, window_button_height, window_button_height };
rect.center_horizontally_within(title_bar_rect(window_type, window_rect, palette));
button_rects.append(rect);
pos += window_button_height;
} else {
pos -= window_button_width;
Gfx::IntRect rect { pos, 0, window_button_width, window_button_height };
rect.center_vertically_within(title_bar_text_rect(window_type, window_rect, palette));
button_rects.append(rect);
}
}
return button_rects;
}
}

4
Libraries/LibGfx/ClassicWindowTheme.h
View file

@ -26,8 +26,8 @@
#pragma once
#include <LibGfx/WindowTheme.h>
#include <LibGfx/Color.h>
#include <LibGfx/WindowTheme.h>
namespace Gfx {
@ -45,6 +45,8 @@ public:
virtual IntRect frame_rect_for_window(WindowType, const IntRect& window_rect, const Palette&) const override;
virtual Vector<IntRect> layout_buttons(WindowType, const IntRect& window_rect, const Palette&, size_t buttons) const override;
private:
struct FrameColors {
Color title_color;

2
Libraries/LibGfx/WindowTheme.h
View file

@ -59,6 +59,8 @@ public:
virtual IntRect frame_rect_for_window(WindowType, const IntRect& window_rect, const Palette&) const = 0;
virtual Vector<IntRect> layout_buttons(WindowType, const IntRect& window_rect, const Palette&, size_t buttons) const = 0;
protected:
WindowTheme() { }
};

26
Services/WindowServer/WindowFrame.cpp
View file

@ -261,29 +261,9 @@ void WindowFrame::notify_window_rect_changed(const Gfx::IntRect& old_rect, const
void WindowFrame::layout_buttons()
{
auto palette = WindowManager::the().palette();
int window_button_width = palette.window_title_button_width();
int window_button_height = palette.window_title_button_height();
int pos;
if (m_window.type() == WindowType::Notification)
pos = title_bar_rect().top() + 2;
else
pos = title_bar_text_rect().right() + 1;
for (auto& button : m_buttons) {
if (m_window.type() == WindowType::Notification) {
// The button height & width have to be equal or it leaks out of its area
Gfx::IntRect rect { 0, pos, window_button_height, window_button_height };
rect.center_horizontally_within(title_bar_rect());
button.set_relative_rect(rect);
pos += window_button_height;
} else {
pos -= window_button_width;
Gfx::IntRect rect { pos, 0, window_button_width, window_button_height };
rect.center_vertically_within(title_bar_text_rect());
button.set_relative_rect(rect);
}
}
auto button_rects = Gfx::WindowTheme::current().layout_buttons(to_theme_window_type(m_window.type()), m_window.rect(), WindowManager::the().palette(), m_buttons.size());
for (size_t i = 0; i < m_buttons.size(); i++)
m_buttons[i].set_relative_rect(button_rects[i]);
}
void WindowFrame::on_mouse_event(const MouseEvent& event)