Browse Source

LibGUI: Let's make F2 the standard "edit key"

This matches what other systems do, and allows Return to become
the unambiguous "activation key" instead. :^)
Andreas Kling 4 years ago
parent
commit
1847219cbf
1 changed files with 10 additions and 4 deletions
  1. 10 4
      Libraries/LibGUI/TableView.cpp

+ 10 - 4
Libraries/LibGUI/TableView.cpp

@@ -165,11 +165,17 @@ void TableView::keydown_event(KeyEvent& event)
 {
     if (!model())
         return;
-    if (event.key() == KeyCode::Key_Return) {
-        if (is_editable() && edit_triggers() & EditTrigger::EditKeyPressed)
+
+    if (event.key() == KeyCode::Key_F2) {
+        if (is_editable() && edit_triggers() & EditTrigger::EditKeyPressed) {
             begin_editing(cursor_index());
-        else
-            activate(cursor_index());
+            event.accept();
+            return;
+        }
+    }
+
+    if (event.key() == KeyCode::Key_Return) {
+        activate(cursor_index());
         return;
     }