From 3027cf7e99ee417233ec89ba0f5a7e040706b6fd Mon Sep 17 00:00:00 2001 From: thankyouverycool <66646555+thankyouverycool@users.noreply.github.com> Date: Thu, 17 Nov 2022 09:06:41 -0500 Subject: LibGUI+WindowServer: Remove InputPreemptor concept This functionality will be superceded by WindowType:Popups --- Userland/Libraries/LibGUI/ComboBox.cpp | 1 - Userland/Libraries/LibGUI/CommandPalette.cpp | 4 ---- Userland/Libraries/LibGUI/ConnectionToWindowServer.cpp | 6 ------ Userland/Libraries/LibGUI/ConnectionToWindowServer.h | 1 - Userland/Libraries/LibGUI/EmojiInputDialog.cpp | 5 ----- Userland/Libraries/LibGUI/Window.cpp | 6 ------ Userland/Libraries/LibGUI/Window.h | 2 -- Userland/Libraries/LibGUI/WindowMode.h | 1 - 8 files changed, 26 deletions(-) (limited to 'Userland/Libraries') diff --git a/Userland/Libraries/LibGUI/ComboBox.cpp b/Userland/Libraries/LibGUI/ComboBox.cpp index 58615c6948..f7dd1f8711 100644 --- a/Userland/Libraries/LibGUI/ComboBox.cpp +++ b/Userland/Libraries/LibGUI/ComboBox.cpp @@ -124,7 +124,6 @@ ComboBox::ComboBox() } m_open_button->set_enabled(true); }; - m_list_window->on_input_preemption = [this](auto) { close(); }; m_list_view = m_list_window->set_main_widget(); m_list_view->set_should_hide_unnecessary_scrollbars(true); diff --git a/Userland/Libraries/LibGUI/CommandPalette.cpp b/Userland/Libraries/LibGUI/CommandPalette.cpp index 56799988e9..1b05e6e276 100644 --- a/Userland/Libraries/LibGUI/CommandPalette.cpp +++ b/Userland/Libraries/LibGUI/CommandPalette.cpp @@ -229,10 +229,6 @@ CommandPalette::CommandPalette(GUI::Window& parent_window, ScreenPosition screen close(); }; - on_input_preemption = [this](InputPreemptor preemptor) { - if (preemptor != InputPreemptor::ContextMenu) - close(); - }; } void CommandPalette::collect_actions(GUI::Window& parent_window) diff --git a/Userland/Libraries/LibGUI/ConnectionToWindowServer.cpp b/Userland/Libraries/LibGUI/ConnectionToWindowServer.cpp index 60b03b1192..e658dab496 100644 --- a/Userland/Libraries/LibGUI/ConnectionToWindowServer.cpp +++ b/Userland/Libraries/LibGUI/ConnectionToWindowServer.cpp @@ -374,12 +374,6 @@ void ConnectionToWindowServer::window_state_changed(i32 window_id, bool minimize window->notify_state_changed({}, minimized, maximized, occluded); } -void ConnectionToWindowServer::window_input_preempted(i32 window_id, i32 preemptor) -{ - if (auto* window = Window::from_window_id(window_id)) - window->notify_input_preempted({}, static_cast(preemptor)); -} - void ConnectionToWindowServer::display_link_notification() { if (m_display_link_notification_pending) diff --git a/Userland/Libraries/LibGUI/ConnectionToWindowServer.h b/Userland/Libraries/LibGUI/ConnectionToWindowServer.h index 19965ae59f..c3c176e502 100644 --- a/Userland/Libraries/LibGUI/ConnectionToWindowServer.h +++ b/Userland/Libraries/LibGUI/ConnectionToWindowServer.h @@ -55,7 +55,6 @@ private: virtual void update_system_fonts(String const&, String const&, String const&) override; virtual void update_system_effects(Vector const&) override; virtual void window_state_changed(i32, bool, bool, bool) override; - virtual void window_input_preempted(i32, i32) override; virtual void display_link_notification() override; virtual void track_mouse_move(Gfx::IntPoint const&) override; virtual void ping() override; diff --git a/Userland/Libraries/LibGUI/EmojiInputDialog.cpp b/Userland/Libraries/LibGUI/EmojiInputDialog.cpp index 551af22f71..51f36164fc 100644 --- a/Userland/Libraries/LibGUI/EmojiInputDialog.cpp +++ b/Userland/Libraries/LibGUI/EmojiInputDialog.cpp @@ -144,11 +144,6 @@ EmojiInputDialog::EmojiInputDialog(Window* parent_window) close(); }; - on_input_preemption = [this](InputPreemptor preemptor) { - if (preemptor != InputPreemptor::ContextMenu) - close(); - }; - m_search_box->on_change = [this]() { update_displayed_emoji(); }; diff --git a/Userland/Libraries/LibGUI/Window.cpp b/Userland/Libraries/LibGUI/Window.cpp index 8dd89c7e13..9f1945bb4d 100644 --- a/Userland/Libraries/LibGUI/Window.cpp +++ b/Userland/Libraries/LibGUI/Window.cpp @@ -1163,12 +1163,6 @@ void Window::notify_state_changed(Badge, bool minimize } } -void Window::notify_input_preempted(Badge, InputPreemptor preemptor) -{ - if (on_input_preemption) - on_input_preemption(preemptor); -} - Action* Window::action_for_shortcut(Shortcut const& shortcut) { return Action::find_action_for_shortcut(*this, shortcut); diff --git a/Userland/Libraries/LibGUI/Window.h b/Userland/Libraries/LibGUI/Window.h index bacb09346c..3597dc539e 100644 --- a/Userland/Libraries/LibGUI/Window.h +++ b/Userland/Libraries/LibGUI/Window.h @@ -99,7 +99,6 @@ public: Function on_active_input_change; Function on_input_preemption_change; Function on_active_window_change; - Function on_input_preemption; int x() const { return rect().x(); } int y() const { return rect().y(); } @@ -206,7 +205,6 @@ public: static void for_each_window(Badge, Function); static void update_all_windows(Badge); void notify_state_changed(Badge, bool minimized, bool maximized, bool occluded); - void notify_input_preempted(Badge, InputPreemptor); virtual bool is_visible_for_timer_purposes() const override { return m_visible_for_timer_purposes; } diff --git a/Userland/Libraries/LibGUI/WindowMode.h b/Userland/Libraries/LibGUI/WindowMode.h index 9aeff8c803..93bbad0701 100644 --- a/Userland/Libraries/LibGUI/WindowMode.h +++ b/Userland/Libraries/LibGUI/WindowMode.h @@ -11,6 +11,5 @@ namespace GUI { using WindowMode = WindowServer::WindowMode; -using InputPreemptor = WindowServer::InputPreemptor; } -- cgit v1.2.3