Skip to content

Commit

Permalink
Added data-external (#52)
Browse files Browse the repository at this point in the history
* Added data-external

* bump version, add to news

Co-authored-by: Ian Lyttle <ian.lyttle@se.com>
  • Loading branch information
jnolis and ijlyttle authored Dec 6, 2021
1 parent ea37116 commit 3171d5f
Show file tree
Hide file tree
Showing 7 changed files with 14 additions and 7 deletions.
2 changes: 1 addition & 1 deletion DESCRIPTION
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
Package: vembedr
Title: Embed Video in HTML
Version: 0.1.4.9002
Version: 0.1.4.9003
Authors@R: c(
person(
given = "Ian",
Expand Down
4 changes: 3 additions & 1 deletion NEWS.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
# vembedr 0.1.4.9000 (development version)

- updates `<iframe>` implementation to keep up with pandoc, see [rstudio/rmarkdown#2255](https://github.com/rstudio/rmarkdown/issues/2255) (#52, @jnolis).

- version bump for development

## vembedr 0.1.4
Expand All @@ -17,7 +19,7 @@

## vembedr 0.1.3

- updates README to show some custom formatting (#25, thanks @koncina and @ginolhac)
- updates README to show some custom formatting (#25, @koncina & @ginolhac)
- deprecates `hms()` and `secs()` in favor of `use_start_time()` (#24)
- adds function `embed_user2017()`: embed videos from UseR!2017
- version bump for development
Expand Down
3 changes: 2 additions & 1 deletion R/service-box.R
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,8 @@ embed_box <- function(id, custom_domain = getOption("vembedr.box_custom_domain")
frameborder = frameborder,
allowfullscreen = allowfullscreen,
webkitallowfullscreen = allowfullscreen,
msallowfullscreen = allowfullscreen
msallowfullscreen = allowfullscreen,
`data-external` = 1
)

embed <- create_embed(iframe, "vembedr_embed_box", ratio)
Expand Down
3 changes: 2 additions & 1 deletion R/service-channel9.R
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,8 @@ embed_channel9 <- function(id,
width = dim$width,
height = dim$height,
frameborder = frameborder,
allowfullscreen = allowfullscreen
allowfullscreen = allowfullscreen,
`data-external` = 1
)

embed <- create_embed(iframe, "vembedr_embed_channel9", ratio)
Expand Down
3 changes: 2 additions & 1 deletion R/service-msstream.R
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,8 @@ embed_msstream <- function(id, width = NULL, height = 300,
width = dim$width,
height = dim$height,
allowfullscreen = NULL,
style = "border:none;"
style = "border:none;",
`data-external` = 1
)

embed <- create_embed(iframe, "vembedr_embed_msstream", ratio)
Expand Down
3 changes: 2 additions & 1 deletion R/service-vimeo.R
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,8 @@ embed_vimeo <- function(id,
frameborder = frameborder,
webkitallowfullscreen = allowfullscreen,
mozallowfullscreen = allowfullscreen,
allowfullscreen = allowfullscreen
allowfullscreen = allowfullscreen,
`data-external` = 1
)

embed <- create_embed(iframe, "vembedr_embed_vimeo", ratio)
Expand Down
3 changes: 2 additions & 1 deletion R/service-youtube.R
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,8 @@ embed_youtube <- function(id,
width = dim$width,
height = dim$height,
frameborder = frameborder,
allowfullscreen = allowfullscreen
allowfullscreen = allowfullscreen,
`data-external` = 1
)

embed <- create_embed(iframe, "vembedr_embed_youtube", ratio)
Expand Down

0 comments on commit 3171d5f

Please sign in to comment.