From b67378112fd536662207152c8583c4eda39a1d36 Mon Sep 17 00:00:00 2001
From: GiM <gim913@gmail.com>
Date: Fri, 31 Jan 2025 22:47:33 +0100
Subject: [PATCH] split TextInputComponent::process_inputs into separate
 methods

---
 src/components/textinput.rs | 424 ++++++++++++------------------------
 1 file changed, 142 insertions(+), 282 deletions(-)

diff --git a/src/components/textinput.rs b/src/components/textinput.rs
index e6586b8355..1e388826bf 100644
--- a/src/components/textinput.rs
+++ b/src/components/textinput.rs
@@ -259,360 +259,220 @@ impl TextInputComponent {
 		}
 	}
 
-	#[allow(clippy::too_many_lines, clippy::unnested_or_patterns)]
-	fn process_inputs(ta: &mut TextArea<'_>, input: &Input) -> bool {
-		match input {
-			Input {
-				key: Key::Char(c),
-				ctrl: false,
-				alt: false,
-				..
-			} => {
-				ta.insert_char(*c);
-				true
-			}
-			Input {
-				key: Key::Tab,
-				ctrl: false,
-				alt: false,
-				..
-			} => {
-				ta.insert_tab();
+	fn process_non_modded(ta: &mut TextArea<'_>, key: Key) -> bool {
+		match key {
+			Key::Char(c) => {
+				ta.insert_char(c);
 				true
 			}
-			Input {
-				key: Key::Char('h'),
-				ctrl: true,
-				alt: false,
-				..
+			Key::Tab => {
+				ta.insert_tab();
+				true
 			}
-			| Input {
-				key: Key::Backspace,
-				ctrl: false,
-				alt: false,
-				..
-			} => {
+			Key::Backspace => {
 				ta.delete_char();
 				true
 			}
-			Input {
-				key: Key::Char('d'),
-				ctrl: true,
-				alt: false,
-				..
+			Key::Delete => {
+				ta.delete_next_char();
+				true
+			}
+			Key::Down => {
+				ta.move_cursor(CursorMove::Down);
+				true
+			}
+			Key::Up => {
+				ta.move_cursor(CursorMove::Up);
+				true
+			}
+			Key::Right => {
+				ta.move_cursor(CursorMove::Forward);
+				true
 			}
-			| Input {
-				key: Key::Delete,
-				ctrl: false,
-				alt: false,
-				..
-			} => {
+			Key::Left => {
+				ta.move_cursor(CursorMove::Back);
+				true
+			}
+			Key::Home => {
+				ta.move_cursor(CursorMove::Head);
+				true
+			}
+			Key::End => {
+				ta.move_cursor(CursorMove::End);
+				true
+			}
+			Key::PageDown => {
+				ta.scroll(Scrolling::PageDown);
+				true
+			}
+			Key::PageUp => {
+				ta.scroll(Scrolling::PageUp);
+				true
+			}
+			_ => false,
+		}
+	}
+
+	fn process_command_key(ta: &mut TextArea<'_>, key: Key) -> bool {
+		match key {
+			Key::Char('h') => {
+				ta.delete_char();
+				true
+			}
+			Key::Char('d') => {
 				ta.delete_next_char();
 				true
 			}
-			Input {
-				key: Key::Char('k'),
-				ctrl: true,
-				alt: false,
-				..
-			} => {
+			Key::Char('k') => {
 				ta.delete_line_by_end();
 				true
 			}
-			Input {
-				key: Key::Char('j'),
-				ctrl: true,
-				alt: false,
-				..
-			} => {
+			Key::Char('j') => {
 				ta.delete_line_by_head();
 				true
 			}
-			Input {
-				key: Key::Char('w'),
-				ctrl: true,
-				alt: false,
-				..
-			}
-			| Input {
-				key: Key::Char('h'),
-				ctrl: false,
-				alt: true,
-				..
-			}
-			| Input {
-				key: Key::Backspace,
-				ctrl: false,
-				alt: true,
-				..
-			} => {
+			Key::Char('w') => {
 				ta.delete_word();
 				true
 			}
-			Input {
-				key: Key::Delete,
-				ctrl: false,
-				alt: true,
-				..
+			Key::Char('n') => {
+				ta.move_cursor(CursorMove::Down);
+				true
 			}
-			| Input {
-				key: Key::Char('d'),
-				ctrl: false,
-				alt: true,
-				..
-			} => {
-				ta.delete_next_word();
+			Key::Char('p') => {
+				ta.move_cursor(CursorMove::Up);
 				true
 			}
-			Input {
-				key: Key::Char('n'),
-				ctrl: true,
-				alt: false,
-				..
+			Key::Char('f') => {
+				ta.move_cursor(CursorMove::Forward);
+				true
 			}
-			| Input {
-				key: Key::Down,
-				ctrl: false,
-				alt: false,
-				..
-			} => {
-				ta.move_cursor(CursorMove::Down);
+			Key::Char('b') => {
+				ta.move_cursor(CursorMove::Back);
 				true
 			}
-			Input {
-				key: Key::Char('p'),
-				ctrl: true,
-				alt: false,
-				..
+			Key::Char('a') => {
+				ta.move_cursor(CursorMove::Head);
+				true
 			}
-			| Input {
-				key: Key::Up,
-				ctrl: false,
-				alt: false,
-				..
-			} => {
-				ta.move_cursor(CursorMove::Up);
+			Key::Char('e') => {
+				ta.move_cursor(CursorMove::End);
 				true
 			}
-			Input {
-				key: Key::Char('f'),
-				ctrl: true,
-				alt: false,
-				..
+			Key::Right => {
+				ta.move_cursor(CursorMove::WordForward);
+				true
 			}
-			| Input {
-				key: Key::Right,
-				ctrl: false,
-				alt: false,
-				..
-			} => {
-				ta.move_cursor(CursorMove::Forward);
+			Key::Left => {
+				ta.move_cursor(CursorMove::WordBack);
 				true
 			}
-			Input {
-				key: Key::Char('b'),
-				ctrl: true,
-				alt: false,
-				..
+			Key::Down => {
+				ta.move_cursor(CursorMove::ParagraphForward);
+				true
 			}
-			| Input {
-				key: Key::Left,
-				ctrl: false,
-				alt: false,
-				..
-			} => {
-				ta.move_cursor(CursorMove::Back);
+			Key::Up => {
+				ta.move_cursor(CursorMove::ParagraphBack);
 				true
 			}
-			Input {
-				key: Key::Char('a'),
-				ctrl: true,
-				alt: false,
-				..
+			Key::Char('u') => {
+				ta.undo();
+				true
 			}
-			| Input { key: Key::Home, .. }
-			| Input {
-				key: Key::Left | Key::Char('b'),
-				ctrl: true,
-				alt: true,
-				..
-			} => {
-				ta.move_cursor(CursorMove::Head);
+			Key::Char('r') => {
+				ta.redo();
 				true
 			}
-			Input {
-				key: Key::Char('e'),
-				ctrl: true,
-				alt: false,
-				..
+			Key::Char('y') => {
+				ta.paste();
+				true
 			}
-			| Input { key: Key::End, .. }
-			| Input {
-				key: Key::Right | Key::Char('f'),
-				ctrl: true,
-				alt: true,
-				..
-			} => {
-				ta.move_cursor(CursorMove::End);
+			// ctrl+v = down, alt+v = up
+			Key::Char('v') => {
+				ta.scroll(Scrolling::PageDown);
+				true
+			}
+			_ => false,
+		}
+	}
+
+	fn process_meta_key(ta: &mut TextArea<'_>, key: Key) -> bool {
+		match key {
+			Key::Char('h') | Key::Backspace => {
+				ta.delete_word();
 				true
 			}
-			Input {
-				key: Key::Char('<'),
-				ctrl: false,
-				alt: true,
-				..
+			Key::Delete | Key::Char('d') => {
+				ta.delete_next_word();
+				true
 			}
-			| Input {
-				key: Key::Up | Key::Char('p'),
-				ctrl: true,
-				alt: true,
-				..
-			} => {
+			Key::Char('<') => {
 				ta.move_cursor(CursorMove::Top);
 				true
 			}
-			Input {
-				key: Key::Char('>'),
-				ctrl: false,
-				alt: true,
-				..
-			}
-			| Input {
-				key: Key::Down | Key::Char('n'),
-				ctrl: true,
-				alt: true,
-				..
-			} => {
+			Key::Char('>') => {
 				ta.move_cursor(CursorMove::Bottom);
 				true
 			}
-			Input {
-				key: Key::Char('f'),
-				ctrl: false,
-				alt: true,
-				..
-			}
-			| Input {
-				key: Key::Right,
-				ctrl: true,
-				alt: false,
-				..
-			} => {
+			Key::Char('f') => {
 				ta.move_cursor(CursorMove::WordForward);
 				true
 			}
-			Input {
-				key: Key::Char('b'),
-				ctrl: false,
-				alt: true,
-				..
-			}
-			| Input {
-				key: Key::Left,
-				ctrl: true,
-				alt: false,
-				..
-			} => {
+			Key::Char('b') => {
 				ta.move_cursor(CursorMove::WordBack);
 				true
 			}
-
-			Input {
-				key: Key::Char(']'),
-				ctrl: false,
-				alt: true,
-				..
-			}
-			| Input {
-				key: Key::Char('n'),
-				ctrl: false,
-				alt: true,
-				..
-			}
-			| Input {
-				key: Key::Down,
-				ctrl: true,
-				alt: false,
-				..
-			} => {
+			Key::Char(']' | 'n') => {
 				ta.move_cursor(CursorMove::ParagraphForward);
 				true
 			}
-			Input {
-				key: Key::Char('['),
-				ctrl: false,
-				alt: true,
-				..
-			}
-			| Input {
-				key: Key::Char('p'),
-				ctrl: false,
-				alt: true,
-				..
-			}
-			| Input {
-				key: Key::Up,
-				ctrl: true,
-				alt: false,
-				..
-			} => {
+			Key::Char('[' | 'p') => {
 				ta.move_cursor(CursorMove::ParagraphBack);
 				true
 			}
-			Input {
-				key: Key::Char('u'),
-				ctrl: true,
-				alt: false,
-				..
-			} => {
-				ta.undo();
+			Key::Char('v') => {
+				ta.scroll(Scrolling::PageUp);
 				true
 			}
-			Input {
-				key: Key::Char('r'),
-				ctrl: true,
-				alt: false,
-				..
-			} => {
-				ta.redo();
+			_ => false,
+		}
+	}
+
+	fn process_meta_command_key(
+		ta: &mut TextArea<'_>,
+		key: Key,
+	) -> bool {
+		match key {
+			Key::Left | Key::Char('b') => {
+				ta.move_cursor(CursorMove::Head);
 				true
 			}
-			Input {
-				key: Key::Char('y'),
-				ctrl: true,
-				alt: false,
-				..
-			} => {
-				ta.paste();
+			Key::Right | Key::Char('f') => {
+				ta.move_cursor(CursorMove::End);
 				true
 			}
-			Input {
-				key: Key::Char('v'),
-				ctrl: true,
-				alt: false,
-				..
-			}
-			| Input {
-				key: Key::PageDown, ..
-			} => {
-				ta.scroll(Scrolling::PageDown);
+			Key::Up | Key::Char('p') => {
+				ta.move_cursor(CursorMove::Top);
 				true
 			}
-			Input {
-				key: Key::Char('v'),
-				ctrl: false,
-				alt: true,
-				..
-			}
-			| Input {
-				key: Key::PageUp, ..
-			} => {
-				ta.scroll(Scrolling::PageUp);
+			Key::Down | Key::Char('n') => {
+				ta.move_cursor(CursorMove::Bottom);
 				true
 			}
 			_ => false,
 		}
 	}
+
+	fn process_inputs(ta: &mut TextArea<'_>, input: &Input) -> bool {
+		match (input.ctrl, input.alt) {
+			(true, true) => {
+				Self::process_meta_command_key(ta, input.key)
+			}
+			(true, false) => Self::process_command_key(ta, input.key),
+			(false, true) => Self::process_meta_key(ta, input.key),
+			(false, false) => Self::process_non_modded(ta, input.key),
+		}
+	}
 }
 
 impl DrawableComponent for TextInputComponent {