jekyll-aspect-ratio 0.0.0 → 0.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '058a9c7412bba98f8aeea49537c2df702f44d8420d10c6a49953ba4557a6dc6f'
-  data.tar.gz: b573af089a4a2e3fd23fbb08d44dcf5b5cd7f56369e7e349af2bef5542f32b58
+  metadata.gz: fd37b32b0b923b0c43e4fdc08a9496743d449898162d6c763c24268f3e720e01
+  data.tar.gz: 42fae0e5c906dfe525a6710a926fee8b9f751df7cd8e86a8636902aa23c42174
 SHA512:
-  metadata.gz: cba3c22da811c376dbfc6b8648889c2f3cd56f85fe12108f53a219455d3d13e97e8982fb383f75f16d2cb4d1b26854404b1ba26ac73fafa824b231ef89b36c97
-  data.tar.gz: 64c586c4d3e58ac7e98da9ef1bd06ac5f2a854eedc78f5e84f421a996184f211d04aea67812483f7e487c6b03415659a01260909f6bd5010d1b5366eb2f57ff4
+  metadata.gz: 03d2c8c683e19481c6a2238eae8c92ffbde451477b1f2ecc670c47f25d3820d2e324dca6bd35f23c7f42eb6e05641661b8435e5d35adb66f7e554d247858c8b2
+  data.tar.gz: ad954441f040a353146e6b6e6d1d578c1e8d34992f3b09b813b1ecfe52b1d01de91f0f16393aae9786a0d66d298a15a5312744dcc3c4423d5642642edc22dcfe

data/lib/jekyll-aspect-ratio.rb

@@ -3,34 +3,47 @@ require 'mini_exiftool'
 
 class AspectRatio < Liquid::Tag
 
-  def initialize(tagName, input, tokens)
+  def initialize(tagName, content, tokens)
     super
-    @input = input
+    @input = content
   end
 
   def render(context)
-    filepath = context[@input]
-    # Strip off the first '/', as that confuses things with relative filepaths
+    # If this is a parameter name (from {% assign x = "path" %})
+    # then we need to pass that in as a key to context to get its value
+    if context.key?(@input)
+        filepath = context[@input]
+    else
+        filepath = @input
+    end
+
+    # Strip leading spaces and quotes
+    filepath = filepath.strip.tr('"', '').tr('\'', '')
+
+    # Strip off the first '/', if there is one
+    #  as that confuses things with relative filepaths
     if filepath[0] == "/"
         filepath = filepath[1..-1]
     end
+
+    # MiniExfitool doesn't like relative paths
     abspath = File.expand_path(filepath)
-    output = 0
 
     if (File.file?(abspath))
         mediaObj = MiniExiftool.new abspath
         height = mediaObj.image_height
         width = mediaObj.image_width
 
-        # TODO is it always giving me the longer side as width? if so I could pass in a flag for if it's a horizontal or vertical video?
-        if mediaObj.rotation == 90 or mediaObj.rotation == 180
+        # If the video is rotated, we need to swap width and height
+        if mediaObj.rotation == 90 or mediaObj.rotation == 270
             temp = height
             height = width
             width = temp
         end
 
+        # Avoid division by zero errors
         if height != 0
-            output = width.to_f/height.to_f
+            output = (width.to_f/height.to_f).truncate(2)
         end
     end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-aspect-ratio
 version: !ruby/object:Gem::Version
-  version: 0.0.0
+  version: 0.0.1
 platform: ruby
 authors:
 - Beth Crane
@@ -10,12 +10,13 @@ bindir: bin
 cert_chain: []
 date: 2020-05-10 00:00:00.000000000 Z
 dependencies: []
-description: "Jekyll-Aspect-Ratio enables the use of a custom Liquid tag {% aspectratio
-  filepath %}, to find the aspect ratio of an image or video file. \n\nExample use
-  cases:\n- Specify the flex-grow value of an item so that a series of items of different
-  aspect ratios can fill up one row, all being the same height\n- Sort or separate
-  items by aspect ratio, to display horizontal vs vertical videos together (aspect
-  ratio < 1 is a portrait item, >1 is landscape)\n"
+description: 'This plugin enables the use of a custom Liquid tag {% aspectratio filepath
+  %}, to find the aspect ratio of an image or video file. This can be used for e.g.
+  specifying the flex-grow value of an item to display items of different aspect ratios
+  on one line at the same height, or to separate out horizontal or vertical videos
+  (aspect ratio > or < than 1).
+
+  '
 email: hello@bethcrane.com
 executables: []
 extensions: []