Don't export LayerSf

[clauswilke]

When I implemented custom layers for sf objects, I exported the relevant layer object, LayerSf():

ggplot2/R/layer-sf.R

Lines 3 to 8 in 033fb52

	#' @export
	#' @include layer.r
	#' @rdname ggsf
	#' @usage NULL
	#' @format NULL
	LayerSf <- ggproto("LayerSf", Layer,

This is bad, because we currently don't export its parent, Layer(), and we don't want to export it at this time.

The correct way to make the sf-specific layer available is to add a constructor function, layer_sf(), which replaces the layer() function for sf layers.

[yutannihilation]

Probably, we should not expose layer_class argument of layer() as well; the user cannot provide any Layer class as it's not exported.

ggplot2/R/layer.r

Line 66 in 033fb52

show.legend = NA, layer_class = Layer) {

[paleolimbot]

@clauswilke : I think you asked...I've never needed to use layer() when creating a geom_sf() layer...the only thing I've changed is the Stat passed to geom_sf().

[lock]

This old issue has been automatically locked. If you believe you have found a related problem, please file a new issue (with reprex) and link to this issue. https://reprex.tidyverse.org/