chore: refactor app layout computations to take them out of the hot loop (#996)

* chore: refactor app layout computations to take them out of the hot loop

* chore: use Default for default theme

* fix: recursion loop on default theme

Author: LoricAndre

src/theme.rs

@@ -1,13 +1,8 @@
 //! Handle the color theme
-use std::sync::LazyLock;
-
 use ratatui::style::{Color, Modifier, Style};
 
 use crate::options::SkimOptions;
 
-/// Theme defaults to Dark256
-pub static DEFAULT_THEME: LazyLock<ColorTheme> = LazyLock::new(ColorTheme::dark256);
-
 /// The color scheme of skim's UI
 ///
 /// <pre>
@@ -19,7 +14,7 @@ pub static DEFAULT_THEME: LazyLock<ColorTheme> = LazyLock::new(ColorTheme::dark2
 /// |> query         |  --> prompt & query
 /// +----------------+
 /// </pre>
-#[derive(Copy, Clone, Debug, Default)]
+#[derive(Copy, Clone, Debug)]
 pub struct ColorTheme {
     /// Non-selected lines and general text
     pub normal: Style,
@@ -47,6 +42,13 @@ pub struct ColorTheme {
     pub border: Style,
 }
 
+impl Default for ColorTheme {
+    /// Theme defaults to Dark256
+    fn default() -> Self {
+        ColorTheme::dark256()
+    }
+}
+
 #[allow(dead_code)]
 impl ColorTheme {
     /// Setup the theme from the skim options
@@ -64,9 +66,20 @@ impl ColorTheme {
     }
 
     fn none() -> Self {
+        let def = Style::default();
         Self {
             spinner: Style::default().bold(),
-            ..ColorTheme::default()
+            normal: def,
+            matched: def,
+            current: def,
+            current_match: def,
+            query: def,
+            info: def,
+            prompt: def,
+            cursor: def,
+            selected: def,
+            header: def,
+            border: def,
         }
     }