From aaf60053f19c68091221a1f34e482dda05a09f1c Mon Sep 17 00:00:00 2001
From: thankyouverycool <66646555+thankyouverycool@users.noreply.github.com>
Date: Wed, 10 May 2023 17:01:22 -0400
Subject: [PATCH] LibGUI: Use ColorRole::DisabledText when AbstractView is
 disabled

And only paint GlyphMapWidget's frame if disabled
---
 Userland/Libraries/LibGUI/AbstractView.cpp   | 4 +++-
 Userland/Libraries/LibGUI/GlyphMapWidget.cpp | 3 +++
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/Userland/Libraries/LibGUI/AbstractView.cpp b/Userland/Libraries/LibGUI/AbstractView.cpp
index cc78fb985b..4bc0fb2140 100644
--- a/Userland/Libraries/LibGUI/AbstractView.cpp
+++ b/Userland/Libraries/LibGUI/AbstractView.cpp
@@ -715,7 +715,9 @@ void AbstractView::draw_item_text(Gfx::Painter& painter, ModelIndex const& index
         return;
 
     Color text_color;
-    if (is_selected)
+    if (!is_enabled())
+        text_color = palette().color(Gfx::ColorRole::DisabledText);
+    else if (is_selected)
         text_color = is_focused() ? palette().selection_text() : palette().inactive_selection_text();
     else
         text_color = index.data(ModelRole::ForegroundColor).to_color(palette().color(foreground_role()));
diff --git a/Userland/Libraries/LibGUI/GlyphMapWidget.cpp b/Userland/Libraries/LibGUI/GlyphMapWidget.cpp
index 3908934d8c..a4d1acd49c 100644
--- a/Userland/Libraries/LibGUI/GlyphMapWidget.cpp
+++ b/Userland/Libraries/LibGUI/GlyphMapWidget.cpp
@@ -120,6 +120,9 @@ void GlyphMapWidget::paint_event(PaintEvent& event)
 {
     Frame::paint_event(event);
 
+    if (!is_enabled())
+        return;
+
     Painter painter(*this);
     painter.add_clip_rect(widget_inner_rect());
     painter.add_clip_rect(event.rect());