Merge branch 'master' into fix/invalid-selector-character

Author: ANAMASGARD

DESCRIPTION

@@ -1,6 +1,6 @@
 Package: animint2
 Title: Animated Interactive Grammar of Graphics
-Version: 2025.10.31
+Version: 2025.11.17
 URL: https://animint.github.io/animint2
 BugReports: https://github.com/animint/animint2/issues
 Authors@R: c(
@@ -64,7 +64,7 @@ Authors@R: c(
      comment="Animint2 GSoC 2025"),
     person("Gaurav", "Chaudhary",
      role="ctb",
-     comment="Remove unused css.file parameter; fix issue #233 selector values"))
+     comment="Remove unused css.file parameter; fix issue #233 selector values; fix issue #273 update_axes tick font-size"))
 Description: Functions are provided for defining animated,
  interactive data visualizations in R code, and rendering
  on a web page. The 2018 Journal of Computational and 

NEWS.md

@@ -1,3 +1,7 @@
+# Changes in version 2025.11.17 (PR#274)
+
+- `update_axes`: Fixed issue #273 where axis tick text font-size was inconsistent between plots with and without `update_axes`. Previously, plots using `theme_animint(update_axes="x")` would lose `theme(axis.text = element_text(size=...))` styling after axis updates.
+
 # Changes in version 2025.10.31 (PR#271)
 
 - `geom_point()` now warns when shape parameter is set to a value other than 21, since animint2 web rendering only supports shape=21 for proper display of both color and fill aesthetics.

inst/htmljs/animint.js

@@ -647,10 +647,7 @@ var animint = function (to_select, json_file) {
           var axis_path = xaxis_g.select("path.domain");
           axis_path.remove();
         }
-        xaxis_g.selectAll("text")
-          .style("text-anchor", p_info.xanchor)
-          .style("font-size", p_info.xsize)
-          .attr("transform", "rotate(" + p_info.xangle + " 0 9)");
+        apply_axis_text_styles(xaxis_g, "x", p_info);
       }
       if(draw_y){
         var yaxis = d3.svg.axis()
@@ -669,8 +666,7 @@ var animint = function (to_select, json_file) {
           var axis_path = yaxis_g.select("path.domain");
           axis_path.remove();
         }
-  yaxis_g.selectAll(".tick text")
-    .style("font-size", p_info.ysize);
+        apply_axis_text_styles(yaxis_g, "y", p_info);
       }
 
       if(!axis.xline) {
@@ -1930,6 +1926,20 @@ var animint = function (to_select, json_file) {
       }
     }
 
+  // Helper function to apply axis text styling
+  // Used by both axis initialization and update_axes to ensure consistency
+  function apply_axis_text_styles(axis_g, axes, p_info){
+    if(axes == "x"){
+      axis_g.selectAll("text")
+        .style("text-anchor", p_info.xanchor)
+        .style("font-size", p_info.xsize)
+        .attr("transform", "rotate(" + p_info.xangle + " 0 9)");
+    }else{
+      axis_g.selectAll(".tick text")
+        .style("font-size", p_info.ysize);
+    }
+  }
+
   // update scales for the plots that have update_axes option in
   // theme_animint
   function update_scales(p_name, axes, v_name, value){
@@ -1999,10 +2009,13 @@ var animint = function (to_select, json_file) {
           .orient(orientation)
           .tickValues(tick_vals);
     // update existing axis
-    var xyaxis_g = element.select("#plot_"+p_name).select("."+axes+"axis_"+panel_i)
+    var xyaxis_sel = element.select("#plot_"+p_name).select("."+axes+"axis_"+panel_i);
+    var xyaxis_g = xyaxis_sel
           .transition()
           .duration(1000)
           .call(xyaxis);
+    // Fix for issue #273: preserve axis text styling after update
+    apply_axis_text_styles(xyaxis_sel, axes, Plots[p_name]);
   }
 
   // Update major/minor grids once axes ticks have been updated

tests/testthat/test-renderer3-update-axes-tick-font-size.R

@@ -0,0 +1,29 @@
+acontext("update_axes tick font-size consistency - Issue #273")
+mtcars_test <- mtcars
+mtcars_test$cyl <- as.factor(mtcars_test$cyl)
+viz_tick_size <- list(
+  noUpdate = ggplot() +
+    geom_point(aes(mpg, disp, colour=cyl), data=mtcars_test) +
+    ggtitle("No update_axes") +
+    theme(axis.text = element_text(size=12)),
+  withUpdateX = ggplot() +
+    geom_point(aes(mpg, disp, colour=cyl), showSelected="cyl", data=mtcars_test) +
+    theme_animint(update_axes=c("x")) +
+    ggtitle("With update_axes x") +
+    theme(axis.text = element_text(size=12)),
+  selector.types = list(cyl="single")
+)
+info <- animint2HTML(viz_tick_size)
+test_that("x-axis tick text font-size is consistent with and without update_axes", {
+  no_update_xpath <- '//svg[@id="plot_noUpdate"]//g[@class="xaxis axis xaxis_1"]//g[@class="tick major"]//text'
+  with_update_xpath <- '//svg[@id="plot_withUpdateX"]//g[@class="xaxis axis xaxis_1"]//g[@class="tick major"]//text'
+  no_update_font_size <- getStyleValue(info$html, no_update_xpath, "font-size")
+  with_update_font_size <- getStyleValue(info$html, with_update_xpath, "font-size")
+  expect_gt(length(no_update_font_size), 0)
+  expect_gt(length(with_update_font_size), 0)
+  no_update_size_unique <- unique(no_update_font_size)
+  with_update_size_unique <- unique(with_update_font_size)
+  expect_equal(length(no_update_size_unique), 1)
+  expect_equal(length(with_update_size_unique), 1)
+  expect_equal(with_update_size_unique, no_update_size_unique, info=sprintf("Axis tick text font-size should be consistent regardless of update_axes. Expected %s but got %s", no_update_size_unique, with_update_size_unique))
+})