`cols_hide()` has a documentation issue and error msg

[olivroy]

Description

library(gt)

exibble |> 
  gt() |> 
  cols_hide()
#> Error in translate_legacy_resolver_expr(quo, null_means): l'argument "expr" est manquant, avec aucune valeur par défaut

^{Created on 2024-04-24 with reprex v2.1.0}

The message when you forget to supply input to cols_hide() is a bit ugly and the docs is misleading.

Docs are this currently

Columns to target

⁠<column-targeting expression>⁠ // default: everything()

Expected result

I think that the documentation should be updated and say that columns must be supplied.

I also think it would be better to use rlang::check_required() in cols_hide() to avoid an ugly message.

I don't think having the documented everything() as a default is a good idea. (documentation was not updated when #1342 was resolved)

I think the implentation of the function is (almost) okay.

[olivroy]

Preparing a PR, but the edge case gives a bad message. (not affected by the PR)

 gt(exibble) |> cols_hide(c(everything()))

would be good if it behaved like

gt(data.frame())

This worked properly in gt 0.10.0, but stopped working in gt 0.10.1....

# blank in gt 0.10.0
exibble |> gt() |> cols_hide(everything())
# errors in gt 0.10.1
#> Error in has_rtl[i, ] : subscript out of bounds