Don't imply autoplay when t option set in embed context

element
Marcin Kulik 9 years ago
parent a86235cc42
commit 1f85e0cb1c

@ -30,7 +30,7 @@ module Api
end
format.json do
render json: asciicast, playback_options: PlaybackOptions.new
render json: asciicast, v0: params[:v0]
end
end
end

@ -0,0 +1,3 @@
class EmbedPlaybackOptions < PlaybackOptions
attribute :autoplay, Boolean, default: false
end

@ -25,7 +25,7 @@ class PlaybackOptions
def as_json(*)
opts = {
speed: speed,
autoPlay: autoplay,
autoPlay: autoplay.nil? ? !!t : autoplay,
loop: loop,
fontSize: size,
theme: theme,

@ -9,7 +9,7 @@ class BareAsciicastPagePresenter
'theme' => decorated_asciicast.theme_name
}.merge(playback_options)
new(decorated_asciicast, PlaybackOptions.new(playback_options))
new(decorated_asciicast, EmbedPlaybackOptions.new(playback_options))
end
def initialize(asciicast, playback_options)

@ -10,7 +10,7 @@ class AsciicastSerializer < ActiveModel::Serializer
end
def url
if playback_options.v0
if v0_url?
object.stdout_frames_url
else
object.data_url
@ -39,8 +39,8 @@ class AsciicastSerializer < ActiveModel::Serializer
private
def playback_options
@options[:playback_options]
def v0_url?
!!@options[:v0]
end
end

Loading…
Cancel
Save