Allow use of rep() within ggplot2::margin()

[cararthompson]

In the previous version, I believe it was possible to use rep() within ggplot2::margin() to set margins to all be the same value. This seems to no longer be possible. Is this change deliberate? It was handy when wanting to create margins based on the base text size argument of a custom theme function, for example (e.g plot.margin = margin(rep(base_text_size * 1.5, 4)))

# This plot has no margins
babynames::babynames |>
  dplyr::filter(name %in% c("Mary", "Hannah"), sex == "F") |>
  ggplot(aes(x = year, y = n, group = name, color = name)) +
  geom_line() +
  theme(plot.margin = margin(rep(20, 4)))

# This plot does have margins
babynames::babynames |>
  dplyr::filter(name %in% c("Mary", "Hannah"), sex == "F") |>
  ggplot(aes(x = year, y = n, group = name, color = name)) +
  geom_line() +
  theme(plot.margin = margin(20, 20, 20, 20))

[teunbrand]

Using margin(rep(20, 4)) was also incorrect previously, but it worked by accident. The t argument would receive 4 values, whereas r, b and l arguments would receive none. We now require each of these arguments to receive one value. You can use margin_auto(20) to populate all 4 simultaneously.

[cararthompson]

Perfect, thank you for the clarification! I did think it was odd, but it was handy. Glad to see there's a better solution!

[teunbrand]

Alright then, I'll consider this issue closed then :)