refactor: share grid consts across crates; fix stale docs

Author: tamirelazar

src/app/mod.rs

@@ -27,7 +27,7 @@ use crate::render::palette::{
 use crate::render::palette_editor::{
     EditorComponent, EditorMode, PaletteEditorOverlay, PaletteEditorState,
 };
-use crate::render::window::{FRAME_RING_COLS, FRAME_RING_ROWS};
+use crate::render::window::{FRAME_RING_COLS, FRAME_RING_ROWS, GRID_COLOR, GRID_OPACITY};
 use crate::simulation;
 use crate::simulation::config::{
     Attractor, DiffusionKernel, InitMode, Preset, SimConfig, TerrainType,
@@ -697,15 +697,6 @@ pub fn run() -> io::Result<()> {
     Ok(())
 }
 
-/// Grid overlay color for the framed ANSI background — matches the wasm's
-/// `render_ansi_frame` look (`tslime-wasm/src/lib.rs`).
-const HEADLESS_GRID_COLOR: RgbColor = RgbColor {
-    r: 0x8f,
-    g: 0x8f,
-    b: 0x55,
-};
-const HEADLESS_GRID_OPACITY: f32 = 0.35;
-
 /// Renders the exact ANSI frame the vendored wasm's `render_ansi_frame`
 /// produces for a `cols`×`rows` terminal, from fresh isolated state.
 ///
@@ -740,13 +731,7 @@ pub fn headless_ansi_frame(
     // Matches the wasm's default render brightness (`TslimeWasm::new`).
     let brightness: f32 = 2.2;
 
-    let mut grid = GridRenderer::new(
-        GridStyle::Cross,
-        5,
-        HEADLESS_GRID_COLOR,
-        HEADLESS_GRID_OPACITY,
-        false,
-    );
+    let mut grid = GridRenderer::new(GridStyle::Cross, 5, GRID_COLOR, GRID_OPACITY, false);
     grid.initialize(iw, ih);
 
     let mut frame = DownsampledFrame::new(iw, ih);
@@ -771,8 +756,8 @@ pub fn headless_ansi_frame(
             Charset::Ascii,
             gain,
             Some(&grid),
-            HEADLESS_GRID_COLOR,
-            HEADLESS_GRID_OPACITY,
+            GRID_COLOR,
+            GRID_OPACITY,
             Some(accent),
         );
     }

src/render/ansi.rs

@@ -133,8 +133,9 @@ fn ascii_cell_fg_glyph(
     (fg, glyph)
 }
 
-/// Full-terminal frame geometry: an outer ring (glow, added in Task 2) around
-/// an interior field of cells.
+/// Full-terminal frame geometry: an outer ring around an interior field of
+/// cells. The ring is drawn as a glow when a glow accent is supplied to
+/// [`render_ansi_framed`], or left blank otherwise.
 pub struct FrameGeometry {
     /// Full terminal columns.
     pub cols: usize,
@@ -157,8 +158,8 @@ impl FrameGeometry {
 }
 
 /// Render a full terminal frame from interior-sized field cells, with an
-/// optional grid overlay on the interior. The outer ring (glow, added in
-/// Task 2) is emitted as blank space for now.
+/// optional grid overlay on the interior. The outer ring is drawn as a glow
+/// when `glow_accent` is `Some(_)`, or emitted as blank space when `None`.
 ///
 /// * `field_cells` — the FIELD downsampled to interior dims (`geom.interior()`).
 /// * `grid` — pre-initialized to interior dims, or `None` for no grid.

src/render/window.rs

@@ -4,6 +4,7 @@
 //! the simulation window inside the terminal, given an aspect ratio, padding
 //! settings, and fallback thresholds. No I/O is performed here.
 
+use crate::render::palette::RgbColor;
 use crate::simulation::config::{Aspect, TerminalSizeThreshold, WindowPadding};
 
 /// Default frame ring thickness in **columns** on each of the left/right sides.
@@ -20,6 +21,17 @@ pub const FRAME_RING_COLS: usize = crate::config_defaults::frame_matte::DEFAULT_
 /// See [`FRAME_RING_COLS`].
 pub const FRAME_RING_ROWS: usize = crate::config_defaults::frame_matte::DEFAULT_ROWS + 1;
 
+/// Shared grid overlay color for the framed ANSI/WebGL background, used by both
+/// the native headless renderer and the wasm renderer so they stay in sync.
+pub const GRID_COLOR: RgbColor = RgbColor {
+    r: 0x8f,
+    g: 0x8f,
+    b: 0x55,
+};
+
+/// Shared grid overlay opacity. See [`GRID_COLOR`].
+pub const GRID_OPACITY: f32 = 0.35;
+
 /// How the window layout was resolved when terminal space was limited.
 pub enum FallbackMode {
     /// Normal layout with padding and centered frame.

tslime-wasm/src/lib.rs

@@ -6,10 +6,8 @@ use tslime::render::ansi::{render_ansi_cells, render_ansi_framed, FrameGeometry}
 use tslime::render::charset::Charset;
 use tslime::render::downsample::{downsample, DownsampledFrame};
 use tslime::render::grid::{GridRenderer, GridStyle};
-use tslime::render::palette::{
-    palette_accent_color, IntensityMapping, Palette, RgbColor, ALL_PALETTES,
-};
-use tslime::render::window::{FRAME_RING_COLS, FRAME_RING_ROWS};
+use tslime::render::palette::{palette_accent_color, IntensityMapping, Palette, ALL_PALETTES};
+use tslime::render::window::{FRAME_RING_COLS, FRAME_RING_ROWS, GRID_COLOR, GRID_OPACITY};
 use tslime::simulation::{
     config::{InitMode, SimConfig, PRESETS},
     Simulation,
@@ -18,14 +16,6 @@ use wasm_bindgen::prelude::*;
 
 mod renderer;
 
-/// Grid overlay color for the framed ANSI background — matches the info look.
-const GRID_COLOR: RgbColor = RgbColor {
-    r: 0x8f,
-    g: 0x8f,
-    b: 0x55,
-};
-const GRID_OPACITY: f32 = 0.35;
-
 #[wasm_bindgen]
 pub struct TslimeWasm {
     simulation: Simulation,
@@ -121,8 +111,9 @@ impl TslimeWasm {
     /// (downsampled to the interior, i.e. `cols`×`rows` minus the frame ring)
     /// plus a grid overlay and a glow ring — matching the info look. The white
     /// point is tracked adaptively across frames (matching the TUI's
-    /// auto-normalize), so no manual gain is needed. Independent of WebGL —
-    /// works in headless mode.
+    /// auto-normalize); the resulting gain divides that adaptive white point
+    /// by the `brightness` knob (`gain = get_max_brightness() /
+    /// brightness.max(0.05)`). Independent of WebGL — works in headless mode.
     ///
     /// `render_ansi_framed` only supports the ASCII charset (it asserts this);
     /// in half-block mode this falls back to the original unframed render.