From 0b47bd71e7db665b92239d33f6936aa7d6e9ccc1 Mon Sep 17 00:00:00 2001
From: temportalflux <dustin.yost.t@gmail.com>
Date: Thu, 25 Jun 2026 14:03:08 -0400
Subject: [PATCH] add some documentation

---
 crates/gpui_elements/src/editable_text.rs     | 17 ++++++------
 .../gpui_elements/src/editable_text/state.rs  | 26 ++++++++++++-------
 2 files changed, 24 insertions(+), 19 deletions(-)

diff --git a/crates/gpui_elements/src/editable_text.rs b/crates/gpui_elements/src/editable_text.rs
index d638c47934..c2ff354fc0 100644
--- a/crates/gpui_elements/src/editable_text.rs
+++ b/crates/gpui_elements/src/editable_text.rs
@@ -1,3 +1,11 @@
+//! Implementation for editable-text elements (gpui equivalent of html `<input>` and `<textarea>`).
+//! TODO: More documentation
+//!
+//! Backlog of not-yet implemented features:
+//! - text sanitation & validation (see no-op implementation of [`EditableTextState::validate_incoming_text`])
+//! - nav & select via PageUp/PageDown
+//! - masking text (e.g. for passwords)
+
 pub mod actions;
 mod caret;
 mod element;
@@ -9,12 +17,3 @@ mod storage;
 pub use element::*;
 pub use state::*;
 pub use storage::*;
-
-/* TODO list
-- text sanitation
-- add page up/down actions to nav by an entire page or expand selection by an entire page
-- test IME (char palette only available on macos)
-*/
-
-/* Open questions:
-*/
diff --git a/crates/gpui_elements/src/editable_text/state.rs b/crates/gpui_elements/src/editable_text/state.rs
index 29027cce89..448615856c 100644
--- a/crates/gpui_elements/src/editable_text/state.rs
+++ b/crates/gpui_elements/src/editable_text/state.rs
@@ -7,10 +7,9 @@ use crate::editable_text::{
 };
 use gpui::{
     App, Bounds, ClipboardItem, Context, Entity, EntityInputHandler, EventEmitter, FocusHandle,
-    Focusable, NavigationDirection, Pixels, Point, Size, UTF16Selection, Window, WrappedLine,
-    point,
+    Focusable, NavigationDirection, Pixels, Point, UTF16Selection, Window, point,
 };
-use std::{borrow::Cow, ops::Range, sync::Arc};
+use std::{borrow::Cow, ops::Range};
 
 /// Event emitted via EditableText elements when the internal storage contents have changed
 pub struct TextChanged;
@@ -129,23 +128,30 @@ impl EditableTextState {
     /// Validates/santizes incoming text according to the rules of the field.
     fn validate_incoming_text<'text>(
         &self,
-        range: &Range<usize>,
-        mut text_to_insert: &'text str,
+        _range: &Range<usize>,
+        text_to_insert: &'text str,
     ) -> &'text str {
-        // TODO: Apply text sanitization
-        // single-line fields should prune \n and \r
-        // fields should be able to provide a max_length or other validations on text-input
-
-        let max_length = None::<usize>;
+        // TODO: Apply text sanitization, ideally using externally-sourced implementations.
+        // example optional/opt-in sanitations include:
+        // - single-line fields should prune /n & /r
+        // - maximum utf8 length
+        // - numbers only
+        // should also consider validation support, for features such as:
+        // - total syntax evaluation (e.g. passwords)
+        // - conforms to regex or math (e.g. ssn, phone number, email, etc)
 
+        /* A sample implementation of max-length sampled from gpuikit
         // Decide the effective new text up front (honouring `max_length`).
         // This avoids the "apply, then truncate" path which would leave the caret past the end.
+        let max_length = None::<usize>;
         if let Some(cap) = max_length {
             let existing_len = self.storage().content_utf8().len() - (range.end - range.start);
             let room = cap.saturating_sub(existing_len);
             text_to_insert = &text_to_insert[..text_to_insert.len().min(room)];
         }
+        */
 
+        // for now, this function is no-op
         text_to_insert
     }