Fix vegalite legends (#119)

Author: thomasp85

doc/syntax/scale/type/binned.qmd

@@ -155,6 +155,11 @@ Besides direct renaming you can also provide a formatting string if you want the
 
 You can combine formatting with direct renaming in which case the direct renaming has priority over the formatting.
 
+### Labels in binned legends
+With some writers the legend for binned scales looks like the standard legend but with the label showing the range of the bin. In these situations the renaming is applied before the range label is being created. For example, if you have a `RENAMING 0 => 'zero'`, then the final label will become "zero – 10" (assuming the upper end of the bin is 10). There is currently no way to take control over the format of the range label.
+
+If `oob => 'squish'` then the terminal labels are formatted as e.g. "≥ 10" to reflect the terminal bins are open-ended. It still applies that the renaming is applied before constructing the final label
+
 ### Examples
 
 #### Rename a select break

ggsql-jupyter/src/display.rs

@@ -69,7 +69,7 @@ fn format_vegalite(spec: String) -> Value {
         paths: {{
           'dom-ready': 'https://cdn.jsdelivr.net/npm/domready@1/ready.min',
           'vega': 'https://cdn.jsdelivr.net/npm/vega@6/build/vega.min',
-          'vega-lite': 'https://cdn.jsdelivr.net/npm/vega-lite@6/build/vega-lite.min',
+          'vega-lite': 'https://cdn.jsdelivr.net/npm/vega-lite@6.4.1/build/vega-lite.min',
           'vega-embed': 'https://cdn.jsdelivr.net/npm/vega-embed@7/build/vega-embed.min'
         }}
       }});
@@ -101,7 +101,7 @@ fn format_vegalite(spec: String) -> Value {
 
       Promise.all([
         loadScript('https://cdn.jsdelivr.net/npm/vega@6'),
-        loadScript('https://cdn.jsdelivr.net/npm/vega-lite@6'),
+        loadScript('https://cdn.jsdelivr.net/npm/vega-lite@6.4.1'),
         loadScript('https://cdn.jsdelivr.net/npm/vega-embed@7')
       ])
         .then(() => {{

src/plot/scale/scale_type/binned.rs

@@ -429,8 +429,14 @@ impl ScaleTypeTrait for Binned {
                     breaks.first().unwrap().clone(),
                     breaks.last().unwrap().clone(),
                 ];
-                let expanded = expand_numeric_range(&terminal_range, mult, add);
-                new_input_range = Some(expanded);
+                // Only expand for positional aesthetics (x, y, etc.)
+                // Non-positional aesthetics (color, fill, size) don't get expansion
+                let final_range = if super::is_positional_aesthetic(aesthetic) {
+                    expand_numeric_range(&terminal_range, mult, add)
+                } else {
+                    terminal_range
+                };
+                new_input_range = Some(final_range);
             }
 
             // Update the breaks in the scale