Fix expression labels in guide_coloursteps() and guide_bins()

[teunbrand]

This PR aims to fix #6005.

Briefly, internally we dance around <expression> labels by casting them to lists of expressions when storing them in <data.frame>s and later unpacking such lists to expressions when handing them off to {grid}. I forgot to do this for guide_coloursteps() and guide_bins(), which have very custom code for getting the keys from the scale. This PR rectifies that.

Reprex from the issue:

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

p <- data.frame(x = 1:5, y = 1:5) |>
  ggplot(aes(x = x, y = y, color = y))+
  geom_text(label = paste0("x^", 1:5), parse = TRUE)+
  scale_color_stepsn(
    breaks = 1:5,
    colors = c("red", "orange", "yellow","green", "blue"),
    labels = parse(text = paste0("x^", 1:5))
  )
p

p + guides(colour = "bins")

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

[teunbrand]

Wondering if it wouldn't be better if this was the responsibility of Scale$get_labels().

[thomasp85]

yeah... Do we have this call in other places so that moving it to Scale$get_labels() will allow us to remove some code elsewhere?

[teunbrand]

In some places for sure. The issue is that, at the moment, it is a balanced trade-off. We have 3 guides where this guard is used, but we also have 3 Scale$get_labels() methods that might generate expressions.

[thomasp85]

I think that it logically belongs in get_labels() as it ensures the output of that function is usable. It shouldn't be the responsibility of the caller to further process the return value to make it work