LibGUI: Make Ctrl-Shift-Home/-End work again

Previously, the initial call to update_selection() was missing, so if no
text was already selected, then Ctrl-Shift-End would only move the
cursor to the document end, but not select any text.

1 files changed, 10 insertions, 18 deletions

diff --git a/Userland/Libraries/LibGUI/EditingEngine.cpp b/Userland/Libraries/LibGUI/EditingEngine.cpp
index 051d6e5b6c..b43721dca2 100644
--- a/Userland/Libraries/LibGUI/EditingEngine.cpp
+++ b/Userland/Libraries/LibGUI/EditingEngine.cpp
@@ -120,37 +120,29 @@ bool EditingEngine::on_key(const KeyEvent& event)
     }
 
     if (event.key() == KeyCode::Key_Home) {
+        m_editor->update_selection(event.shift());
         if (event.ctrl()) {
             move_to_first_line();
-            if (event.shift() && m_editor->selection().start().is_valid()) {
-                m_editor->selection().set_end(m_editor->cursor());
-                m_editor->did_update_selection();
-            }
         } else {
-            m_editor->update_selection(event.shift());
             move_to_line_beginning();
-            if (event.shift() && m_editor->selection().start().is_valid()) {
-                m_editor->selection().set_end(m_editor->cursor());
-                m_editor->did_update_selection();
-            }
+        }
+        if (event.shift() && m_editor->selection().start().is_valid()) {
+            m_editor->selection().set_end(m_editor->cursor());
+            m_editor->did_update_selection();
         }
         return true;
     }
 
     if (event.key() == KeyCode::Key_End) {
+        m_editor->update_selection(event.shift());
         if (event.ctrl()) {
             move_to_last_line();
-            if (event.shift() && m_editor->selection().start().is_valid()) {
-                m_editor->selection().set_end(m_editor->cursor());
-                m_editor->did_update_selection();
-            }
         } else {
-            m_editor->update_selection(event.shift());
             move_to_line_end();
-            if (event.shift() && m_editor->selection().start().is_valid()) {
-                m_editor->selection().set_end(m_editor->cursor());
-                m_editor->did_update_selection();
-            }
+        }
+        if (event.shift() && m_editor->selection().start().is_valid()) {
+            m_editor->selection().set_end(m_editor->cursor());
+            m_editor->did_update_selection();
         }
         return true;
     }