feat(layout): minor adjustment to layout config

This addresses the first request in aome510#535. The change is mainly in adjusting how the borders are drawn.
DOD-101 · Aug 14, 2024 · efd1ab8 · efd1ab8
1 parent c3e9a4e
commit efd1ab8
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/spotify_player/src/config/mod.rs b/spotify_player/src/config/mod.rs
@@ -328,8 +328,8 @@ impl Default for LayoutConfig {
 
 impl LayoutConfig {
     fn check_values(&self) -> anyhow::Result<()> {
-        if self.library.album_percent + self.library.playlist_percent > 99 {
-            anyhow::bail!("Invalid library layout: summation of album_percent and playlist_percent cannot be greater than 99!");
+        if self.library.album_percent + self.library.playlist_percent > 100 {
+            anyhow::bail!("Invalid library layout: summation of album_percent and playlist_percent cannot be greater than 100!");
         } else {
             Ok(())
         }

diff --git a/spotify_player/src/ui/page.rs b/spotify_player/src/ui/page.rs
@@ -341,14 +341,22 @@ pub fn render_library_page(
     let playlist_rect = construct_and_render_block(
         "Playlists",
         &ui.theme,
-        Borders::TOP | Borders::LEFT | Borders::BOTTOM,
+        if chunks[1].area() != 0 {
+            Borders::TOP | Borders::LEFT | Borders::BOTTOM
+        } else {
+            Borders::ALL
+        },
         frame,
         chunks[0],
     );
     let album_rect = construct_and_render_block(
         "Albums",
         &ui.theme,
-        Borders::TOP | Borders::LEFT | Borders::BOTTOM,
+        if chunks[2].area() != 0 {
+            Borders::TOP | Borders::LEFT | Borders::BOTTOM
+        } else {
+            Borders::ALL
+        },
         frame,
         chunks[1],
     );