diff options
author | Rémy Coutable <remy@rymai.me> | 2016-07-12 20:31:20 +0300 |
---|---|---|
committer | Rémy Coutable <remy@rymai.me> | 2016-07-19 19:51:09 +0300 |
commit | 98e540532cc2706e4cdc027bd2acb8406e954ddc (patch) | |
tree | 92f4157e6a50ae72ead3ccf60cf6a035fefc7f3b /lib/banzai | |
parent | 29ea8d09e05d749f0b6a784311aea942f7d2d0f1 (diff) |
Use a more powerful query to match videos in img tags
Also, always add a link to download videos since video playback is
tricky. Also, it solves the issue with email client not supporting
videos.
Signed-off-by: Rémy Coutable <remy@rymai.me>
Diffstat (limited to 'lib/banzai')
-rw-r--r-- | lib/banzai/filter/video_link_filter.rb | 43 | ||||
-rw-r--r-- | lib/banzai/pipeline/gfm_pipeline.rb | 2 |
2 files changed, 35 insertions, 10 deletions
diff --git a/lib/banzai/filter/video_link_filter.rb b/lib/banzai/filter/video_link_filter.rb index a8316e72a68..4dfbff8ec86 100644 --- a/lib/banzai/filter/video_link_filter.rb +++ b/lib/banzai/filter/video_link_filter.rb @@ -8,8 +8,8 @@ module Banzai include ActionView::Context def call - doc.search('img').each do |el| - el.replace(video_tag(doc, el)) if video?(el) + doc.xpath(query).each do |el| + el.replace(video_node(doc, el)) end doc @@ -17,19 +17,44 @@ module Banzai private - def video?(element) - extension = File.extname(element.attribute('src').value).delete('.') - UploaderHelper::VIDEO_EXT.include?(extension) + def query + @query ||= begin + src_query = UploaderHelper::VIDEO_EXT.map do |ext| + "'.#{ext}' = substring(@src, string-length(@src) - #{ext.size})" + end + + "descendant-or-self::img[not(ancestor::a) and (#{src_query.join(' or ')})]" + end end # Return a video tag Nokogiri node # def video_node(doc, element) - doc.document.create_element( + container = doc.document.create_element( + 'div', + class: 'video-container' + ) + + video = doc.document.create_element( 'video', - src: element.attribute('src').value, - class: 'video-js', - controls: true) + src: element['src'], + class: 'video-js vjs-sublime-skin', + controls: true, + "data-setup": '{}') + + link = doc.document.create_element( + 'a', + element['title'] || element['alt'], + href: element['src'], + target: '_blank', + title: "Downlad '#{element['title'] || element['alt']}'") + download_paragraph = doc.document.create_element('p') + download_paragraph.children = link + + container.add_child(video) + container.add_child(download_paragraph) + + container end end diff --git a/lib/banzai/pipeline/gfm_pipeline.rb b/lib/banzai/pipeline/gfm_pipeline.rb index d9edca7046c..8d94b199c66 100644 --- a/lib/banzai/pipeline/gfm_pipeline.rb +++ b/lib/banzai/pipeline/gfm_pipeline.rb @@ -7,8 +7,8 @@ module Banzai Filter::SanitizationFilter, Filter::UploadLinkFilter, - Filter::ImageLinkFilter, Filter::VideoLinkFilter, + Filter::ImageLinkFilter, Filter::EmojiFilter, Filter::TableOfContentsFilter, Filter::AutolinkFilter, |