Pct column widths

[teunbrand]

Summary

This PR aims to fix #1771.
Briefly, it works by accounting for pct()-widths in grid_layout_widths().
The reprex from the issue now renders:

devtools::load_all("~/packages/gt/")
#> ℹ Loading gt

start_date <- "2010-06-07"
end_date <- "2010-06-08"

sp_tab <- 
  sp500 |>
  dplyr::filter(date >= start_date & date <= end_date) |>
  gt() |>
  cols_width(date ~ pct(50)) |>
  as_gtable(plot = TRUE)

^{Created on 2024-07-12 with reprex v2.1.1}

Related GitHub Issues and PRs

• Ref: cols_width specifications and as_gtable #1771

Checklist

• I understand and agree to the Code of Conduct.
• I have listed any major changes in the NEWS.
• I have added testthat unit tests to tests/testthat for any new functionality.

[olivroy]

Awesome! I have a small request. Do you know how grid output could convert
to be a linebreak \n (\n is recognized by grid). I tried searching for the right place to implement, but couldn't figure it out.

This reprex:

data.frame(estimate = pi, std.error = pi / 10) |> 
  gt::gt() |> 
  gt::fmt_number() |> 
  gt::cols_merge(columns = c(estimate, std.error), pattern = "{1}  <br> ({2})")

renders to this in html

It'd be great if it rendered to the same thing in gtable, but currently, it just shows
.

A well-placed

gsub("<br>", "\n", x)

would do the trick, but I couldn't figure out where to put this. (for #1767 )

[teunbrand]

It isn't really related to this PR, but this is exactly the kind of thing why the as_gtable(text_grob) argument exists.
If you don't want to render text verbatim, you can use:

data.frame(estimate = pi, std.error = pi / 10) |> 
  gt::gt() |> 
  gt::fmt_number() |> 
  gt::cols_merge(columns = c(estimate, std.error), pattern = "{1} \n ({2})") |>
  gt::as_gtable(
    plot = TRUE, text_grob = function(label, ...)  {
      grid::textGrob(gsub("<br>", "\n", label), ...)
    }
  )

^{Created on 2024-07-12 with reprex v2.1.1}

[rich-iannone]

Thank you!