Skip to content

Commit

Permalink
fix: build popup called directly
Browse files Browse the repository at this point in the history
h/t @djb96
  • Loading branch information
jack-davison committed Nov 18, 2024
1 parent 71ae179 commit 4b3d62a
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
2 changes: 1 addition & 1 deletion DESCRIPTION
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
Type: Package
Package: openairmaps
Title: Create Maps of Air Pollution Data
Version: 0.9.0.9005
Version: 0.9.0.9006
Authors@R: c(
person("Jack", "Davison", , "[email protected]", role = c("cre", "aut"),
comment = c(ORCID = "0000-0003-2653-6615")),
Expand Down
2 changes: 2 additions & 0 deletions NEWS.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,8 @@

* The order in which `trajMapStatic()` draws points and paths has been tweaked, which should ensure that markers are drawn on top of their respective path, rather than on top of all paths.

* `buildPopup()` will now work correctly if no `type` is provided.

# openairmaps 0.9.0

## Breaking changes
Expand Down
3 changes: 2 additions & 1 deletion R/utilities.R
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,8 @@ buildPopup <-
fun.dttm = function(x) paste(lubridate::floor_date(range(x, na.rm = TRUE), "day"), collapse = " to "),
...) {
# check for old facet/control opts
type <- type %||% check_facet_control(...)
dots <- rlang::list2(...)
type <- type %||% check_facet_control(control = dots$control)

# assume latitude/longitude
latlon <- assume_latlon(
Expand Down

0 comments on commit 4b3d62a

Please sign in to comment.