From db5178fb8fe9b5da67f77e316c743f2f606f1b09 Mon Sep 17 00:00:00 2001
From: Andreas Kling <awesomekling@gmail.com>
Date: Sun, 27 Oct 2019 11:10:32 +0100
Subject: [PATCH] GTextEditor: Double-clicking on a span should select the span

This makes double-clicking on a C++ token in HackStudio select the
whole token, which is pretty nice. It's not perfect in all cases,
but a lot nicer than just expanding until we hit whitespace. :^)
---
 Libraries/LibGUI/GTextEditor.cpp | 28 ++++++++++++++++++++--------
 1 file changed, 20 insertions(+), 8 deletions(-)

diff --git a/Libraries/LibGUI/GTextEditor.cpp b/Libraries/LibGUI/GTextEditor.cpp
index 3af90498d5..40efa48388 100644
--- a/Libraries/LibGUI/GTextEditor.cpp
+++ b/Libraries/LibGUI/GTextEditor.cpp
@@ -164,16 +164,28 @@ void GTextEditor::doubleclick_event(GMouseEvent& event)
     auto start = text_position_at(event.position());
     auto end = start;
     auto& line = m_lines[start.line()];
-    while (start.column() > 0) {
-        if (isspace(line.characters()[start.column() - 1]))
-            break;
-        start.set_column(start.column() - 1);
-    }
 
-    while (end.column() < line.length()) {
-        if (isspace(line.characters()[end.column()]))
+    if (m_spans.is_empty()) {
+        while (start.column() > 0) {
+            if (isspace(line.characters()[start.column() - 1]))
+                break;
+            start.set_column(start.column() - 1);
+        }
+
+        while (end.column() < line.length()) {
+            if (isspace(line.characters()[end.column()]))
+                break;
+            end.set_column(end.column() + 1);
+        }
+    } else {
+        for (auto& span : m_spans) {
+            if (!span.range.contains(start))
+                continue;
+            start = span.range.start();
+            end = span.range.end();
+            end.set_column(end.column() + 1);
             break;
-        end.set_column(end.column() + 1);
+        }
     }
 
     m_selection.set(start, end);