Skip to content

Commit d9d1305

Browse files
committed
wrap palettes
1 parent c0e9b80 commit d9d1305

File tree

1 file changed

+18
-16
lines changed

1 file changed

+18
-16
lines changed

R/scale-colour.R

Lines changed: 18 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@
7878
#' v
7979
#' options(ggplot2.continuous.fill = tmp) # restore previous setting
8080
#' @export
81-
scale_colour_continuous <- function(..., aesthetics = "colour",
81+
scale_colour_continuous <- function(..., palette = NULL, aesthetics = "colour",
8282
guide = "colourbar", na.value = "grey50",
8383
type = getOption("ggplot2.continuous.colour")) {
8484

@@ -89,16 +89,16 @@ scale_colour_continuous <- function(..., aesthetics = "colour",
8989
)
9090
return(scale)
9191
}
92-
92+
pal <- if (!is.null(palette)) as_continuous_pal(palette)
9393
continuous_scale(
94-
aesthetics, palette = NULL, guide = guide, na.value = na.value,
94+
aesthetics, palette = palette, guide = guide, na.value = na.value,
9595
...
9696
)
9797
}
9898

9999
#' @rdname scale_colour_continuous
100100
#' @export
101-
scale_fill_continuous <- function(..., aesthetics = "fill", guide = "colourbar",
101+
scale_fill_continuous <- function(..., palette = NULL, aesthetics = "fill", guide = "colourbar",
102102
na.value = "grey50",
103103
type = getOption("ggplot2.continuous.fill")) {
104104

@@ -109,16 +109,16 @@ scale_fill_continuous <- function(..., aesthetics = "fill", guide = "colourbar",
109109
)
110110
return(scale)
111111
}
112-
112+
palette <- if (!is.null(palette)) as_continuous_pal(palette)
113113
continuous_scale(
114-
aesthetics, palette = NULL, guide = guide, na.value = na.value,
114+
aesthetics, palette = palette, guide = guide, na.value = na.value,
115115
...
116116
)
117117
}
118118

119119
#' @export
120120
#' @rdname scale_colour_continuous
121-
scale_colour_binned <- function(..., aesthetics = "colour", guide = "coloursteps",
121+
scale_colour_binned <- function(..., palette = NULL, aesthetics = "colour", guide = "coloursteps",
122122
na.value = "grey50",
123123
type = getOption("ggplot2.binned.colour")) {
124124
if (!is.null(type)) {
@@ -128,16 +128,16 @@ scale_colour_binned <- function(..., aesthetics = "colour", guide = "coloursteps
128128
)
129129
return(scale)
130130
}
131-
131+
palette <- if (!is.null(palette)) pal_binned(as_discrete_pal(palette))
132132
binned_scale(
133-
aesthetics, palette = NULL, guide = guide, na.value = na.value,
133+
aesthetics, palette = palette, guide = guide, na.value = na.value,
134134
...
135135
)
136136
}
137137

138138
#' @export
139139
#' @rdname scale_colour_continuous
140-
scale_fill_binned <- function(..., aesthetics = "fill", guide = "coloursteps",
140+
scale_fill_binned <- function(..., palette = NULL, aesthetics = "fill", guide = "coloursteps",
141141
na.value = "grey50",
142142
type = getOption("ggplot2.binned.fill")) {
143143
if (!is.null(type)) {
@@ -147,9 +147,9 @@ scale_fill_binned <- function(..., aesthetics = "fill", guide = "coloursteps",
147147
)
148148
return(scale)
149149
}
150-
150+
palette <- if (!is.null(palette)) pal_binned(as_discrete_pal(palette))
151151
binned_scale(
152-
aesthetics, palette = NULL, guide = guide, na.value = na.value,
152+
aesthetics, palette = palette, guide = guide, na.value = na.value,
153153
...
154154
)
155155
}
@@ -210,7 +210,7 @@ scale_fill_binned <- function(..., aesthetics = "fill", guide = "coloursteps",
210210
#' print(cty_by_var(fl))
211211
#' })
212212
#'
213-
scale_colour_discrete <- function(..., aesthetics = "colour", na.value = "grey50",
213+
scale_colour_discrete <- function(..., palette = NULL, aesthetics = "colour", na.value = "grey50",
214214
type = getOption("ggplot2.discrete.colour")) {
215215
if (!is.null(type)) {
216216
scale <- scale_backward_compatibility(
@@ -219,15 +219,16 @@ scale_colour_discrete <- function(..., aesthetics = "colour", na.value = "grey50
219219
)
220220
return(scale)
221221
}
222+
palette <- if (!is.null(palette)) as_discrete_pal(palette)
222223
discrete_scale(
223-
aesthetics, palette = NULL, na.value = na.value,
224+
aesthetics, palette = palette, na.value = na.value,
224225
...
225226
)
226227
}
227228

228229
#' @rdname scale_colour_discrete
229230
#' @export
230-
scale_fill_discrete <- function(..., aesthetics = "fill", na.value = "grey50",
231+
scale_fill_discrete <- function(..., palette = NULL, aesthetics = "fill", na.value = "grey50",
231232
type = getOption("ggplot2.discrete.fill")) {
232233
if (!is.null(type)) {
233234
scale <- scale_backward_compatibility(
@@ -236,8 +237,9 @@ scale_fill_discrete <- function(..., aesthetics = "fill", na.value = "grey50",
236237
)
237238
return(scale)
238239
}
240+
palette <- if (!is.null(palette)) as_discrete_pal(palette)
239241
discrete_scale(
240-
aesthetics, palette = NULL, na.value = na.value,
242+
aesthetics, palette = palette, na.value = na.value,
241243
...
242244
)
243245
}

0 commit comments

Comments
 (0)