Skip to content

Commit 6c1fde5

Browse files
committed
Improve time_trans error message in ScaleContinuous
Adds a suggestion to use scale_x_date() instead of scale_x_datetime() This is more helpful for the specific example in issue tidyverse#3533 but may be misleading if the error is triggered in another way.
1 parent 9f0fc2d commit 6c1fde5

File tree

1 file changed

+10
-1
lines changed

1 file changed

+10
-1
lines changed

R/scale-.r

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -549,7 +549,16 @@ ScaleContinuous <- ggproto("ScaleContinuous", Scale,
549549
},
550550

551551
transform = function(self, x) {
552-
new_x <- self$trans$transform(x)
552+
new_x <- tryCatch({
553+
self$trans$transform(x)
554+
}, error = function(err) {
555+
err$message <- paste(
556+
err$message,
557+
"\n\t",
558+
"Do you want to use scale_x_date() instead of scale_x_datetime()?"
559+
)
560+
stop(err)
561+
})
553562
axis <- if ("x" %in% self$aesthetics) "x" else "y"
554563
check_transformation(x, new_x, self$scale_name, axis)
555564
new_x

0 commit comments

Comments
 (0)