Cargo.toml

@@ -1,13 +1,13 @@
 [package]
 name = "ced"
-version = "0.0.9"
+version = "0.0.4"
 edition = "2024"
 
 [dependencies]
-eframe = "0.32"
+eframe = "0.31"
-egui = "0.32"
+egui = "0.31"
-serde = { version = "1.0.219", features = ["derive"] }
+serde = { version = "1.0", features = ["derive"] }
-rfd = "0.15.4"
+rfd = "0.15"
-toml = "0.9.2"
+toml = "0.8"
-dirs = "6.0"
+dirs = "5.0"
-libc = "0.2.174"
+libc = "0.2"

README.md

@@ -53,7 +53,6 @@ font_size = 16.0
 | Option | Default | Description |
 |--------|---------|-------------|
 | `auto_hide_toolbar` | `false` | If `true`, the menu bar at the top will be hidden. Move your mouse to the top of the window to reveal it. |
-| `hide_tab_bar` | 'true' | If `false`, a separate tab bar will be drawn below the toolbar. |
 | `show_line_numbers` | `false` | If `true`, line numbers will be displayed on the side specified by `line_side`. |
 | `line_side` | `false` | If `false`, line numbers are on the left. If `true`, they are on the right. |
 | `word_wrap` | `false` | If `true`, lines will wrap when they reach the edge of the window. |
@@ -65,7 +64,7 @@ font_size = 16.0
 In order of importance.
 | Feature | Info |
 | ------- | ---- |
-| **Find/Replace:** | Functioning. |
+| **Find/Replace:** | In progress. |
 | **State/Cache:** | A toggleable option to keep an application state and prevent "Quit without saving" warnings. |
 | **Syntax Highlighting/LSP:** | Looking at allowing you to use/attach your own tools for this. |
 | **Choose Font** | More than just Monospace/Proportional. |

src/app/config.rs

@@ -5,52 +5,22 @@ use super::theme::Theme;
 
 #[derive(Debug, Clone, Serialize, Deserialize)]
 pub struct Config {
-    #[serde(default = "default_auto_hide_toolbar")]
     pub auto_hide_toolbar: bool,
-    #[serde(default = "default_hide_tab_bar")]
+    pub auto_hide_tab_bar: bool,
-    pub hide_tab_bar: bool,
-    #[serde(default = "default_show_line_numbers")]
     pub show_line_numbers: bool,
-    #[serde(default = "default_word_wrap")]
     pub word_wrap: bool,
-    #[serde(default = "Theme::default")]
     pub theme: Theme,
-    #[serde(default = "default_line_side")]
     pub line_side: bool,
-    #[serde(default = "default_font_family")]
     pub font_family: String,
-    #[serde(default = "default_font_size")]
     pub font_size: f32,
     // pub vim_mode: bool,
 }
 
