From 5de7775d22b5b0cde49a99632e8504fbb083efaf Mon Sep 17 00:00:00 2001
From: Sam Atkins <atkinssj@gmail.com>
Date: Fri, 30 Jul 2021 12:16:58 +0100
Subject: [PATCH] Browser: Show a message when attempting to search with no
 search engine

Previously, it just showed an error message saying that "" failed to
load, which made me think that search engines were broken.
---
 Userland/Applications/Browser/Tab.cpp | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/Userland/Applications/Browser/Tab.cpp b/Userland/Applications/Browser/Tab.cpp
index 22805a177a..aa7fe0bc53 100644
--- a/Userland/Applications/Browser/Tab.cpp
+++ b/Userland/Applications/Browser/Tab.cpp
@@ -21,6 +21,7 @@
 #include <LibGUI/Button.h>
 #include <LibGUI/Clipboard.h>
 #include <LibGUI/Menu.h>
+#include <LibGUI/MessageBox.h>
 #include <LibGUI/Statusbar.h>
 #include <LibGUI/TextBox.h>
 #include <LibGUI/Toolbar.h>
@@ -145,6 +146,11 @@ Tab::Tab(BrowserWindow& window, Type type)
     m_location_box->set_placeholder("Address");
 
     m_location_box->on_return_pressed = [this] {
+        if (m_location_box->text().starts_with('?') && g_search_engine.is_empty()) {
+            GUI::MessageBox::show(&this->window(), "Select a search engine in the Settings menu before searching.", "No search engine selected", GUI::MessageBox::Type::Information);
+            return;
+        }
+
         auto url = url_from_user_input(m_location_box->text());
         load(url);
         view().set_focus(true);