refine chart resolution and parameter spacing

Author: shrimbly

sharp_frames/ui/screens/selection.py

@@ -33,6 +33,8 @@ class SharpnessChart(ScrollView):
     """Horizontally scrollable frame-by-frame sharpness timeline."""
 
     FRAME_STRIDE = 2
+    BAR_GLYPHS = (" ", "▁", "▂", "▃", "▄", "▅", "▆", "▇", "█")
+    VERTICAL_SUBDIVISIONS = len(BAR_GLYPHS) - 1
     can_focus = True
 
     BINDINGS = [
@@ -53,7 +55,7 @@ class SharpnessChart(ScrollView):
 
     DEFAULT_CSS = """
     SharpnessChart {
-        height: 12;
+        height: 13;
         width: 100%;
         border: solid $primary;
         margin: 1 0;
@@ -212,16 +214,30 @@ def _render_timeline_column(
 
         frame = self.frames[frame_position]
         normalized = (frame.sharpness_score - self.min_score) / self.score_range
-        bar_height = 1 + int(normalized * (chart_height - 1))
-        if (chart_height - 1 - chart_y) >= bar_height:
+        glyph = self._bar_glyph(normalized, chart_y, chart_height)
+        if glyph == " ":
             return Segment(" ", blank_style)
 
         style = (
             selected_style
             if frame.index in self.selected_indices
             else unselected_style
         )
-        return Segment("█", style)
+        return Segment(glyph, style)
+
+    @classmethod
+    def _bar_glyph(
+        cls, normalized: float, chart_y: int, chart_height: int
+    ) -> str:
+        """Render a bar cell with eighth-row vertical precision."""
+        subdivisions = cls.VERTICAL_SUBDIVISIONS
+        available_units = max(chart_height * subdivisions, 1)
+        clamped = max(0.0, min(float(normalized), 1.0))
+        filled_units = max(1, int((clamped * available_units) + 0.5))
+        row_from_bottom = chart_height - 1 - chart_y
+        units_in_row = filled_units - (row_from_bottom * subdivisions)
+        units_in_row = max(0, min(units_in_row, subdivisions))
+        return cls.BAR_GLYPHS[units_in_row]
 
     def _visible_frame_range(self, width: int) -> tuple[int, int]:
         """Return the frame positions represented by the visible viewport."""

sharp_frames/ui/styles.py

@@ -421,7 +421,7 @@
 
 /* Sharpness chart */
 SharpnessChart {
-    height: 12;
+    height: 13;
     width: 100%;
     border: solid $primary;
     margin: 1 0;
@@ -488,6 +488,25 @@
     margin: 0 0 1 0;
 }
 
+/* Keep the parameter panel dense so its controls sit close to the title and
+   the panel does not leave an oversized empty footer. */
+#parameter_container {
+    padding: 1 2 0 1;
+    min-height: 10;
+}
+
+#parameter_container .control_label,
+#parameter_container .parameter_inputs,
+#parameter_container .param_label,
+#parameter_container .param_input_with_controls,
+#parameter_container InputWithControls {
+    margin: 0;
+}
+
+#parameter_container .parameter_inputs {
+    min-height: 0;
+}
+
 InputWithControls Input {
     width: 20;
     margin: 0 1 0 0;

tests/ui/test_selection_screen.py

@@ -185,6 +185,13 @@ def test_chart_normalizes_scores_without_discarding_frames(extraction_result):
     assert chart.border_title == "Frame selection"
 
 
+def test_chart_uses_fractional_blocks_for_sub_row_precision():
+    assert SharpnessChart._bar_glyph(0.0, chart_y=2, chart_height=3) == "▁"
+    assert SharpnessChart._bar_glyph(1.0, chart_y=0, chart_height=3) == "█"
+    assert SharpnessChart._bar_glyph(0.5, chart_y=1, chart_height=3) == "▄"
+    assert SharpnessChart._bar_glyph(0.5, chart_y=0, chart_height=3) == " "
+
+
 @pytest.mark.asyncio
 async def test_chart_scrolls_across_thousands_of_frames():
     frames = [
@@ -220,7 +227,7 @@ def compose(self) -> ComposeResult:
             for segment in title_line
         )
         bottom_line = chart.render_line(chart.scrollable_content_region.height - 1)
-        assert bottom_line.text[:8] == "█ █ █ █ "
+        assert bottom_line.text[:8] == "▁ ▁ ▁ ▁ "
 
         chart.focus()
         await pilot.press("right")