-fn default_auto_hide_toolbar() -> bool {
-    false
-}
-fn default_hide_tab_bar() -> bool {
-    true
-}
-fn default_show_line_numbers() -> bool {
-    false
-}
-fn default_word_wrap() -> bool {
-    true
-}
-fn default_line_side() -> bool {
-    false
-}
-fn default_font_family() -> String {
-    "Proportional".to_string()
-}
-fn default_font_size() -> f32 {
-    14.0
-}
-
 impl Default for Config {
     fn default() -> Self {
         Self {
             auto_hide_toolbar: false,
-            hide_tab_bar: true,
+            auto_hide_tab_bar: false,
             show_line_numbers: false,
             word_wrap: true,
             theme: Theme::default(),
@@ -65,11 +35,9 @@ impl Default for Config {
 impl Config {
     pub fn config_path() -> Option<PathBuf> {
         let config_dir = if let Some(config_dir) = dirs::config_dir() {
-            config_dir.join(format!("{}", env!("CARGO_PKG_NAME")))
+            config_dir.join("ced")
         } else if let Some(home_dir) = dirs::home_dir() {
-            home_dir
+            home_dir.join(".config").join("ced")
-                .join(".config")
-                .join(format!("{}", env!("CARGO_PKG_NAME")))
         } else {
             return None;
         };
@@ -92,23 +60,17 @@ impl Config {
         }
 
         match std::fs::read_to_string(&config_path) {
-            Ok(content) => {
+            Ok(content) => match toml::from_str::<Config>(&content) {
-                let mut config = match toml::from_str::<Config>(&content) {
+                Ok(config) => config,
-                    Ok(config) => config,
+                Err(e) => {
-                    Err(e) => {
+                    eprintln!(
-                        eprintln!(
+                        "Failed to parse config file {}: {}",
-                            "Failed to parse config file {}: {}",
+                        config_path.display(),
-                            config_path.display(),
+                        e
-                            e
+                    );
-                        );
+                    Self::default()
-                        return Self::default();
+                }
-                    }
+            },
-                };
-
-                let default_config = Self::default();
-                config.merge_with_default(default_config);
-                config
-            }
             Err(e) => {
                 eprintln!(
                     "Failed to read config file {}: {}",
@@ -120,16 +82,6 @@ impl Config {
         }
     }
 
-    fn merge_with_default(&mut self, default: Config) {
-        if self.font_family.is_empty() {
-            self.font_family = default.font_family;
-        }
-
-        if self.font_size <= 0.0 {
-            self.font_size = default.font_size;
-        }
-    }
-
     pub fn save(&self) -> Result<(), Box<dyn std::error::Error>> {
         let config_path = Self::config_path().ok_or("Cannot determine config directory")?;
 

src/app/shortcuts.rs

@@ -15,7 +15,6 @@ enum ShortcutAction {
     ToggleWordWrap,
     ToggleAutoHideToolbar,
     ToggleFind,
-    FocusFind,
     NextTab,
     PrevTab,
     PageUp,
@@ -56,11 +55,6 @@ fn get_shortcuts() -> Vec<ShortcutDefinition> {
             egui::Key::W,
             ShortcutAction::CloseTab,
         ),
-        (
-            egui::Modifiers::CTRL | egui::Modifiers::SHIFT,
-            egui::Key::F,
-            ShortcutAction::FocusFind,
-        ),
         (
             egui::Modifiers::CTRL,
             egui::Key::F,
@@ -149,7 +143,7 @@ fn get_shortcuts() -> Vec<ShortcutDefinition> {
     ]
 }
 
-fn execute_action(action: ShortcutAction, editor: &mut TextEditor) -> bool {
+fn execute_action(action: ShortcutAction, editor: &mut TextEditor, _ctx: &egui::Context) -> bool {
     match action {
         ShortcutAction::NewFile => {
             io::new_file(editor);
@@ -173,12 +167,15 @@ fn execute_action(action: ShortcutAction, editor: &mut TextEditor) -> bool {
         }
         ShortcutAction::CloseTab => {
             if editor.tabs.len() > 1 {
+                // Check if the current tab has unsaved changes
                 if let Some(current_tab) = editor.get_active_tab() {
                     if current_tab.is_modified {
+                        // Show dialog for unsaved changes
                         editor.pending_unsaved_action = Some(
                             super::state::UnsavedAction::CloseTab(editor.active_tab_index),
                         );
                     } else {
+                        // Close tab directly if no unsaved changes
                         editor.close_tab(editor.active_tab_index);
                     }
                 }
@@ -254,25 +251,13 @@ fn execute_action(action: ShortcutAction, editor: &mut TextEditor) -> bool {
         ShortcutAction::Escape => {
             editor.show_about = false;
             editor.show_shortcuts = false;
-            if editor.show_find {
-                editor.should_select_current_match = true;
-            }
             editor.show_find = false;
             editor.show_preferences = false;
             editor.pending_unsaved_action = None;
             false
         }
         ShortcutAction::ToggleFind => {
-            editor.show_find = !editor.show_find;
+            //editor.show_find = !editor.show_find;
-            if editor.show_find && !editor.find_query.is_empty() {
-                editor.update_find_matches();
-            }
-            false
-        }
-        ShortcutAction::FocusFind => {
-            if editor.show_find {
-                editor.focus_find = true;
-            }
             false
         }
         ShortcutAction::Preferences => {
@@ -291,16 +276,16 @@ pub fn handle(editor: &mut TextEditor, ctx: &egui::Context) {
             if i.consume_key(modifiers, key) {
                 match action {
                     ShortcutAction::ZoomIn | ShortcutAction::ZoomOut => {
-                        font_zoom_occurred = execute_action(action, editor);
+                        font_zoom_occurred = execute_action(action, editor, ctx);
                     }
                     ShortcutAction::GlobalZoomIn
                     | ShortcutAction::GlobalZoomOut
                     | ShortcutAction::ResetZoom => {
-                        execute_action(action, editor);
+                        execute_action(action, editor, ctx);
                         global_zoom_occurred = true;
                     }
                     _ => {
-                        execute_action(action, editor);
+                        execute_action(action, editor, ctx);
                     }
                 }
                 break;
@@ -315,9 +300,4 @@ pub fn handle(editor: &mut TextEditor, ctx: &egui::Context) {
     if global_zoom_occurred {
         ctx.set_zoom_factor(editor.zoom_factor);
     }
-
-    if editor.should_select_current_match {
-        editor.select_current_match(ctx);
-        editor.should_select_current_match = false;
-    }
 }

src/app/state/app_impl.rs

@@ -24,7 +24,7 @@ impl eframe::App for TextEditor {
 
         menu_bar(self, ctx);
 
-        if !self.hide_tab_bar {
+        if !self.auto_hide_tab_bar {
             tab_bar(self, ctx);
         }
 
@@ -46,6 +46,7 @@ impl eframe::App for TextEditor {
             self.show_unsaved_changes_dialog(ctx);
         }
 
+        // Update the previous find state for next frame
         self.prev_show_find = self.show_find;
     }
 }

src/app/state/config.rs

@@ -19,7 +19,7 @@ impl TextEditor {
             show_line_numbers: config.show_line_numbers,
             word_wrap: config.word_wrap,
             auto_hide_toolbar: config.auto_hide_toolbar,
-            hide_tab_bar: config.hide_tab_bar,
+            auto_hide_tab_bar: config.auto_hide_tab_bar,
             theme: config.theme,
             line_side: config.line_side,
             font_family: config.font_family,
@@ -32,22 +32,21 @@ impl TextEditor {
             text_processing_result: std::sync::Arc::new(std::sync::Mutex::new(Default::default())),
             _processing_thread_handle: None,
             find_query: String::new(),
-            replace_query: String::new(),
             find_matches: Vec::new(),
             current_match_index: None,
             case_sensitive_search: false,
-            show_replace_section: false,
             prev_show_find: false,
-            focus_find: false,
             // vim_mode: config.vim_mode,
+
+            // Cursor tracking for smart scrolling
             previous_cursor_position: None,
+
+            // Track previous content for incremental processing
             previous_content: String::new(),
             previous_cursor_char_index: None,
             current_cursor_line: 0,
             previous_cursor_line: 0,
             font_settings_changed: false,
-            text_needs_processing: false,
-            should_select_current_match: false,
         }
     }
 
@@ -57,7 +56,7 @@ impl TextEditor {
 
         cc.egui_ctx.options_mut(|o| o.zoom_with_keyboard = false);
 
-        let mut style = (*cc.egui_ctx.style()).to_owned();
+        let mut style = (*cc.egui_ctx.style()).clone();
         style
             .text_styles
             .insert(egui::TextStyle::Body, egui::FontId::proportional(16.0));
@@ -81,11 +80,11 @@ impl TextEditor {
         Config {
             auto_hide_toolbar: self.auto_hide_toolbar,
             show_line_numbers: self.show_line_numbers,
-            hide_tab_bar: self.hide_tab_bar,
+            auto_hide_tab_bar: self.auto_hide_tab_bar,
             word_wrap: self.word_wrap,
             theme: self.theme,
             line_side: self.line_side,
-            font_family: self.font_family.to_string(),
+            font_family: self.font_family.clone(),
             font_size: self.font_size,
             // vim_mode: self.vim_mode,
         }

src/app/state/default.rs

@@ -19,7 +19,7 @@ impl Default for TextEditor {
             show_line_numbers: false,
             word_wrap: true,
             auto_hide_toolbar: false,
-            hide_tab_bar: true,
+            auto_hide_tab_bar: true,
             theme: Theme::default(),
             line_side: false,
             font_family: "Proportional".to_string(),
@@ -31,22 +31,22 @@ impl Default for TextEditor {
             menu_bar_stable_until: None,
             text_processing_result: Arc::new(Mutex::new(TextProcessingResult::default())),
             _processing_thread_handle: None,
+            // Find functionality
             find_query: String::new(),
-            replace_query: String::new(),
             find_matches: Vec::new(),
             current_match_index: None,
             case_sensitive_search: false,
-            show_replace_section: false,
             prev_show_find: false,
-            focus_find: false,
+
+            // Cursor tracking for smart scrolling
             previous_cursor_position: None,
+
+            // Track previous content for incremental processing
             previous_content: String::new(),
             previous_cursor_char_index: None,
             current_cursor_line: 0,
             previous_cursor_line: 0,
             font_settings_changed: false,
-            text_needs_processing: false,
-            should_select_current_match: false,
         }
     }
 }

src/app/state/editor.rs

@@ -13,9 +13,9 @@ pub enum UnsavedAction {
 #[derive(Clone)]
 pub struct TextProcessingResult {
     pub line_count: usize,
-    pub longest_line_index: usize,
+    pub longest_line_index: usize, // Which line is the longest (0-based)
-    pub longest_line_length: usize,
+    pub longest_line_length: usize, // Character count of the longest line
-    pub longest_line_pixel_width: f32,
+    pub longest_line_pixel_width: f32, // Actual pixel width of the longest line
 }
 
 impl Default for TextProcessingResult {
@@ -33,7 +33,7 @@ impl Default for TextProcessingResult {
 pub struct TextEditor {
     pub(crate) tabs: Vec<Tab>,
     pub(crate) active_tab_index: usize,
-    pub(crate) tab_counter: usize,
+    pub(crate) tab_counter: usize, // Counter for numbering new tabs
     pub(crate) show_about: bool,
     pub(crate) show_shortcuts: bool,
     pub(crate) show_find: bool,
@@ -44,7 +44,7 @@ pub struct TextEditor {
     pub(crate) show_line_numbers: bool,
     pub(crate) word_wrap: bool,
     pub(crate) auto_hide_toolbar: bool,
-    pub(crate) hide_tab_bar: bool,
+    pub(crate) auto_hide_tab_bar: bool,
     pub(crate) theme: Theme,
     pub(crate) line_side: bool,
     pub(crate) font_family: String,
@@ -57,19 +57,18 @@ pub struct TextEditor {
     pub(crate) text_processing_result: Arc<Mutex<TextProcessingResult>>,
     pub(crate) _processing_thread_handle: Option<thread::JoinHandle<()>>,
     pub(crate) find_query: String,
-    pub(crate) replace_query: String,
+    pub(crate) find_matches: Vec<(usize, usize)>, // (start_pos, end_pos) byte positions
-    pub(crate) find_matches: Vec<(usize, usize)>,
     pub(crate) current_match_index: Option<usize>,
     pub(crate) case_sensitive_search: bool,
-    pub(crate) show_replace_section: bool,
+    pub(crate) prev_show_find: bool, // Track previous state to detect transitions
-    pub(crate) prev_show_find: bool,
+
-    pub(crate) focus_find: bool,
+    // Cursor tracking for smart scrolling
+    pub(crate) previous_cursor_position: Option<usize>,
+
+    // Track previous content for incremental processing
     pub(crate) previous_content: String,
     pub(crate) previous_cursor_char_index: Option<usize>,
-    pub(crate) current_cursor_line: usize,
+    pub(crate) current_cursor_line: usize, // Track current line number incrementally
-    pub(crate) previous_cursor_line: usize,
+    pub(crate) previous_cursor_line: usize, // Track previous line for comparison
-    pub(crate) font_settings_changed: bool,
+    pub(crate) font_settings_changed: bool, // Flag to trigger text reprocessing when font changes
-    pub(crate) text_needs_processing: bool,
-    pub(crate) should_select_current_match: bool,
-    pub(crate) previous_cursor_position: Option<usize>,
 }

src/app/state/find.rs

@@ -1,9 +1,7 @@
 use super::editor::TextEditor;
-use eframe::egui;
 
 impl TextEditor {
     pub fn update_find_matches(&mut self) {
-        let previous_match_index = self.current_match_index;
         self.find_matches.clear();
         self.current_match_index = None;
 
@@ -14,62 +12,32 @@ impl TextEditor {
         if let Some(tab) = self.get_active_tab() {
             let content = &tab.content;
             let query = if self.case_sensitive_search {
-                self.find_query.to_owned()
+                self.find_query.clone()
             } else {
                 self.find_query.to_lowercase()
             };
 
             let search_content = if self.case_sensitive_search {
-                content.to_string()
+                content.clone()
             } else {
                 content.to_lowercase()
             };
 
             let mut start = 0;
-            while start < search_content.len() {
+            while let Some(pos) = search_content[start..].find(&query) {
-                let search_slice = if search_content.is_char_boundary(start) {
+                let absolute_pos = start + pos;
-                    &search_content[start..]
+                self.find_matches
-                } else {
+                    .push((absolute_pos, absolute_pos + query.len()));
-                    // Find next valid boundary
+                start = absolute_pos + 1;
-                    while start < search_content.len() && !search_content.is_char_boundary(start) {
-                        start += 1;
-                    }
-                    if start >= search_content.len() {
-                        break;
-                    }
-                    &search_content[start..]
-                };
-
-                if let Some(pos) = search_slice.find(&query) {
-                    let absolute_pos = start + pos;
-                    self.find_matches
-                        .push((absolute_pos, absolute_pos + query.len()));
-
-                    // Advance to next valid character boundary instead of just +1
-                    start = absolute_pos + 1;
-                    while start < search_content.len() && !search_content.is_char_boundary(start) {
-                        start += 1;
-                    }
-                } else {
-                    break;
-                }
             }
 
             if !self.find_matches.is_empty() {
-                if let Some(prev_index) = previous_match_index {
+                self.current_match_index = Some(0);
-                    if prev_index < self.find_matches.len() {
-                        self.current_match_index = Some(prev_index);
-                    } else {
-                        self.current_match_index = Some(0);
-                    }
-                } else {
-                    self.current_match_index = Some(0);
-                }
             }
         }
     }
 
-    pub fn find_next(&mut self, ctx: &egui::Context) {
+    pub fn find_next(&mut self) {
         if self.find_matches.is_empty() {
             return;
         }
@@ -79,12 +47,9 @@ impl TextEditor {
         } else {
             self.current_match_index = Some(0);
         }
-
-        self.select_current_match(ctx);
-        self.should_select_current_match = true;
     }
 
-    pub fn find_previous(&mut self, ctx: &egui::Context) {
+    pub fn find_previous(&mut self) {
         if self.find_matches.is_empty() {
             return;
         }
@@ -98,9 +63,6 @@ impl TextEditor {
         } else {
             self.current_match_index = Some(0);
         }
-
-        self.select_current_match(ctx);
-        self.should_select_current_match = true;
     }
 
     pub fn get_current_match_position(&self) -> Option<(usize, usize)> {
@@ -110,112 +72,4 @@ impl TextEditor {
             None
         }
     }
-
-    pub fn select_current_match(&self, ctx: &egui::Context) {
-        if let Some((start_byte, end_byte)) = self.get_current_match_position() {
-            if let Some(active_tab) = self.get_active_tab() {
-                let content = &active_tab.content;
-
-                let start_char = Self::safe_slice_to_pos(content, start_byte).chars().count();
-                let end_char = Self::safe_slice_to_pos(content, end_byte).chars().count();
-
-                let text_edit_id = egui::Id::new("main_text_editor");
-                if let Some(mut state) = egui::TextEdit::load_state(ctx, text_edit_id) {
-                    let selection_range = egui::text::CCursorRange::two(
-                        egui::text::CCursor::new(start_char),
-                        egui::text::CCursor::new(end_char),
-                    );
-                    state.cursor.set_char_range(Some(selection_range));
-                    egui::TextEdit::store_state(ctx, text_edit_id, state);
-                }
-            }
-        }
-    }
-
-    pub fn replace_current_match(&mut self, ctx: &egui::Context) {
-        if self.find_query.is_empty() || self.find_matches.is_empty() {
-            return;
-        }
-
-        if let Some((start_byte, end_byte)) = self.get_current_match_position() {
-            let replace_query = self.replace_query.to_owned();
-            let replacement_end = start_byte + replace_query.len();
-
-            if let Some(active_tab) = self.get_active_tab_mut() {
-                let content = &active_tab.content;
-
-                let mut new_content = content.to_string();
-                new_content.replace_range(start_byte..end_byte, &replace_query);
-
-                active_tab.content = new_content;
-                active_tab.is_modified = true;
-            }
-
-            self.update_find_matches();
-
-            if let Some(active_tab) = self.get_active_tab() {
-                let replacement_end_char =
-                    Self::safe_slice_to_pos(&active_tab.content, replacement_end)
-                        .chars()
-                        .count();
-
-                let text_edit_id = egui::Id::new("main_text_editor");
-                if let Some(mut state) = egui::TextEdit::load_state(ctx, text_edit_id) {
-                    state
-                        .cursor
-                        .set_char_range(Some(egui::text::CCursorRange::one(
-                            egui::text::CCursor::new(replacement_end_char),
-                        )));
-                    egui::TextEdit::store_state(ctx, text_edit_id, state);
-                }
-            }
-        }
-    }
-
-    pub fn replace_all(&mut self, ctx: &egui::Context) {
-        if self.find_query.is_empty() || self.find_matches.is_empty() {
-            return;
-        }
-
-        let find_query = self.find_query.to_owned();
-        let replace_query = self.replace_query.to_owned();
-        let case_sensitive = self.case_sensitive_search;
-        let find_matches = self.find_matches.to_owned();
-
-        if let Some(active_tab) = self.get_active_tab_mut() {
-            let content = &active_tab.content;
-
-            let new_content = if case_sensitive {
-                content.replace(&find_query, &replace_query)
-            } else {
-                let mut result = String::new();
-                let mut last_end = 0;
-
-                for (start_byte, end_byte) in &find_matches {
-                    result.push_str(&content[last_end..*start_byte]);
-                    result.push_str(&replace_query);
-                    last_end = *end_byte;
-                }
-                result.push_str(&content[last_end..]);
-                result
-            };
-
-            active_tab.content = new_content;
-            active_tab.is_modified = true;
-        }
-
-        self.update_find_matches();
-
-        self.current_match_index = None;
-
-        let text_edit_id = egui::Id::new("main_text_editor");
-        if let Some(mut state) = egui::TextEdit::load_state(ctx, text_edit_id) {
-            state
-                .cursor
-                .set_char_range(Some(egui::text::CCursorRange::one(
-                    egui::text::CCursor::new(0),
-                )));
-            egui::TextEdit::store_state(ctx, text_edit_id, state);
-        }
-    }
 }

src/app/state/lifecycle.rs

@@ -10,7 +10,7 @@ impl TextEditor {
         self.tabs
             .iter()
             .filter(|tab| tab.is_modified)
-            .map(|tab| tab.title.to_owned())
+            .map(|tab| tab.title.clone())
             .collect()
     }
 
@@ -35,33 +35,29 @@ impl TextEditor {
         let (files_to_list, title, confirmation_text, button_text, action) =
             if let Some(action) = &self.pending_unsaved_action {
                 match action {
-                    UnsavedAction::Quit => {
+                    UnsavedAction::Quit => (
-                        let files = self.get_unsaved_files();
+                        self.get_unsaved_files(),
-                        let file_plural = if files.len() > 1 { "s" } else { "" };
+                        "Unsaved Changes".to_string(),
-                        (
+                        "You have unsaved changes.".to_string(),
-                            files,
+                        "Quit Without Saving".to_string(),
-                            "Unsaved Changes".to_string(),
+                        action.clone(),
-                            format!("File{file_plural} with unsaved changes:"),
+                    ),
-                            "Quit Without Saving".to_string(),
-                            action.to_owned(),
-                        )
-                    }
                     UnsavedAction::CloseTab(tab_index) => {
                         let file_name = self
                             .tabs
                             .get(*tab_index)
-                            .map_or_else(|| "unknown file".to_string(), |tab| tab.title.to_owned());
+                            .map_or_else(|| "unknown file".to_string(), |tab| tab.title.clone());
                         (
                             vec![file_name],
                             "Unsaved Changes".to_string(),
-                            "This file has unsaved changes:".to_string(),
+                            "The file has unsaved changes.".to_string(),
                             "Close Without Saving".to_string(),
-                            action.to_owned(),
+                            action.clone(),
                         )
                     }
                 }
             } else {
-                return;
+                return; // Should not happen if called correctly
             };
 
         let visuals = &ctx.style().visuals;

src/app/state/processing.rs

@@ -2,21 +2,12 @@ use super::editor::{TextEditor, TextProcessingResult};
 use eframe::egui;
 
 impl TextEditor {
-    pub(crate) fn safe_slice_to_pos(content: &str, pos: usize) -> &str {
+    /// Process text content and find the longest line (only used for initial scan)
-        let pos = pos.min(content.len());
-        let mut boundary_pos = pos;
-        while boundary_pos > 0 && !content.is_char_boundary(boundary_pos) {
-            boundary_pos -= 1;
-        }
-        &content[..boundary_pos]
-    }
-
     pub fn process_text_for_rendering(&mut self, content: &str, ui: &egui::Ui) {
-        let line_count = content.bytes().filter(|&b| b == b'\n').count() + 1;
-
         let lines: Vec<&str> = content.lines().collect();
+        let line_count = lines.len().max(1);
 
-        if content.is_empty() {
+        if lines.is_empty() {
             self.update_processing_result(TextProcessingResult {
                 line_count: 1,
                 longest_line_index: 0,
@@ -29,16 +20,6 @@ impl TextEditor {
         let mut longest_line_index = 0;
         let mut longest_line_length = 0;
 
-        if lines.is_empty() {
-            self.update_processing_result(TextProcessingResult {
-                line_count,
-                longest_line_index: 0,
-                longest_line_length: 0,
-                longest_line_pixel_width: 0.0,
-            });
-            return;
-        }
-
         for (index, line) in lines.iter().enumerate() {
             let char_count = line.chars().count();
             if char_count > longest_line_length {
@@ -75,6 +56,7 @@ impl TextEditor {
         self.update_processing_result(result);
     }
 
+    /// Efficiently detect and process line changes without full content iteration
     pub fn process_incremental_change(
         &mut self,
         old_content: &str,
@@ -121,6 +103,7 @@ impl TextEditor {
         self.previous_cursor_line = self.current_cursor_line;
     }
 
+    /// Calculate the change in cursor line without full iteration
     fn calculate_cursor_line_change(
         &self,
         old_content: &str,
@@ -128,12 +111,12 @@ impl TextEditor {
         old_cursor_pos: usize,
         new_cursor_pos: usize,
     ) -> isize {
-        let old_newlines = Self::safe_slice_to_pos(old_content, old_cursor_pos)
+        let old_newlines = old_content[..old_cursor_pos.min(old_content.len())]
             .bytes()
             .filter(|&b| b == b'\n')
             .count();
 
-        let new_newlines = Self::safe_slice_to_pos(new_content, new_cursor_pos)
+        let new_newlines = new_content[..new_cursor_pos.min(new_content.len())]
             .bytes()
             .filter(|&b| b == b'\n')
             .count();
@@ -141,6 +124,7 @@ impl TextEditor {
         new_newlines as isize - old_newlines as isize
     }
 
+    /// Handle character replacement (same length change)
     fn handle_character_replacement(
         &mut self,
         _old_content: &str,
@@ -160,6 +144,7 @@ impl TextEditor {
         );
     }
 
+    /// Handle content addition
     fn handle_content_addition(
         &mut self,
         old_content: &str,
@@ -171,6 +156,7 @@ impl TextEditor {
         let min_len = old_content.len().min(new_content.len());
         let mut common_prefix = 0;
         let mut common_suffix = 0;
+
         for i in 0..min_len {
             if old_content.as_bytes()[i] == new_content.as_bytes()[i] {
                 common_prefix += 1;
@@ -197,38 +183,21 @@ impl TextEditor {
         if newlines_added > 0 {
             let mut current_result = self.get_text_processing_result();
             current_result.line_count += newlines_added;
-
+            self.update_processing_result(current_result);
-            let addition_start_line = Self::safe_slice_to_pos(old_content, added_start)
-                .bytes()
-                .filter(|&b| b == b'\n')
-                .count();
-            let addition_end_line = Self::safe_slice_to_pos(old_content, added_end)
-                .bytes()
-                .filter(|&b| b == b'\n')
-                .count();
-
-            if current_result.longest_line_index >= addition_start_line
-                && current_result.longest_line_index <= addition_end_line
-            {
-                self.process_text_for_rendering(new_content, ui);
-            } else {
-                if addition_end_line < current_result.longest_line_index {
-                    current_result.longest_line_index += newlines_added;
-                }
-                self.update_processing_result(current_result);
-            }
-        } else {
-            let current_line = self.extract_current_line(new_content, new_cursor_pos);
-            let current_line_length = current_line.chars().count();
-            self.update_line_if_longer(
-                self.current_cursor_line,
-                &current_line,
-                current_line_length,
-                ui,
-            );
         }
+
+        let current_line = self.extract_current_line(new_content, new_cursor_pos);
+        let current_line_length = current_line.chars().count();
+
+        self.update_line_if_longer(
+            self.current_cursor_line,
+            &current_line,
+            current_line_length,
+            ui,
+        );
     }
 
+    /// Handle content removal
     fn handle_content_removal(
         &mut self,
         old_content: &str,
@@ -268,27 +237,11 @@ impl TextEditor {
             let mut current_result = self.get_text_processing_result();
             current_result.line_count = current_result.line_count.saturating_sub(newlines_removed);
 
-            let removal_start_line = Self::safe_slice_to_pos(old_content, removed_start)
+            if self.current_cursor_line <= current_result.longest_line_index {
-                .bytes()
-                .filter(|&b| b == b'\n')
-                .count();
-            let removal_end_line = Self::safe_slice_to_pos(old_content, removed_end)
-                .bytes()
-                .filter(|&b| b == b'\n')
-                .count();
-
-            if current_result.longest_line_index >= removal_start_line
-                && current_result.longest_line_index <= removal_end_line
-            {
                 self.process_text_for_rendering(new_content, ui);
-            } else {
-                if removal_end_line < current_result.longest_line_index {
-                    current_result.longest_line_index = current_result
-                        .longest_line_index
-                        .saturating_sub(newlines_removed);
-                }
-                self.update_processing_result(current_result);
             }
+
+            self.update_processing_result(current_result);
         }
 
         let current_line = self.extract_current_line(new_content, new_cursor_pos);
@@ -309,32 +262,24 @@ impl TextEditor {
         }
     }
 
+    /// Extract the current line efficiently without full content scan
     fn extract_current_line(&self, content: &str, cursor_pos: usize) -> String {
         let bytes = content.as_bytes();
-        let safe_cursor_pos = cursor_pos.min(bytes.len());
 
-        let mut line_start = safe_cursor_pos;
+        let mut line_start = cursor_pos;
         while line_start > 0 && bytes[line_start - 1] != b'\n' {
             line_start -= 1;
         }
 
-        let mut line_end = safe_cursor_pos;
+        let mut line_end = cursor_pos;
         while line_end < bytes.len() && bytes[line_end] != b'\n' {
             line_end += 1;
         }
 
-        let line_start_boundary = line_start;
+        content[line_start..line_end].to_string()
-        let line_end_boundary = line_end;
-
-        if content.is_char_boundary(line_start_boundary)
-            && content.is_char_boundary(line_end_boundary)
-        {
-            content[line_start_boundary..line_end_boundary].to_string()
-        } else {
-            Self::safe_slice_to_pos(content, line_end_boundary)[line_start_boundary..].to_string()
-        }
     }
 
+    /// Update longest line info if the current line is longer
     fn update_line_if_longer(
         &mut self,
         line_index: usize,
@@ -369,13 +314,15 @@ impl TextEditor {
         }
     }
 
+    /// Get the current text processing result
     pub fn get_text_processing_result(&self) -> TextProcessingResult {
         self.text_processing_result
             .lock()
-            .map(|result| result.to_owned())
+            .map(|result| result.clone())
             .unwrap_or_default()
     }
 
+    /// Update the processing result atomically
     fn update_processing_result(&self, result: TextProcessingResult) {
         if let Ok(mut processing_result) = self.text_processing_result.lock() {
             *processing_result = result;

src/app/state/tabs.rs

@@ -14,10 +14,6 @@ impl TextEditor {
         self.tab_counter += 1;
         self.tabs.push(Tab::new_empty(self.tab_counter));
         self.active_tab_index = self.tabs.len() - 1;
-        if self.show_find && !self.find_query.is_empty() {
-            self.update_find_matches();
-        }
-        self.text_needs_processing = true;
     }
 
     pub fn close_tab(&mut self, tab_index: usize) {
@@ -28,20 +24,12 @@ impl TextEditor {
             } else if self.active_tab_index > tab_index {
                 self.active_tab_index -= 1;
             }
-            if self.show_find && !self.find_query.is_empty() {
-                self.update_find_matches();
-            }
-            self.text_needs_processing = true;
         }
     }
 
     pub fn switch_to_tab(&mut self, tab_index: usize) {
         if tab_index < self.tabs.len() {
             self.active_tab_index = tab_index;
-            if self.show_find && !self.find_query.is_empty() {
-                self.update_find_matches();
-            }
-            self.text_needs_processing = true;
         }
     }
 }

src/app/state/ui.rs

@@ -45,7 +45,7 @@ impl TextEditor {
             _ => egui::FontFamily::Proportional,
         };
 
-        let mut style = (*ctx.style()).to_owned();
+        let mut style = (*ctx.style()).clone();
         style.text_styles.insert(
             egui::TextStyle::Monospace,
             egui::FontId::new(self.font_size, font_family),
@@ -66,7 +66,10 @@ impl TextEditor {
     /// Trigger immediate text reprocessing when font settings change
     pub fn reprocess_text_for_font_change(&mut self, ui: &egui::Ui) {
         if let Some(active_tab) = self.get_active_tab() {
-            self.process_text_for_rendering(&active_tab.content.to_string(), ui);
+            let content = active_tab.content.clone();
+            if !content.is_empty() {
+                self.process_text_for_rendering(&content, ui);
+            }
         }
     }
 
@@ -122,8 +125,7 @@ impl TextEditor {
             return self.calculate_editor_dimensions(ui).text_width;
         }
 
-        let longest_line_width =
+        let longest_line_width = processing_result.longest_line_pixel_width + self.font_size;
-            processing_result.longest_line_pixel_width + (self.font_size * 2.0);
 
         let dimensions = self.calculate_editor_dimensions(ui);
         longest_line_width.max(dimensions.text_width)

src/app/tab.rs

@@ -60,6 +60,8 @@ impl Tab {
     }
 
     pub fn update_modified_state(&mut self) {
+        // Compare current content hash with original content hash to determine if modified
+        // Special case: new_X tabs are only considered modified if they have content
         if self.title.starts_with("new_") {
             self.is_modified = !self.content.is_empty();
         } else {
@@ -70,6 +72,7 @@ impl Tab {
     }
 
     pub fn mark_as_saved(&mut self) {
+        // Update the original content hash to match current content after saving
         self.original_content_hash = compute_content_hash(&self.content, &mut self.hasher);
         self.last_content_hash = self.original_content_hash;
         self.is_modified = false;

src/io.rs

@@ -14,6 +14,7 @@ pub(crate) fn open_file(app: &mut TextEditor) {
     {
         match fs::read_to_string(&path) {
             Ok(content) => {
+                // Check if the current active tab is empty/clean and can be replaced
                 let should_replace_current_tab = if let Some(active_tab) = app.get_active_tab() {
                     active_tab.file_path.is_none()
                         && active_tab.content.is_empty()
@@ -23,25 +24,22 @@ pub(crate) fn open_file(app: &mut TextEditor) {
                 };
 
                 if should_replace_current_tab {
+                    // Replace the current empty tab
                     if let Some(active_tab) = app.get_active_tab_mut() {
-                        let title = path
+                        active_tab.content = content;
+                        active_tab.file_path = Some(path.clone());
+                        active_tab.title = path
                             .file_name()
                             .and_then(|n| n.to_str())
-                            .unwrap_or("Untitled");
+                            .unwrap_or("Untitled")
-                        active_tab.content = content;
+                            .to_string();
-                        active_tab.file_path = Some(path.to_path_buf());
+                        active_tab.mark_as_saved(); // This will set the hash and mark as not modified
-                        active_tab.title = title.to_string();
-                        active_tab.mark_as_saved();
                     }
-                    app.text_needs_processing = true;
                 } else {
+                    // Create a new tab as before
                     let new_tab = Tab::new_with_file(content, path);
                     app.tabs.push(new_tab);
                     app.active_tab_index = app.tabs.len() - 1;
-                    app.text_needs_processing = true;
-                }
-                if app.show_find && !app.find_query.is_empty() {
-                    app.update_find_matches();
                 }
             }
             Err(err) => {
@@ -54,7 +52,7 @@ pub(crate) fn open_file(app: &mut TextEditor) {
 pub(crate) fn save_file(app: &mut TextEditor) {
     if let Some(active_tab) = app.get_active_tab() {
         if let Some(path) = &active_tab.file_path {
-            save_to_path(app, path.to_path_buf());
+            save_to_path(app, path.clone());
         } else {
             save_as_file(app);
         }
@@ -74,12 +72,12 @@ pub(crate) fn save_to_path(app: &mut TextEditor, path: PathBuf) {
     if let Some(active_tab) = app.get_active_tab_mut() {
         match fs::write(&path, &active_tab.content) {
             Ok(()) => {
-                let title = path
+                active_tab.file_path = Some(path.clone());
+                active_tab.title = path
                     .file_name()
                     .and_then(|n| n.to_str())
-                    .unwrap_or("Untitled");
+                    .unwrap_or("Untitled")
-                active_tab.file_path = Some(path.to_path_buf());
+                    .to_string();
-                active_tab.title = title.to_string();
                 active_tab.mark_as_saved();
             }
             Err(err) => {

src/ui/about_window.rs

@@ -11,8 +11,6 @@ pub(crate) fn about_window(app: &mut TextEditor, ctx: &egui::Context) {
     .collapsible(false)
     .resizable(false)
     .anchor(egui::Align2::CENTER_CENTER, egui::Vec2::ZERO)
-    .fade_in(true)
-    .fade_out(true)
     .frame(egui::Frame {
         fill: visuals.window_fill,
         stroke: visuals.window_stroke,

src/ui/central_panel.rs

@@ -4,7 +4,6 @@ mod line_numbers;
 
 use crate::app::TextEditor;
 use eframe::egui;
-use egui::UiKind;
 
 use self::editor::editor_view_ui;
 use self::line_numbers::{get_visual_line_mapping, render_line_numbers};
@@ -36,7 +35,7 @@ pub(crate) fn central_panel(app: &mut TextEditor, ctx: &egui::Context) {
                                 editor_view_ui(ui, app);
                             });
 
-                            handle_empty(ui, app, &context_response);
+                            show_context_menu(ui, app, &context_response);
                         });
                 return;
             }
@@ -76,7 +75,7 @@ pub(crate) fn central_panel(app: &mut TextEditor, ctx: &egui::Context) {
                 ui.add_space(3.0);
                 let separator_x = ui.cursor().left();
                 let mut y_range = ui.available_rect_before_wrap().y_range();
-                y_range.max += 2.0 * font_size;
+                y_range.max += 2.0 * font_size; // Extend separator to cover more vertical space
                 ui.painter()
                     .vline(separator_x, y_range, ui.visuals().window_stroke);
                 ui.add_space(4.0);
@@ -86,22 +85,26 @@ pub(crate) fn central_panel(app: &mut TextEditor, ctx: &egui::Context) {
                 .auto_shrink([false; 2])
                 .show(ui, |ui| {
                     if line_side {
+                        // Line numbers on the right
                         let text_editor_width =
                             editor_dimensions.text_width + editor_dimensions.total_reserved_width;
                         ui.allocate_ui_with_layout(
                             egui::vec2(text_editor_width, editor_height),
                             egui::Layout::left_to_right(egui::Align::TOP),
                             |ui| {
+                                // Constrain editor to specific width to leave space for line numbers
                                 ui.allocate_ui_with_layout(
                                     egui::vec2(editor_dimensions.text_width, editor_height),
                                     egui::Layout::left_to_right(egui::Align::TOP),
                                     |ui| {
+                                        // Create an invisible interaction area for context menu
                                         let full_rect = ui.available_rect_before_wrap();
                                         let context_response = ui.allocate_response(
                                             full_rect.size(),
                                             egui::Sense::click(),
                                         );
 
+                                        // Reset cursor to render editor at the top
                                         ui.scope_builder(
                                             egui::UiBuilder::new().max_rect(full_rect),
                                             |ui| {
@@ -109,7 +112,7 @@ pub(crate) fn central_panel(app: &mut TextEditor, ctx: &egui::Context) {
                                             },
                                         );
 
-                                        handle_empty(ui, app, &context_response);
+                                        show_context_menu(ui, app, &context_response);
                                     },
                                 );
                                 separator_widget(ui);
@@ -117,6 +120,7 @@ pub(crate) fn central_panel(app: &mut TextEditor, ctx: &egui::Context) {
                             },
                         );
                     } else {
+                        // Line numbers on the left
                         let text_editor_width =
                             editor_dimensions.text_width + editor_dimensions.total_reserved_width;
                         ui.allocate_ui_with_layout(
@@ -126,10 +130,12 @@ pub(crate) fn central_panel(app: &mut TextEditor, ctx: &egui::Context) {
                                 line_numbers_widget(ui);
                                 separator_widget(ui);
 
+                                // Create an invisible interaction area for context menu
                                 let editor_area = ui.available_rect_before_wrap();
                                 let context_response =
                                     ui.allocate_response(editor_area.size(), egui::Sense::click());
 
+                                // Reset cursor to render editor at the current position
                                 ui.scope_builder(
                                     egui::UiBuilder::new().max_rect(editor_area),
                                     |ui| {
@@ -137,7 +143,7 @@ pub(crate) fn central_panel(app: &mut TextEditor, ctx: &egui::Context) {
                                     },
                                 );
 
-                                handle_empty(ui, app, &context_response);
+                                show_context_menu(ui, app, &context_response);
                             },
                         );
                     }
@@ -145,25 +151,7 @@ pub(crate) fn central_panel(app: &mut TextEditor, ctx: &egui::Context) {
         });
 }
 
-fn handle_empty(_ui: &mut egui::Ui, app: &mut TextEditor, context_response: &egui::Response) {
+fn show_context_menu(_ui: &mut egui::Ui, app: &mut TextEditor, context_response: &egui::Response) {
-    if context_response.clicked() {
-        let text_edit_id = egui::Id::new("main_text_editor");
-        if let Some(mut state) = egui::TextEdit::load_state(_ui.ctx(), text_edit_id) {
-            if let Some(active_tab) = app.get_active_tab() {
-                let text_len = active_tab.content.len();
-                let cursor_pos = egui::text::CCursor::new(text_len);
-                state
-                    .cursor
-                    .set_char_range(Some(egui::text::CCursorRange::one(cursor_pos)));
-                egui::TextEdit::store_state(_ui.ctx(), text_edit_id, state);
-
-                _ui.ctx().memory_mut(|mem| {
-                    mem.request_focus(text_edit_id);
-                });
-            }
-        }
-    }
-
     context_response.context_menu(|ui| {
         let text_len = app.get_active_tab().unwrap().content.len();
         let reset_zoom_key = egui::Id::new("editor_reset_zoom");
@@ -171,17 +159,17 @@ fn handle_empty(_ui: &mut egui::Ui, app: &mut TextEditor, context_response: &egu
         if ui.button("Cut").clicked() {
             ui.ctx()
                 .send_viewport_cmd(egui::ViewportCommand::RequestCut);
-            ui.close_kind(UiKind::Menu);
+            ui.close_menu();
         }
         if ui.button("Copy").clicked() {
             ui.ctx()
                 .send_viewport_cmd(egui::ViewportCommand::RequestCopy);
-            ui.close_kind(UiKind::Menu);
+            ui.close_menu();
         }
         if ui.button("Paste").clicked() {
             ui.ctx()
                 .send_viewport_cmd(egui::ViewportCommand::RequestPaste);
-            ui.close_kind(UiKind::Menu);
+            ui.close_menu();
         }
         if ui.button("Delete").clicked() {
             ui.ctx().input_mut(|i| {
@@ -193,7 +181,7 @@ fn handle_empty(_ui: &mut egui::Ui, app: &mut TextEditor, context_response: &egu
                     modifiers: egui::Modifiers::NONE,
                 })
             });
-            ui.close_kind(UiKind::Menu);
+            ui.close_menu();
         }
         if ui.button("Select All").clicked() {
             let text_edit_id = egui::Id::new("main_text_editor");
@@ -205,14 +193,14 @@ fn handle_empty(_ui: &mut egui::Ui, app: &mut TextEditor, context_response: &egu
                 state.cursor.set_char_range(Some(select_all_range));
                 egui::TextEdit::store_state(ui.ctx(), text_edit_id, state);
             }
-            ui.close_kind(UiKind::Menu);
+            ui.close_menu();
         }
         ui.separator();
         if ui.button("Reset Zoom").clicked() {
             ui.ctx().memory_mut(|mem| {
                 mem.data.insert_temp(reset_zoom_key, true);
             });
-            ui.close_kind(UiKind::Menu);
+            ui.close_menu();
         }
     });
 }

src/ui/central_panel/editor.rs

@@ -1,8 +1,6 @@
 use crate::app::TextEditor;
 use eframe::egui;
 
-use super::find_highlight;
-
 pub(super) fn editor_view_ui(ui: &mut egui::Ui, app: &mut TextEditor) -> egui::Response {
     let _current_match_position = app.get_current_match_position();
     let show_find = app.show_find;
@@ -32,18 +30,6 @@ pub(super) fn editor_view_ui(ui: &mut egui::Ui, app: &mut TextEditor) -> egui::R
         0.0
     };
 
-    let find_data = if show_find && !app.find_matches.is_empty() {
-        app.get_active_tab().map(|tab| {
-            (
-                tab.content.to_owned(),
-                app.find_matches.to_owned(),
-                app.current_match_index,
-            )
-        })
-    } else {
-        None
-    };
-
     let Some(active_tab) = app.get_active_tab_mut() else {
         return ui.label("No file open, how did you get here?");
     };
@@ -52,44 +38,6 @@ pub(super) fn editor_view_ui(ui: &mut egui::Ui, app: &mut TextEditor) -> egui::R
     let editor_rect = ui.available_rect_before_wrap();
     ui.painter().rect_filled(editor_rect, 0.0, bg_color);
 
-    if let Some((content, matches, current_match_index)) = &find_data {
-        let font_id = ui
-            .style()
-            .text_styles
-            .get(&egui::TextStyle::Monospace)
-            .unwrap_or(&egui::FontId::monospace(font_size))
-            .to_owned();
-
-        let desired_width = if word_wrap {
-            ui.available_width()
-        } else {
-            f32::INFINITY
-        };
-
-        let temp_galley = ui.fonts(|fonts| {
-            fonts.layout(
-                content.to_owned(),
-                font_id.to_owned(),
-                ui.visuals().text_color(),
-                desired_width,
-            )
-        });
-
-        let text_area_left = editor_rect.left() + 4.0;
-        let text_area_top = editor_rect.top() + 2.0;
-
-        find_highlight::draw_find_highlights(
-            ui,
-            content,
-            matches,
-            *current_match_index,
-            &temp_galley,
-            text_area_left,
-            text_area_top,
-            font_size,
-        );
-    }
-
     let desired_width = if word_wrap {
         ui.available_width()
     } else {
@@ -102,7 +50,7 @@ pub(super) fn editor_view_ui(ui: &mut egui::Ui, app: &mut TextEditor) -> egui::R
         .code_editor()
         .desired_width(desired_width)
         .desired_rows(0)
-        .lock_focus(!show_find)
+        .lock_focus(true)
         .cursor_at_end(false)
         .id(egui::Id::new("main_text_editor"));
 
@@ -125,15 +73,11 @@ pub(super) fn editor_view_ui(ui: &mut egui::Ui, app: &mut TextEditor) -> egui::R
     let content_changed = output.response.changed();
     let content_for_processing = if content_changed {
         active_tab.update_modified_state();
-        Some(active_tab.content.to_owned())
+        Some(active_tab.content.clone())
     } else {
         None
     };
 
-    if content_changed && app.show_find && !app.find_query.is_empty() {
-        app.update_find_matches();
-    }
-
     let current_cursor_pos = output
         .state
         .cursor
@@ -141,7 +85,7 @@ pub(super) fn editor_view_ui(ui: &mut egui::Ui, app: &mut TextEditor) -> egui::R
         .map(|range| range.primary.index);
 
     if let Some(content) = content_for_processing {
-        let previous_content = app.previous_content.to_owned();
+        let previous_content = app.previous_content.clone();
         let previous_cursor_pos = app.previous_cursor_char_index;
 
         if !previous_content.is_empty() {
@@ -155,12 +99,19 @@ pub(super) fn editor_view_ui(ui: &mut egui::Ui, app: &mut TextEditor) -> egui::R
                     curr_cursor_pos,
                     ui,
                 );
+            } else {
+                app.process_text_for_rendering(&content, ui);
+
+                if let Some(cursor_pos) = current_cursor_pos {
+                    app.current_cursor_line = content[..cursor_pos]
+                        .bytes()
+                        .filter(|&b| b == b'\n')
+                        .count();
+                }
             }
-        } else {
-            app.process_text_for_rendering(&content, ui);
         }
 
-        app.previous_content = content.to_owned();
+        app.previous_content = content.clone();
         app.previous_cursor_char_index = current_cursor_pos;
 
         if let Some(active_tab) = app.get_active_tab_mut() {
@@ -169,13 +120,15 @@ pub(super) fn editor_view_ui(ui: &mut egui::Ui, app: &mut TextEditor) -> egui::R
         }
     }
 
-    if app.font_settings_changed || app.text_needs_processing {
+    // Check if font settings changed and trigger reprocessing
+    if app.font_settings_changed {
         if let Some(active_tab) = app.get_active_tab() {
-            let content = active_tab.content.to_owned();
+            let content = active_tab.content.clone();
-            app.process_text_for_rendering(&content, ui);
+            if !content.is_empty() {
+                app.process_text_for_rendering(&content, ui);
+            }
         }
         app.font_settings_changed = false;
-        app.text_needs_processing = false;
     }
 
     if !word_wrap {
@@ -197,7 +150,7 @@ pub(super) fn editor_view_ui(ui: &mut egui::Ui, app: &mut TextEditor) -> egui::R
                         .text_styles
                         .get(&egui::TextStyle::Monospace)
                         .unwrap_or(&egui::FontId::monospace(font_size))
-                        .to_owned();
+                        .clone();
                     let line_height = ui.fonts(|fonts| fonts.row_height(&font_id));
 
                     let y_pos = output.response.rect.top() + (cursor_line as f32 * line_height);
@@ -212,10 +165,12 @@ pub(super) fn editor_view_ui(ui: &mut egui::Ui, app: &mut TextEditor) -> egui::R
                     }
                 }
             }
+
             app.previous_cursor_position = Some(cursor_pos);
         }
     }
 
+    // Request focus if no dialogs are open
     if !output.response.has_focus()
         && !show_preferences
         && !show_about

src/ui/central_panel/find_highlight.rs

@@ -1,23 +1,11 @@
 use eframe::egui;
 
-/// Safely get a string slice up to a byte position, ensuring UTF-8 boundaries
+pub(super) fn _draw_find_highlight(
-fn safe_slice_to_pos(content: &str, pos: usize) -> &str {
-    let pos = pos.min(content.len());
-    let mut boundary_pos = pos;
-    while boundary_pos > 0 && !content.is_char_boundary(boundary_pos) {
-        boundary_pos -= 1;
-    }
-    &content[..boundary_pos]
-}
-
-pub(super) fn draw_find_highlights(
     ui: &mut egui::Ui,
     content: &str,
-    matches: &[(usize, usize)],
+    start_pos: usize,
-    current_match_index: Option<usize>,
+    end_pos: usize,
-    galley: &std::sync::Arc<egui::Galley>,
+    editor_rect: egui::Rect,
-    text_area_left: f32,
-    text_area_top: f32,
     font_size: f32,
 ) {
     let font_id = ui
@@ -25,47 +13,15 @@ pub(super) fn draw_find_highlights(
         .text_styles
         .get(&egui::TextStyle::Monospace)
         .unwrap_or(&egui::FontId::monospace(font_size))
-        .to_owned();
+        .clone();
 
-    for (match_index, &(start_pos, end_pos)) in matches.iter().enumerate() {
+    let text_up_to_start = &content[..start_pos.min(content.len())];
-        let is_current_match = current_match_index == Some(match_index);
-        draw_single_highlight(
-            ui,
-            content,
-            start_pos,
-            end_pos,
-            text_area_left,
-            text_area_top,
-            galley,
-            &font_id,
-            is_current_match,
-        );
-    }
-}
 
-fn draw_single_highlight(
-    ui: &mut egui::Ui,
-    content: &str,
-    start_pos: usize,
-    end_pos: usize,
-    text_area_left: f32,
-    text_area_top: f32,
-    galley: &std::sync::Arc<egui::Galley>,
-    font_id: &egui::FontId,
-    is_current_match: bool,
-) {
-    let text_up_to_start = safe_slice_to_pos(content, start_pos);
     let start_line = text_up_to_start.chars().filter(|&c| c == '\n').count();
 
-    if start_line >= galley.rows.len() {
-        return;
-    }
-
     let line_start_byte_pos = text_up_to_start.rfind('\n').map(|pos| pos + 1).unwrap_or(0);
-    let line_start_char_pos = safe_slice_to_pos(content, line_start_byte_pos)
+    let line_start_char_pos = content[..line_start_byte_pos].chars().count();
-        .chars()
+    let start_char_pos = content[..start_pos].chars().count();
-        .count();
-    let start_char_pos = safe_slice_to_pos(content, start_pos).chars().count();
     let start_col = start_char_pos - line_start_char_pos;
 
     let lines: Vec<&str> = content.lines().collect();
@@ -76,11 +32,18 @@ fn draw_single_highlight(
     let line_text = lines[start_line];
     let text_before_match: String = line_text.chars().take(start_col).collect();
 
+    let line_height = ui.fonts(|fonts| fonts.row_height(&font_id));
+
+    let horizontal_margin = ui.spacing().button_padding.x - 4.0;
+    let vertical_margin = ui.spacing().button_padding.y - 1.0;
+    let text_area_left = editor_rect.left() + horizontal_margin;
+    let text_area_top = editor_rect.top() + vertical_margin;
+
     let text_before_width = ui.fonts(|fonts| {
         fonts
             .layout(
                 text_before_match,
-                font_id.to_owned(),
+                font_id.clone(),
                 egui::Color32::WHITE,
                 f32::INFINITY,
             )
@@ -88,35 +51,30 @@ fn draw_single_highlight(
             .x
     });
 
-    let galley_row = &galley.rows[start_line];
+    let start_y = text_area_top + (start_line as f32 * line_height);
-    let start_y = text_area_top + galley_row.min_y();
-    let line_height = galley_row.height();
     let start_x = text_area_left + text_before_width;
 
-    let match_text = &content[start_pos..end_pos.min(content.len())];
+    {
-    let match_width = ui.fonts(|fonts| {
+        let match_text = &content[start_pos..end_pos.min(content.len())];
-        fonts
-            .layout(
-                match_text.to_string(),
-                font_id.to_owned(),
-                ui.visuals().text_color(),
-                f32::INFINITY,
-            )
-            .size()
-            .x
-    });
 
-    let highlight_rect = egui::Rect::from_min_size(
+        let match_width = ui.fonts(|fonts| {
-        egui::pos2(start_x, start_y),
+            fonts
-        egui::vec2(match_width, line_height),
+                .layout(
-    );
+                    match_text.to_string(),
+                    font_id.clone(),
+                    ui.visuals().text_color(),
+                    f32::INFINITY,
+                )
+                .size()
+                .x
+        });
 
-    let highlight_color = if is_current_match {
+        let highlight_rect = egui::Rect::from_min_size(
-        ui.visuals().selection.bg_fill
+            egui::pos2(start_x, start_y),
-    } else {
+            egui::vec2(match_width, line_height),
-        ui.visuals().selection.bg_fill.gamma_multiply(0.6)
+        );
-    };
 
-    let painter = ui.painter();
+        ui.painter()
-    painter.rect_filled(highlight_rect, 0.0, highlight_color);
+            .rect_filled(highlight_rect, 0.0, ui.visuals().selection.bg_fill);
+    }
 }

src/ui/central_panel/line_numbers.rs

@@ -29,7 +29,7 @@ pub(super) fn get_visual_line_mapping(
         cache
             .borrow()
             .as_ref()
-            .map(|(_, _, mapping)| mapping.to_owned())
+            .map(|(_, _, mapping)| mapping.clone())
             .unwrap_or_default()
     })
 }
@@ -52,7 +52,7 @@ fn calculate_visual_line_mapping(
         let galley = ui.fonts(|fonts| {
             fonts.layout(
                 line.to_string(),
-                font_id.to_owned(),
+                font_id.clone(),
                 egui::Color32::WHITE,
                 available_width,
             )
@@ -100,7 +100,7 @@ pub(super) fn render_line_numbers(
                 };
                 ui.label(
                     egui::RichText::new(text)
-                        .font(font_id.to_owned())
+                        .font(font_id.clone())
                         .color(text_color),
                 );
             }
@@ -109,7 +109,7 @@ pub(super) fn render_line_numbers(
                 let text = format!("{:>width$}", i, width = line_count_width);
                 ui.label(
                     egui::RichText::new(text)
-                        .font(font_id.to_owned())
+                        .font(font_id.clone())
                         .color(text_color),
                 );
             }

src/ui/find_window.rs

@@ -6,34 +6,12 @@ pub(crate) fn find_window(app: &mut TextEditor, ctx: &egui::Context) {
 
     let mut should_close = false;
     let mut query_changed = false;
-    let mut should_focus_editor = false;
 
-    let just_opened = app.show_find && !app.prev_show_find;
+    egui::Window::new("Find")
-
-    if just_opened && !app.find_query.is_empty() {
-        app.update_find_matches();
-        if app.current_match_index.is_some() {
-            app.select_current_match(ctx);
-            app.should_select_current_match = true;
-        }
-    }
-
-    let focus_requested = ctx.memory(|mem| {
-        mem.data
-            .get_temp::<bool>(egui::Id::new("focus_find_input"))
-            .unwrap_or(false)
-    });
-
-    let top_right_pos = egui::Pos2::new(ctx.available_rect().right(), 22.0);
-
-    egui::Window::new("")
         .collapsible(false)
         .resizable(false)
         .movable(true)
-        .title_bar(false)
+        .anchor(egui::Align2::CENTER_CENTER, egui::Vec2::ZERO)
-        .default_pos(top_right_pos)
-        .fade_in(true)
-        .fade_out(true)
         .frame(egui::Frame {
             fill: visuals.window_fill,
             stroke: visuals.window_stroke,
@@ -44,20 +22,13 @@ pub(crate) fn find_window(app: &mut TextEditor, ctx: &egui::Context) {
         })
         .show(ctx, |ui| {
             ui.vertical(|ui| {
-                ui.horizontal(|ui| {
+                ui.set_min_width(300.0);
-                    let arrow_text = if app.show_replace_section {
-                        "⏷"
-                    } else {
-                        "⏵"
-                    };
-                    if ui.button(arrow_text).clicked() {
-                        app.show_replace_section = !app.show_replace_section;
-                    }
 
+                ui.horizontal(|ui| {
                     ui.label("Find:");
                     let response = ui.add(
                         egui::TextEdit::singleline(&mut app.find_query)
-                            .desired_width(250.0)
+                            .desired_width(200.0)
                             .hint_text("Enter search text..."),
                     );
 
@@ -65,29 +36,16 @@ pub(crate) fn find_window(app: &mut TextEditor, ctx: &egui::Context) {
                         query_changed = true;
                     }
 
-                    if just_opened || focus_requested || app.focus_find {
+                    if !response.has_focus() {
                         response.request_focus();
-                        app.focus_find = false;
                     }
 
                     if response.lost_focus() && ui.input(|i| i.key_pressed(egui::Key::Enter)) {
-                        app.find_next(ctx);
+                        app.find_next();
                         response.request_focus();
                     }
                 });
 
-                if app.show_replace_section {
-                    ui.horizontal(|ui| {
-                        ui.add_space(4.0);
-                        ui.label("Replace:");
-                        let _replace_response = ui.add(
-                            egui::TextEdit::singleline(&mut app.replace_query)
-                                .desired_width(250.0)
-                                .hint_text("Enter replacement text..."),
-                        );
-                    });
-                }
-
                 ui.add_space(8.0);
 
                 ui.horizontal(|ui| {
@@ -97,24 +55,6 @@ pub(crate) fn find_window(app: &mut TextEditor, ctx: &egui::Context) {
                     if case_sensitive_changed {
                         query_changed = true;
                     }
-                    if app.show_replace_section {
-                        ui.add_space(8.0);
-
-                        let replace_current_enabled =
-                            !app.find_matches.is_empty() && app.current_match_index.is_some();
-                        ui.add_enabled_ui(replace_current_enabled, |ui| {
-                            if ui.button("Replace").clicked() {
-                                app.replace_current_match(ctx);
-                            }
-                        });
-
-                        let replace_all_enabled = !app.find_matches.is_empty();
-                        ui.add_enabled_ui(replace_all_enabled, |ui| {
-                            if ui.button("Replace All").clicked() {
-                                app.replace_all(ctx);
-                            }
-                        });
-                    }
                 });
 
                 ui.add_space(8.0);
@@ -135,7 +75,7 @@ pub(crate) fn find_window(app: &mut TextEditor, ctx: &egui::Context) {
                     ui.label(egui::RichText::new(match_text).weak());
 
                     ui.with_layout(egui::Layout::right_to_left(egui::Align::Center), |ui| {
-                        if ui.button("❌").clicked() {
+                        if ui.button("✕").clicked() {
                             should_close = true;
                         }
 
@@ -144,14 +84,14 @@ pub(crate) fn find_window(app: &mut TextEditor, ctx: &egui::Context) {
                         let next_enabled = !app.find_matches.is_empty();
                         ui.add_enabled_ui(next_enabled, |ui| {
                             if ui.button("Next").clicked() {
-                                app.find_next(ctx);
+                                app.find_next();
                             }
                         });
 
                         let prev_enabled = !app.find_matches.is_empty();
                         ui.add_enabled_ui(prev_enabled, |ui| {
                             if ui.button("Previous").clicked() {
-                                app.find_previous(ctx);
+                                app.find_previous();
                             }
                         });
                     });
@@ -161,28 +101,21 @@ pub(crate) fn find_window(app: &mut TextEditor, ctx: &egui::Context) {
 
     if query_changed {
         app.update_find_matches();
-        if app.current_match_index.is_some() {
-            app.select_current_match(ctx);
-            app.should_select_current_match = true;
-        }
     }
 
     if should_close {
-        app.select_current_match(ctx);
-        app.should_select_current_match = true;
         app.show_find = false;
     }
 
     ctx.input(|i| {
-        if i.key_pressed(egui::Key::Enter) && i.modifiers.ctrl && app.show_find {
+        if i.key_pressed(egui::Key::Escape) {
-            should_focus_editor = true;
+            app.show_find = false;
-            app.should_select_current_match = true;
+        } else if i.key_pressed(egui::Key::F3) {
+            if i.modifiers.shift {
+                app.find_previous();
+            } else {
+                app.find_next();
+            }
         }
     });
-
-    if should_focus_editor {
-        ctx.memory_mut(|mem| {
-            mem.request_focus(egui::Id::new("main_text_editor"));
-        });
-    }
 }

src/ui/menu_bar.rs

@@ -1,6 +1,5 @@
 use crate::{app::TextEditor, io};
 use eframe::egui::{self, Frame};
-use egui::UiKind;
 
 pub(crate) fn menu_bar(app: &mut TextEditor, ctx: &egui::Context) {
     let now = std::time::Instant::now();
@@ -44,34 +43,34 @@ pub(crate) fn menu_bar(app: &mut TextEditor, ctx: &egui::Context) {
                     }
                 }
 
-                egui::MenuBar::new().ui(ui, |ui| {
+                egui::menu::bar(ui, |ui| {
                     ui.menu_button("File", |ui| {
                         app.menu_interaction_active = true;
                         if ui.button("New").clicked() {
                             io::new_file(app);
-                            ui.close_kind(UiKind::Menu);
+                            ui.close_menu();
                         }
                         if ui.button("Open...").clicked() {
                             io::open_file(app);
-                            ui.close_kind(UiKind::Menu);
+                            ui.close_menu();
                         }
                         ui.separator();
                         if ui.button("Save").clicked() {
                             io::save_file(app);
-                            ui.close_kind(UiKind::Menu);
+                            ui.close_menu();
                         }
                         if ui.button("Save As...").clicked() {
                             io::save_as_file(app);
-                            ui.close_kind(UiKind::Menu);
+                            ui.close_menu();
                         }
                         ui.separator();
                         if ui.button("Preferences").clicked() {
                             app.show_preferences = true;
-                            ui.close_kind(UiKind::Menu);
+                            ui.close_menu();
                         }
                         if ui.button("Exit").clicked() {
                             app.request_quit(ctx);
-                            ui.close_kind(UiKind::Menu);
+                            ui.close_menu();
                         }
                     });
 
@@ -79,16 +78,16 @@ pub(crate) fn menu_bar(app: &mut TextEditor, ctx: &egui::Context) {
                         app.menu_interaction_active = true;
                         if ui.button("Cut").clicked() {
                             ui.ctx().input_mut(|i| i.events.push(egui::Event::Cut));
-                            ui.close_kind(UiKind::Menu);
+                            ui.close_menu();
                         }
                         if ui.button("Copy").clicked() {
                             ui.ctx().input_mut(|i| i.events.push(egui::Event::Copy));
-                            ui.close_kind(UiKind::Menu);
+                            ui.close_menu();
                         }
                         if ui.button("Paste").clicked() {
                             ui.ctx()
                                 .send_viewport_cmd(egui::ViewportCommand::RequestPaste);
-                            ui.close_kind(UiKind::Menu);
+                            ui.close_menu();
                         }
                         if ui.button("Delete").clicked() {
                             ui.ctx().input_mut(|i| {
@@ -100,7 +99,7 @@ pub(crate) fn menu_bar(app: &mut TextEditor, ctx: &egui::Context) {
                                     modifiers: egui::Modifiers::NONE,
                                 })
                             });
-                            ui.close_kind(UiKind::Menu);
+                            ui.close_menu();
                         }
                         if ui.button("Select All").clicked() {
                             let text_edit_id = egui::Id::new("main_text_editor");
@@ -117,7 +116,7 @@ pub(crate) fn menu_bar(app: &mut TextEditor, ctx: &egui::Context) {
                                     egui::TextEdit::store_state(ui.ctx(), text_edit_id, state);
                                 }
                             }
-                            ui.close_kind(UiKind::Menu);
+                            ui.close_menu();
                         }
                         ui.separator();
                         if ui.button("Undo").clicked() {
@@ -128,24 +127,21 @@ pub(crate) fn menu_bar(app: &mut TextEditor, ctx: &egui::Context) {
                                 if let Some(active_tab) = app.get_active_tab_mut() {
                                     let current_state = (
                                         state.cursor.char_range().unwrap_or_default(),
-                                        active_tab.content.to_string(),
+                                        active_tab.content.clone(),
                                     );
                                     let mut undoer = state.undoer();
                                     if let Some((cursor_range, content)) =
                                         undoer.undo(&current_state)
                                     {
-                                        active_tab.content = content.to_string();
+                                        active_tab.content = content.clone();
                                         state.cursor.set_char_range(Some(*cursor_range));
                                         state.set_undoer(undoer);
                                         egui::TextEdit::store_state(ui.ctx(), text_edit_id, state);
                                         active_tab.update_modified_state();
-                                        if app.show_find && !app.find_query.is_empty() {
-                                            app.update_find_matches();
-                                        }
                                     }
                                 }
                             }
-                            ui.close_kind(UiKind::Menu);
+                            ui.close_menu();
                         }
                         if ui.button("Redo").clicked() {
                             let text_edit_id = egui::Id::new("main_text_editor");
@@ -155,24 +151,21 @@ pub(crate) fn menu_bar(app: &mut TextEditor, ctx: &egui::Context) {
                                 if let Some(active_tab) = app.get_active_tab_mut() {
                                     let current_state = (
                                         state.cursor.char_range().unwrap_or_default(),
-                                        active_tab.content.to_string(),
+                                        active_tab.content.clone(),
                                     );
                                     let mut undoer = state.undoer();
                                     if let Some((cursor_range, content)) =
                                         undoer.redo(&current_state)
                                     {
-                                        active_tab.content = content.to_string();
+                                        active_tab.content = content.clone();
                                         state.cursor.set_char_range(Some(*cursor_range));
                                         state.set_undoer(undoer);
                                         egui::TextEdit::store_state(ui.ctx(), text_edit_id, state);
                                         active_tab.update_modified_state();
-                                        if app.show_find && !app.find_query.is_empty() {
-                                            app.update_find_matches();
-                                        }
                                     }
                                 }
                             }
-                            ui.close_kind(UiKind::Menu);
+                            ui.close_menu();
                         }
                     });
 
@@ -183,22 +176,25 @@ pub(crate) fn menu_bar(app: &mut TextEditor, ctx: &egui::Context) {
                             .clicked()
                         {
                             app.save_config();
-                            ui.close_kind(UiKind::Menu);
+                            ui.close_menu();
                         }
                         if ui.checkbox(&mut app.word_wrap, "Word Wrap").clicked() {
                             app.save_config();
-                            ui.close_kind(UiKind::Menu);
+                            ui.close_menu();
                         }
-                        if ui.checkbox(&mut app.hide_tab_bar, "Hide Tab Bar").clicked() {
+                        if ui
+                            .checkbox(&mut app.auto_hide_tab_bar, "Hide Tab Bar")
+                            .clicked()
+                        {
                             app.save_config();
-                            ui.close_kind(UiKind::Menu);
+                            ui.close_menu();
                         }
                         if ui
                             .checkbox(&mut app.auto_hide_toolbar, "Auto Hide Toolbar")
                             .clicked()
                         {
                             app.save_config();
-                            ui.close_kind(UiKind::Menu);
+                            ui.close_menu();
                         }
 
                         ui.separator();
@@ -206,7 +202,7 @@ pub(crate) fn menu_bar(app: &mut TextEditor, ctx: &egui::Context) {
                         if ui.button("Reset Zoom").clicked() {
                             app.zoom_factor = 1.0;
                             ctx.set_zoom_factor(1.0);
-                            ui.close_kind(UiKind::Menu);
+                            ui.close_menu();
                         }
 
                         ui.separator();
@@ -226,7 +222,7 @@ pub(crate) fn menu_bar(app: &mut TextEditor, ctx: &egui::Context) {
                                 if current_theme != crate::app::theme::Theme::System {
                                     app.set_theme(ctx);
                                 }
-                                ui.close_kind(UiKind::Menu);
+                                ui.close_menu();
                             }
                             if ui
                                 .radio_value(
@@ -239,7 +235,7 @@ pub(crate) fn menu_bar(app: &mut TextEditor, ctx: &egui::Context) {
                                 if current_theme != crate::app::theme::Theme::Light {
                                     app.set_theme(ctx);
                                 }
-                                ui.close_kind(UiKind::Menu);
+                                ui.close_menu();
                             }
                             if ui
                                 .radio_value(&mut app.theme, crate::app::theme::Theme::Dark, "Dark")
@@ -248,16 +244,16 @@ pub(crate) fn menu_bar(app: &mut TextEditor, ctx: &egui::Context) {
                                 if current_theme != crate::app::theme::Theme::Dark {
                                     app.set_theme(ctx);
                                 }
-                                ui.close_kind(UiKind::Menu);
+                                ui.close_menu();
                             }
                             ui.separator();
                             if ui.radio_value(&mut app.line_side, false, "Left").clicked() {
                                 app.save_config();
-                                ui.close_kind(UiKind::Menu);
+                                ui.close_menu();
                             }
                             if ui.radio_value(&mut app.line_side, true, "Right").clicked() {
                                 app.save_config();
-                                ui.close_kind(UiKind::Menu);
+                                ui.close_menu();
                             }
                         });
                     });
@@ -266,24 +262,24 @@ pub(crate) fn menu_bar(app: &mut TextEditor, ctx: &egui::Context) {
                         app.menu_interaction_active = true;
                         if ui.button("Shortcuts").clicked() {
                             app.show_shortcuts = true;
-                            ui.close_kind(UiKind::Menu);
+                            ui.close_menu();
                         }
                         if ui.button("About").clicked() {
                             app.show_about = true;
-                            ui.close_kind(UiKind::Menu);
+                            ui.close_menu();
                         }
                     });
 
-                    if app.hide_tab_bar {
+                    if app.auto_hide_tab_bar {
                         let tab_title = if let Some(tab) = app.get_active_tab() {
-                            tab.title.to_owned()
+                            tab.title.clone()
                         } else {
                             let empty_tab = crate::app::tab::Tab::new_empty(1);
-                            empty_tab.title.to_owned()
+                            empty_tab.title.clone()
                         };
 
                         let window_width = ctx.screen_rect().width();
-                        let font_id = ui.style().text_styles[&egui::TextStyle::Body].to_owned();
+                        let font_id = ui.style().text_styles[&egui::TextStyle::Body].clone();
 
                         let tab_title = if app.get_active_tab().is_some_and(|tab| tab.is_modified) {
                             format!("{tab_title}*")

src/ui/preferences_window.rs

@@ -14,8 +14,6 @@ pub(crate) fn preferences_window(app: &mut TextEditor, ctx: &egui::Context) {
         .anchor(egui::Align2::CENTER_CENTER, egui::Vec2::ZERO)
         .default_open(true)
         .max_size(max_size)
-        .fade_in(true)
-        .fade_out(true)
         .frame(egui::Frame {
             fill: visuals.window_fill,
             stroke: visuals.window_stroke,
@@ -78,7 +76,7 @@ pub(crate) fn preferences_window(app: &mut TextEditor, ctx: &egui::Context) {
                         .font_size_input
                         .as_ref()
                         .unwrap_or(&"14".to_string())
-                        .to_owned();
+                        .clone();
                     let response = ui.add(
                         egui::TextEdit::singleline(&mut font_size_text)
                             .desired_width(50.0)
@@ -86,7 +84,7 @@ pub(crate) fn preferences_window(app: &mut TextEditor, ctx: &egui::Context) {
                             .id(egui::Id::new("font_size_input")),
                     );
 
-                    app.font_size_input = Some(font_size_text.to_owned());
+                    app.font_size_input = Some(font_size_text.clone());
 
                     if response.clicked() {
                         response.request_focus();
@@ -132,19 +130,18 @@ pub(crate) fn preferences_window(app: &mut TextEditor, ctx: &egui::Context) {
                                     egui::RichText::new(
                                         "The quick brown fox jumps over the lazy dog.",
                                     )
-                                    .font(preview_font.to_owned()),
+                                    .font(preview_font.clone()),
                                 );
                                 ui.label(
                                     egui::RichText::new("ABCDEFGHIJKLMNOPQRSTUVWXYZ")
-                                        .font(preview_font.to_owned()),
+                                        .font(preview_font.clone()),
                                 );
                                 ui.label(
                                     egui::RichText::new("abcdefghijklmnopqrstuvwxyz")
-                                        .font(preview_font.to_owned()),
+                                        .font(preview_font.clone()),
                                 );
                                 ui.label(
-                                    egui::RichText::new("1234567890 !@#$%^&*()")
+                                    egui::RichText::new("1234567890 !@#$%^&*()").font(preview_font),
-                                        .font(preview_font.to_owned()),
                                 );
                             });
                     });

src/ui/shortcuts_window.rs

@@ -22,7 +22,6 @@ fn render_shortcuts_content(ui: &mut egui::Ui) {
         ui.label(egui::RichText::new("Ctrl + V: Paste").size(14.0));
         ui.label(egui::RichText::new("Ctrl + A: Select All").size(14.0));
         ui.label(egui::RichText::new("Ctrl + D: Delete Line").size(14.0));
-        ui.label(egui::RichText::new("Ctrl + F: Find").size(14.0));
 
         ui.add_space(16.0);
         ui.separator();
@@ -34,6 +33,7 @@ fn render_shortcuts_content(ui: &mut egui::Ui) {
         ui.label(egui::RichText::new("Ctrl + P: Preferences").size(14.0));
         ui.label(egui::RichText::new("Ctrl + =/-: Increase/Decrease Font Size").size(14.0));
         ui.label(egui::RichText::new("Ctrl + Shift + =/-: Zoom In/Out").size(14.0));
+
         // ui.label(
         //     egui::RichText::new("Ctrl + Shift + .: Toggle Vim Mode")
         //         .size(14.0)
@@ -42,8 +42,7 @@ fn render_shortcuts_content(ui: &mut egui::Ui) {
         //     egui::RichText::new("Ctrl + .: Toggle Vim Mode")
         //         .size(14.0)
         // );
-        ui.add_space(16.0);
+        ui.add_space(12.0);
-        ui.separator();
     });
 }
 
@@ -51,6 +50,7 @@ pub(crate) fn shortcuts_window(app: &mut TextEditor, ctx: &egui::Context) {
     let visuals = &ctx.style().visuals;
     let screen_rect = ctx.screen_rect();
 
+    // Calculate appropriate window size that always fits nicely in the main window
     let window_width = (screen_rect.width() * 0.6).clamp(300.0, 400.0);
     let window_height = (screen_rect.height() * 0.7).clamp(250.0, 500.0);
 
@@ -59,8 +59,6 @@ pub(crate) fn shortcuts_window(app: &mut TextEditor, ctx: &egui::Context) {
         .resizable(false)
         .anchor(egui::Align2::CENTER_CENTER, egui::Vec2::ZERO)
         .fixed_size([window_width, window_height])
-        .fade_in(true)
-        .fade_out(true)
         .frame(egui::Frame {
             fill: visuals.window_fill,
             stroke: visuals.window_stroke,
@@ -71,7 +69,8 @@ pub(crate) fn shortcuts_window(app: &mut TextEditor, ctx: &egui::Context) {
         })
         .show(ctx, |ui| {
             ui.vertical(|ui| {
-                let available_height = ui.available_height() - 40.0;
+                // Scrollable content area
+                let available_height = ui.available_height() - 40.0; // Reserve space for close button
                 ui.allocate_ui_with_layout(
                     [ui.available_width(), available_height].into(),
                     egui::Layout::top_down(egui::Align::Center),
@@ -84,6 +83,7 @@ pub(crate) fn shortcuts_window(app: &mut TextEditor, ctx: &egui::Context) {
                     },
                 );
 
+                // Fixed close button at bottom
                 ui.vertical_centered(|ui| {
                     ui.add_space(8.0);
                     let visuals = ui.visuals();