Use StringTree to store the rendered layout

And print the entire layout at once

src/musicplayer/control.gleam

@@ -1,51 +1,30 @@
-import gleam/string
-
 import musicplayer/input/key.{type Key}
 
-pub type Mode {
-  Idle
-  Searching(input: String)
-}
-
 pub type Control {
   TogglePlayPause
-  Search(input: String, capturing: Bool)
+  Search
+
+  Raw(String)
+  Return
+  Backspace
 
   Exit
 }
 
-pub fn from_key(key: Key, mode: Mode) -> Result(Control, Nil) {
+pub fn from_key(key: Key) -> Result(Control, Nil) {
-  case mode {
+  case key {
-    Idle -> idle_from_key(key)
+    key.Return -> Ok(Return)
-    Searching(input) -> searching_from_key(key, input)
+    key.Backspace -> Ok(Backspace)
+    key.Char(char) -> Ok(char_control(char))
+    _ -> Error(Nil)
   }
 }
 
-pub fn idle_from_key(key: Key) -> Result(Control, Nil) {
+fn char_control(char: String) -> Control {
-  case key {
-    key.Char(char) -> {
   case char {
-        " " -> Ok(TogglePlayPause)
+    " " -> TogglePlayPause
-        "/" -> Ok(Search(input: "", capturing: True))
+    "/" -> Search
-        "q" -> Ok(Exit)
+    "q" -> Exit
-
+    _ -> Raw(char)
-        // NOOP
-        _ -> Error(Nil)
-      }
-    }
-
-    // NOOP
-    _ -> Error(Nil)
-  }
-}
-
-pub fn searching_from_key(key: Key, input: String) -> Result(Control, Nil) {
-  case key {
-    key.Char(char) -> Ok(Search(input <> char, True))
-    key.Backspace -> Ok(Search(string.drop_end(input, 1), True))
-    key.Return -> Ok(Search(input, False))
-
-    // NOOP
-    _ -> Error(Nil)
   }
 }

src/musicplayer/musicplayer.gleam

@@ -1,8 +1,9 @@
 import gleam/erlang/process.{type Name, type Pid, type Subject}
 import gleam/otp/actor
+import gleam/result
 import gleam/string
 
-import musicplayer/control.{type Mode}
+import musicplayer/control.{type Control}
 import musicplayer/input/key.{type Key}
 import musicplayer/logging/logging
 import musicplayer/mpv/control as mpv_control
@@ -10,9 +11,19 @@ import musicplayer/time/time
 import musicplayer/ui/control as ui_control
 import musicplayer/ui/layout
 
+type Mode {
+  Idle
+  Searching
+}
+
+type Input {
+  Input(capturing: Bool, content: String)
+}
+
 type State {
   State(
     mode: Mode,
+    input: Input,
     ui: Subject(ui_control.Control),
     mpv: Subject(mpv_control.Control),
   )
@@ -25,8 +36,10 @@ pub fn new(
 ) -> Result(Pid, String) {
   let input_keys = process.named_subject(input_keys_name)
 
+  let input = Input(False, "")
+
   case
-    actor.new(State(control.Idle, ui, mpv))
+    actor.new(State(Idle, input, ui, mpv))
     |> actor.on_message(handle_message)
     |> actor.start
   {
@@ -36,44 +49,75 @@ pub fn new(
       logging.log("musicplayer - started")
       process.spawn(fn() {
         let assert Ok(_) = process.register(process.self(), input_keys_name)
-        forward_key(musicplayer, input_keys)
+        handle_key(musicplayer, input_keys)
       })
 
-      process.spawn(fn() { update_playback_time_loop(mpv, ui, 250) })
+      process.spawn(fn() { update_playback_time_loop(mpv, ui, 1000) })
 
       Ok(pid)
     }
   }
 }
 
-fn handle_message(state: State, key: Key) -> actor.Next(State, Key) {
+fn handle_message(state: State, control: Control) -> actor.Next(State, Control) {
-  case control.from_key(key, state.mode) {
+  case control {
-    Error(_) -> actor.continue(state)
+    control.Search -> {
-    Ok(c) ->
+      logging.log("musicplayer - initiating search")
-      case c {
-        control.Search(input, capturing) -> {
-          case capturing {
-            True -> {
-              logging.log("musicplayer - searching: " <> input)
 
-              update_search(state.ui, "searching: " <> input)
+      update_search(state.ui, "searching: ")
 
-              actor.continue(State(..state, mode: control.Searching(input)))
+      actor.continue(
+        State(
+          ..state,
+          mode: Searching,
+          input: Input(..state.input, capturing: True),
+        ),
+      )
     }
-            False -> {
+
+    control.Raw(content) -> {
+      logging.log("musicplayer - recieved raw input: " <> content)
+
+      let content = case state.mode {
+        Idle -> state.input.content
+        Searching -> {
+          let updated = state.input.content <> content
+          update_search(state.ui, "searching: " <> updated)
+          updated
+        }
+      }
+
+      actor.continue(State(..state, input: Input(..state.input, content:)))
+    }
+    control.Backspace -> {
+      logging.log("musicplayer - recieved backspace")
+
+      let content = case state.mode {
+        Idle -> state.input.content
+        Searching -> string.drop_end(state.input.content, 1)
+      }
+      actor.continue(State(..state, input: Input(..state.input, content:)))
+    }
+    control.Return -> {
       logging.log(
-                "musicplayer - recieved return. `input`: "
+        "musicplayer - recieved return. `input.capture`: "
         <> "'"
-                <> input
+        <> state.input.content
         <> "'",
       )
 
-              update_search(state.ui, "")
+      // Note: state.input.content is now the final input, use it
+      // before it is reset
+      case state.mode {
+        Idle -> Nil
+        Searching -> update_search(state.ui, "")
+      }
 
-              actor.continue(State(..state, mode: control.Idle))
+      actor.continue(
-            }
+        State(..state, mode: Idle, input: Input(capturing: False, content: "")),
-          }
+      )
     }
+
     control.TogglePlayPause -> {
       logging.log("musicplayer - toggling play/pause")
 
@@ -84,21 +128,16 @@ fn handle_message(state: State, key: Key) -> actor.Next(State, Key) {
     control.Exit -> {
       logging.log("musicplayer - initiating musicplayer shutdown")
       // Close `mpv` socket
-          process.call(state.mpv, 1000, fn(reply_to) {
+      process.call(state.mpv, 1000, fn(reply_to) { mpv_control.Exit(reply_to) })
-            mpv_control.Exit(reply_to)
-          })
 
       // Reset terminal state (show cursor etc.)
-          process.call(state.ui, 1000, fn(reply_to) {
+      process.call(state.ui, 1000, fn(reply_to) { ui_control.Exit(reply_to) })
-            ui_control.Exit(reply_to)
-          })
 
       logging.log("musicplayer - stopped")
 
       actor.stop()
     }
   }
-  }
 }
 
 fn update_playback_time_loop(
@@ -107,7 +146,6 @@ fn update_playback_time_loop(
   interval_ms: Int,
 ) {
   process.sleep(interval_ms)
-  // TODO only update if state is playing
   update_playback_time(mpv, ui)
 
   update_playback_time_loop(mpv, ui, interval_ms)
@@ -146,14 +184,14 @@ fn update_search(ui: Subject(ui_control.Control), content: String) -> Nil {
   process.send(ui, ui_control.UpdateState(layout.Search, content))
 }
 
-/// `forward_key` listens to a subject onto which `input` will send messages with `Key`s
+/// `handle_key` listens to a subject onto which `input` will send messages with `Key`s
-/// that is then forwarded to the `musicplayer` agent to handle
+fn handle_key(musicplayer: Subject(Control), input_keys: Subject(Key)) -> Nil {
-fn forward_key(musicplayer: Subject(Key), input_keys: Subject(Key)) -> Nil {
   let _ =
     process.new_selector()
     |> process.select(input_keys)
     |> process.selector_receive_forever
-    |> process.send(musicplayer, _)
+    |> control.from_key
+    |> result.map(process.send(musicplayer, _))
 
-  forward_key(musicplayer, input_keys)
+  handle_key(musicplayer, input_keys)
 }

src/musicplayer/ui/control.gleam

@@ -3,9 +3,8 @@ import gleam/erlang/process.{type Subject}
 import musicplayer/ui/layout.{type Section}
 
 pub type Control {
-  UpdateDimensions(columns: Int, rows: Int)
+  UpdateDimensions(width: Int, height: Int)
   UpdateState(section: Section, content: String)
-  SetView(view_idx: layout.ViewIdx)
 
   Exit(reply_to: Subject(Nil))
 }

src/musicplayer/ui/internal.gleam

@@ -3,16 +3,6 @@ import gleam/io
 
 // https://en.wikipedia.org/wiki/ANSI_escape_code#Control_Sequence_Introducer_commands
 
-pub const move_to_home = "\u{001B}[H"
-
-pub const disable_auto_wrap = "\u{001B}[?7l"
-
-pub const enable_auto_wrap = "\u{001B}[?7h"
-
-pub fn update(frame: String) -> Nil {
-  io.print(disable_auto_wrap <> move_to_home <> frame <> enable_auto_wrap)
-}
-
 pub fn clear_screen() -> Nil {
   io.print("\u{001B}[2J\u{001B}[H")
 }
@@ -22,6 +12,10 @@ pub fn chars_at(chars: String, x: Int, y: Int) -> String {
   seq <> chars
 }
 
+pub fn print(chars: String) -> Nil {
+  io.print(chars)
+}
+
 pub fn hide_cursor() -> Nil {
   io.print("\u{001B}[?25l")
 }

src/musicplayer/ui/layout.gleam

@@ -1,95 +1,41 @@
 import gleam/dict
 import gleam/list
-import gleam/pair
+import gleam/string_tree.{type StringTree}
-import gleam/set
-import gleam/string
 
 import musicplayer/ui/internal
-import musicplayer/ui/plot.{type Buffer}
 
-pub const root_section = "reserved_root_section"
+pub type Layout {
+  Layout(width: Int, height: Int, nodes: dict.Dict(Section, Node))
+}
 
 pub type Section {
-  Section(String)
+  Root
-
   Header
   Search
   PlaybackTime
 }
 
-pub type Dimension {
-  Percent(width: Int, height: Int)
-  // TODO add Flex that flows
-}
-
-pub type Style {
-  Style(dimensions: Dimension)
-}
-
 pub type Node {
-  Row(content: String, style: Style, children: List(Section))
+  Node(content: String, x: Int, y: Int, children: List(Section))
-  Cell(content: String, style: Style)
 }
 
-pub type ViewIdx =
+pub fn new() -> Layout {
-  Int
+  let nodes =
-
+    dict.from_list([
-pub type View =
+      #(
-  dict.Dict(Section, Node)
+        Root,
-
+        Node(content: "", x: 0, y: 0, children: [
-// Layout consists of a list Views, and only one View is rendered at a time
+          Header,
-pub type Layout {
+          Search,
-  Layout(
+          PlaybackTime,
-    columns: Int,
+        ]),
-    rows: Int,
-    current_view: ViewIdx,
-    views: dict.Dict(ViewIdx, View),
-  )
-}
-
-pub fn new(
-  columns: Int,
-  rows: Int,
-  views: List(List(#(Section, Node))),
-) -> Layout {
-  let views =
-    list.index_map(views, fn(view_nodes, i) { #(i, view_nodes) })
-    |> list.fold(dict.new(), fn(view_acc, iv) {
-      let #(i, view_nodes) = iv
-
-      dict.insert(view_acc, i, view_loop(i, view_nodes))
-    })
-
-  Layout(columns:, rows:, current_view: 0, views:)
-}
-
-fn view_loop(i: ViewIdx, view_nodes: List(#(Section, Node))) -> View {
-  let children =
-    view_nodes
-    |> list.flat_map(fn(node) {
-      case pair.second(node) {
-        Row(children: c, ..) -> c
-        Cell(..) -> []
-      }
-    })
-    |> set.from_list
-
-  // All sections that are not children of other nodes will be added as
-  // children to the root
-  let orphans =
-    view_nodes
-    |> list.map(pair.first)
-    |> list.filter(fn(node) { !set.contains(children, node) })
-
-  dict.from_list(view_nodes)
-  |> dict.insert(
-    Section(string.append("view_", string.inspect(i))),
-    Row(
-      content: "",
-      style: Style(dimensions: Percent(width: 100, height: 100)),
-      children: orphans,
       ),
-  )
+      #(Header, Node(content: "Music Player", x: 1, y: 1, children: [])),
+      #(Search, Node(content: "", x: 30, y: 1, children: [])),
+      #(PlaybackTime, Node(content: "00:00", x: 1, y: 2, children: [])),
+    ])
+
+  Layout(0, 0, nodes: nodes)
 }
 
 pub fn update_section(
@@ -97,147 +43,48 @@ pub fn update_section(
   section: Section,
   content: String,
 ) -> Layout {
-  case dict.get(layout.views, layout.current_view) {
+  case dict.get(layout.nodes, section) {
     Error(_) -> layout
-    Ok(view) ->
+    Ok(node) ->
-      case dict.get(view, section) {
-        Error(_) -> layout
-        Ok(node) -> {
-          let updated_node = case node {
-            Cell(..) -> Cell(..node, content: content)
-            Row(..) -> Row(..node, content: content)
-          }
-
-          let updated_view = dict.insert(view, section, updated_node)
-
       Layout(
         ..layout,
-            views: dict.insert(layout.views, layout.current_view, updated_view),
+        nodes: dict.insert(
+          layout.nodes,
+          section,
+          Node(..node, content: content),
+        ),
       )
   }
-      }
-  }
 }
 
-pub fn update_dimensions(layout: Layout, columns: Int, rows: Int) -> Layout {
+pub fn update_dimensions(layout: Layout, width: Int, height: Int) -> Layout {
-  Layout(..layout, columns:, rows:)
+  Layout(..layout, width:, height:)
 }
 
-pub fn set_view(layout: Layout, view_idx: ViewIdx) -> Layout {
+pub fn render(layout: Layout, from: Section) -> Nil {
-  Layout(..layout, current_view: view_idx)
+  string_tree.new()
-}
+  |> render_loop(layout, from, _)
-
+  |> string_tree.to_string
-pub fn render(layout: Layout) -> Nil {
+  |> internal.print
-  let context =
-    RenderContext(
-      parent_width: layout.columns,
-      parent_height: layout.rows,
-      parent_top_left_x: 1,
-      parent_top_left_y: 1,
-      position_index: 0,
-    )
-
-  case dict.get(layout.views, layout.current_view) {
-    Error(_) -> Nil
-    Ok(view) -> {
-      let buffer: Buffer = dict.new()
-
-      render_loop(
-        view,
-        context,
-        // TODO extract to function `view_index`
-        Section(string.append("view_", string.inspect(layout.current_view))),
-        buffer,
-      )
-      |> plot.flush_buffer(layout.columns, layout.rows)
-      |> internal.update
-    }
-  }
-}
-
-pub type RenderContext {
-  RenderContext(
-    parent_width: Int,
-    parent_height: Int,
-    parent_top_left_x: Int,
-    parent_top_left_y: Int,
-    position_index: Int,
-  )
 }
 
 pub fn render_loop(
-  view: View,
+  layout: Layout,
-  context: RenderContext,
   from: Section,
-  buffer: Buffer,
+  into: StringTree,
-) -> Buffer {
+) -> StringTree {
-  case dict.get(view, from) {
+  case dict.get(layout.nodes, from) {
-    Error(_) -> buffer
+    Error(_) -> into
     Ok(node) -> {
-      // Margin between container and the node being rendered
-      let margin = 2
-
-      let #(node_width, node_height) = case node.style.dimensions {
-        Percent(width:, height:) -> {
-          let width = { context.parent_width * width } / 100
-          let height = { context.parent_height * height } / 100
-
-          #(width, height)
-        }
-      }
-
-      // Check if this node should be placed to the left or below the parent
-      let #(node_top_left_x, node_top_left_y) = case node {
-        Row(..) -> #(
-          context.parent_top_left_x,
-          context.parent_top_left_y + { context.position_index * node_height },
-        )
-        Cell(..) -> #(
-          context.parent_top_left_x + { context.position_index * node_width },
-          context.parent_top_left_y,
-        )
-      }
-
       let parent =
-        plot.box(
+        string_tree.append(
-          buffer,
+          into,
-          node_top_left_x,
+          internal.chars_at(node.content, node.x, node.y),
-          node_top_left_y,
-          node_width,
-          node_height,
-        )
-        |> plot.text(node.content, node_top_left_x, node_top_left_y)
-
-      case node {
-        Cell(..) -> parent
-        Row(children:, ..) -> {
-          list.index_map(children, fn(child, i) { #(i, child) })
-          |> list.fold(parent, fn(acc_buffer, ic) {
-            let #(i, child) = ic
-
-            let #(child_width, child_height) = case node.style.dimensions {
-              Percent(width:, height:) -> {
-                let width = { { context.parent_width * width } / 100 } - margin
-                let height =
-                  { { context.parent_height * height } / 100 } - margin
-
-                #(width, height)
-              }
-            }
-
-            let context =
-              RenderContext(
-                parent_width: child_width,
-                parent_height: child_height,
-                parent_top_left_x: context.parent_top_left_x + 1,
-                parent_top_left_y: context.parent_top_left_y + 1,
-                position_index: i,
         )
 
-            render_loop(view, context, child, acc_buffer)
+      list.fold(node.children, parent, fn(into_acc, child) {
+        render_loop(layout, child, into_acc)
       })
     }
   }
-    }
-  }
 }

src/musicplayer/ui/layout_examples/layout_examples.gleam

@@ -1,58 +0,0 @@
-import musicplayer/ui/internal
-import musicplayer/ui/layout.{Percent, Section, Style}
-import musicplayer/ui/layout_examples/wait_for_input.{wait_for_input}
-
-pub fn main() {
-  let assert Ok(columns) = internal.io_get_columns()
-  let assert Ok(rows) = internal.io_get_rows()
-
-  two_rows_with_cells(columns, rows)
-  |> layout.render
-
-  wait_for_input()
-}
-
-/// Two rows:
-/// First row has two cells
-/// Second row has no cells
-fn two_rows_with_cells(columns: Int, rows: Int) -> layout.Layout {
-  let views = [
-    [
-      #(
-        Section("Row1"),
-        layout.Row(
-          content: "row 1",
-          style: Style(dimensions: Percent(width: 100, height: 50)),
-          children: [
-            Section("A"),
-            Section("B"),
-          ],
-        ),
-      ),
-      #(
-        Section("A"),
-        layout.Cell(
-          content: "cell 1",
-          style: Style(dimensions: Percent(width: 50, height: 50)),
-        ),
-      ),
-      #(
-        Section("B"),
-        layout.Cell(
-          content: "cell 2",
-          style: Style(dimensions: Percent(width: 50, height: 50)),
-        ),
-      ),
-      #(
-        Section("Row2"),
-        layout.Row(
-          content: "row 2",
-          style: Style(dimensions: Percent(width: 50, height: 50)),
-          children: [],
-        ),
-      ),
-    ],
-  ]
-
-  layout.new(columns, rows, views)
-}

src/musicplayer/ui/layout_examples/wait_for_input.gleam

@@ -1,15 +0,0 @@
-import gleam/erlang/process.{type Name}
-
-import musicplayer/input/input
-import musicplayer/input/key.{type Key}
-
-pub fn wait_for_input() {
-  let input_keys_name: Name(Key) = process.new_name("input_keys")
-  let assert Ok(_) = process.register(process.self(), input_keys_name)
-
-  input.new(input_keys_name)
-
-  process.new_selector()
-  |> process.select(process.named_subject(input_keys_name))
-  |> process.selector_receive_forever
-}

src/musicplayer/ui/plot.gleam

@@ -1,72 +0,0 @@
-import gleam/dict.{type Dict}
-import gleam/list
-import gleam/string
-
-pub type Buffer =
-  Dict(#(Int, Int), String)
-
-pub fn flush_buffer(buffer: Buffer, columns: Int, rows: Int) -> String {
-  list.range(1, rows)
-  |> list.map(fn(y) {
-    list.range(1, columns)
-    |> list.map(fn(x) {
-      case dict.get(buffer, #(x, y)) {
-        Ok(char) -> char
-        Error(_) -> " "
-      }
-    })
-    |> string.join("")
-  })
-  |> string.join("\r\n")
-}
-
-pub fn text(buffer: Buffer, text: String, x: Int, y: Int) -> Buffer {
-  text
-  |> string.to_graphemes
-  |> list.index_fold(buffer, fn(acc, char, i) {
-    dict.insert(acc, #(x + i, y), char)
-  })
-}
-
-pub fn box(buffer: Buffer, x: Int, y: Int, width: Int, height: Int) -> Buffer {
-  // TODO move box style to `layout.Style`
-  let box_chars = #("┌", "┐", "└", "┘", "─", "│")
-  let #(tl, tr, bl, br, hor, ver) = box_chars
-
-  case width < 2 || height < 2 {
-    True -> buffer
-    False -> {
-      buffer
-      |> dict.insert(#(x, y), tl)
-      |> dict.insert(#(x + width - 1, y), tr)
-      |> dict.insert(#(x, y + height - 1), bl)
-      |> dict.insert(#(x + width - 1, y + height - 1), br)
-      |> horizontal_line(x + 1, y, width - 2, hor)
-      |> horizontal_line(x + 1, y + height - 1, width - 2, hor)
-      |> vertical_line(x, y + 1, height - 2, ver)
-      |> vertical_line(x + width - 1, y + 1, height - 2, ver)
-    }
-  }
-}
-
-fn horizontal_line(
-  buffer: Buffer,
-  x: Int,
-  y: Int,
-  len: Int,
-  char: String,
-) -> Buffer {
-  list.range(0, len - 1)
-  |> list.fold(buffer, fn(acc, i) { dict.insert(acc, #(x + i, y), char) })
-}
-
-fn vertical_line(
-  buffer: Buffer,
-  x: Int,
-  y: Int,
-  len: Int,
-  char: String,
-) -> Buffer {
-  list.range(0, len - 1)
-  |> list.fold(buffer, fn(acc, i) { dict.insert(acc, #(x, y + i), char) })
-}

src/musicplayer/ui/ui.gleam

@@ -17,45 +17,7 @@ pub fn new() -> Result(Subject(Control), String) {
   let redraw_name = process.new_name("redraw")
   let redraw: Subject(Layout) = process.named_subject(redraw_name)
 
-  let layout =
+  let layout = layout.new()
-    [
-      [
-        #(
-          layout.Header,
-          layout.Row(
-            content: "Foo (1) | Bar (2) | Baz (3)",
-            style: layout.Style(dimensions: layout.Percent(
-              width: 100,
-              height: 33,
-            )),
-            children: [],
-          ),
-        ),
-        #(
-          layout.Search,
-          layout.Row(
-            content: "",
-            style: layout.Style(dimensions: layout.Percent(
-              width: 100,
-              height: 33,
-            )),
-            children: [],
-          ),
-        ),
-        #(
-          layout.PlaybackTime,
-          layout.Row(
-            content: "00:00",
-            style: layout.Style(dimensions: layout.Percent(
-              width: 100,
-              height: 33,
-            )),
-            children: [],
-          ),
-        ),
-      ],
-    ]
-    |> layout.new(0, 0, _)
 
   case
     actor.new(State(redraw, layout))
@@ -76,7 +38,7 @@ pub fn new() -> Result(Subject(Control), String) {
         internal.clear_screen()
         internal.hide_cursor()
 
-        redraw_loop(redraw)
+        redraw_on_update_loop(redraw)
       })
 
       Ok(ui)
@@ -89,30 +51,33 @@ fn handle_message(
   control: Control,
 ) -> actor.Next(State(redraw, layout), Control) {
   case control {
-    control.UpdateDimensions(columns, rows) -> {
+    control.UpdateDimensions(width, height) -> {
-      let current_dimensions = #(state.layout.columns, state.layout.rows)
+      let current_dimensions = #(state.layout.width, state.layout.height)
 
-      case #(columns, rows) == current_dimensions {
+      case #(width, height) == current_dimensions {
         True -> actor.continue(state)
         False -> {
-          [columns, rows]
+          [width, height]
           |> list.map(int.to_string)
           |> string.join(" ")
           |> string.append("ui - updating dimensions: ", _)
           |> logging.log
 
-          let layout = layout.update_dimensions(state.layout, columns, rows)
+          actor.continue(
-
+            State(
-          process.send(state.redraw, layout)
+              ..state,
-          actor.continue(State(..state, layout:))
+              layout: layout.update_dimensions(state.layout, width, height),
+            ),
+          )
         }
       }
     }
     control.UpdateState(section, content) -> {
       let layout = layout.update_section(state.layout, section, content)
+      let state = State(..state, layout:)
 
       actor.send(state.redraw, layout)
-      actor.continue(State(..state, layout:))
+      actor.continue(state)
     }
 
     control.Exit(reply_to) -> {
@@ -120,20 +85,16 @@ fn handle_message(
       process.send(reply_to, Nil)
       actor.stop()
     }
-    control.SetView(view_idx) -> {
-      let layout = layout.set_view(state.layout, view_idx)
-
-      actor.send(state.redraw, layout)
-      actor.continue(State(..state, layout:))
-    }
   }
 }
 
-fn redraw_loop(redraw: Subject(Layout)) -> Nil {
+fn redraw_on_update_loop(redraw: Subject(Layout)) -> Nil {
-  process.receive_forever(redraw)
+  let layout = process.receive_forever(redraw)
-  |> layout.render
 
-  redraw_loop(redraw)
+  internal.clear_screen()
+  layout.render(layout, layout.Root)
+
+  redraw_on_update_loop(redraw)
 }
 
 fn update_dimensions_on_interval(ui: Subject(Control), interval_ms: Int) {

test/musicplayer/control_test.gleam

@@ -1,7 +1,7 @@
 import gleam/list
 import gleeunit
 
-import musicplayer/control.{type Control, type Mode}
+import musicplayer/control.{type Control}
 import musicplayer/input/key.{type Key, Char}
 
 pub fn main() -> Nil {
@@ -9,24 +9,16 @@ pub fn main() -> Nil {
 }
 
 type TestCase {
-  TestCase(key: Key, mode: Mode, expected: Result(Control, Nil))
+  TestCase(key: Key, expected: Result(Control, Nil))
 }
 
 pub fn control_from_key_test() {
-  let idle_tests = [
+  let test_cases = [
-    TestCase(Char(" "), control.Idle, Ok(control.TogglePlayPause)),
+    TestCase(Char(" "), Ok(control.TogglePlayPause)),
-    TestCase(Char("/"), control.Idle, Ok(control.Search("", True))),
+    TestCase(Char("q"), Ok(control.Exit)),
-    TestCase(Char("q"), control.Idle, Ok(control.Exit)),
   ]
 
-  let search_tests = [
+  list.each(test_cases, fn(tc) {
-    TestCase(Char("a"), control.Searching(""), Ok(control.Search("a", True))),
+    assert tc.expected == control.from_key(tc.key)
-    TestCase(Char("b"), control.Searching("a"), Ok(control.Search("ab", True))),
-  ]
-
-  let test_cases = [idle_tests, search_tests]
-
-  list.each(list.flatten(test_cases), fn(tc) {
-    assert tc.expected == control.from_key(tc.key, tc.mode)
   })
 }

test/musicplayer/ui/layout_test.gleam

@@ -1,116 +0,0 @@
-import gleam/dict
-import gleam/io
-import gleam/string
-import gleeunit
-import gleeunit/should
-
-import musicplayer/ui/layout.{Percent, RenderContext, Section, Style}
-import musicplayer/ui/plot
-
-pub fn main() -> Nil {
-  gleeunit.main()
-}
-
-pub fn percent_layout_test() {
-  let views = [
-    [
-      #(
-        Section("Row1"),
-        layout.Row(
-          content: "row 1",
-          style: Style(dimensions: Percent(width: 100, height: 50)),
-          children: [
-            Section("A"),
-            Section("B"),
-          ],
-        ),
-      ),
-      #(
-        Section("A"),
-        layout.Cell(
-          content: "cell 1",
-          style: Style(dimensions: Percent(width: 50, height: 100)),
-        ),
-      ),
-      #(
-        Section("B"),
-        layout.Cell(
-          content: "cell 2",
-          style: Style(dimensions: Percent(width: 50, height: 100)),
-        ),
-      ),
-      #(
-        Section("Row2"),
-        layout.Row(
-          content: "row 1",
-          style: Style(dimensions: Percent(width: 100, height: 50)),
-          children: [],
-        ),
-      ),
-    ],
-  ]
-
-  let columns = 80
-  let rows = 20
-  let layout = layout.new(columns, rows, views)
-
-  let expected =
-    "
-┌──────────────────────────────────────────────────────────────────────────────┐
-│row 1────────────────────────────────────────────────────────────────────────┐│
-││cell 1───────────────────────────────┐cell 2───────────────────────────────┐││
-│││                                    ││                                    │││
-│││                                    ││                                    │││
-│││                                    ││                                    │││
-│││                                    ││                                    │││
-│││                                    ││                                    │││
-││└────────────────────────────────────┘└────────────────────────────────────┘││
-│└────────────────────────────────────────────────────────────────────────────┘│
-│row 1────────────────────────────────────────────────────────────────────────┐│
-││                                                                            ││
-││                                                                            ││
-││                                                                            ││
-││                                                                            ││
-││                                                                            ││
-││                                                                            ││
-││                                                                            ││
-│└────────────────────────────────────────────────────────────────────────────┘│
-└──────────────────────────────────────────────────────────────────────────────┘
-"
-    |> string.replace(each: "\n", with: "\r\n")
-    |> string.trim
-
-  let context =
-    RenderContext(
-      parent_width: layout.columns,
-      parent_height: layout.rows,
-      parent_top_left_x: 1,
-      parent_top_left_y: 1,
-      position_index: 0,
-    )
-
-  let assert Ok(view) = dict.get(layout.views, layout.current_view)
-
-  let flushed =
-    layout.render_loop(
-      view,
-      context,
-      Section(string.append("view_", string.inspect(layout.current_view))),
-      dict.new(),
-    )
-    |> plot.flush_buffer(layout.columns, layout.rows)
-
-  case flushed == expected {
-    True -> Nil
-    False -> {
-      io.println("Test failed")
-      io.println("Expected:")
-      io.println(string.trim(expected))
-
-      io.println("Got:")
-      io.println(flushed)
-
-      should.equal(flushed, expected)
-    }
-  }
-}