trmnl_preview 0.18.0 → 0.18.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 53759b5b952609676a6313c40932bf0aa015ba665044fa2dae6d6dd25e96dfc1
-  data.tar.gz: 906e518e4193951665996a4ed7a49ff07e94cfec35269412f6fe2a172b1ae4c4
+  metadata.gz: 32924757751f8d15ad7979409c03f18906506af03558a90d27bfd122f0a44d0c
+  data.tar.gz: 5cba7d9f2cbd67868316826218a1237efafafdc0f9967b3670ef538859c11f56
 SHA512:
-  metadata.gz: 51e39569a0ade12fc7f435176438ce3319adbaac3f613a4b36f441b3c01e7a89917f5df96395a62ad3a7a1dce3efb1e021847e1178afe84c47cdabfbec9b6f9d
-  data.tar.gz: a7b3b71f8fb839bca061a9b2083622d076f9290a6f4f128cacaea34b36cc6e0833ea3ae7876b13358dc7670c4fb71a6b973564866614474893efdd5715f898f8
+  metadata.gz: b9b8692c362797cd9df4ee23cf7a081051ec9f6ea18b28c721426fc58860231a5b02541c0a6823a8a940a33eb1a9e81f8fdcba9f3d43ad43f6871a880706fb58
+  data.tar.gz: 1351106bc604919f20f0067a558eab0da46c1f08bf1cf7fd3ccce69db42dcac81136499d295629cfc64777b3f40586332701c312707b22348f56927611232fe0

data/CHANGELOG.md

@@ -1,6 +1,11 @@
 
 # Changelog
 
+## 0.18.1
+
+- Pages narrower than a Firefox window render, such as a 480x800 TRMNL OG in portrait, in `serve`, `build` and `test`. trmnlp sets the page's viewport with WebDriver BiDi, as TRMNL does, instead of resizing the window, which Firefox will not size under about 500px. (#178)
+- `it_behaves_like 'a publishable recipe'` also draws every view on the TRMNL OG (2-bit) in portrait. (#179)
+
 ## 0.18.0
 
 - `it_behaves_like 'a publishable recipe'` checks what a recipe should hold before it is published: every view draws without overflow or page errors on the TRMNL OG (1-bit and 2-bit) and the TRMNL X (landscape and portrait), and the transform runs without error within TRMNL's limits. `trmnlp init`'s starter spec uses it. (#176)

data/README.md

@@ -505,7 +505,7 @@ end
 - `mocks:` answer every request the run makes: polling urls, and the transform's own requests in any language, HTTPS included. Keys are urls, with `*` wildcards, a Regexp, or a method first (`'POST https://...'`). Values are `{ json:, body:, status:, headers:, delay:, body_delay:, advance_clock:, error: :reset }` (times in seconds: `body_delay:` sends the headers first and the body later, `advance_clock:` moves the transform's clock on when it answers), a string body, a lambda that takes the request, or an array of answers used in order. An unmocked request gets a 599. `requests` lists every request with its `status`, and `aborted: true` for one the transform gave up on before its answer arrived; `duration_ms` is how long the transform ran.
 - `now:` starts every clock: Liquid's, the markup's scripts', and the transform's, through libfaketime (`brew install libfaketime` or `apt-get install libfaketime`; already in the Docker image). On macOS the interpreter must come from brew, mise or similar, since macOS will not hand libfaketime to its own `/usr/bin` binaries.
 - `trmnlp test --report report` also writes `report/index.html` and `report/report.json`: every example with each screen it rendered (with a switch that outlines every drawn box, and the page's problems) and each transform it ran (time, memory, requests). Under GitHub Actions the counts and failures go to the run's summary too.
-- `it_behaves_like 'a publishable recipe'` is what a recipe should hold before it is published: every view draws without overflow or page errors on the TRMNL OG (1-bit and 2-bit) and the TRMNL X (landscape and portrait), and the transform runs without error within TRMNL's limits. It uses the group's `mocks` and `custom_fields` when the group defines them; `screens: [{ device: 'kobo_libra_2' }, ...]` draws on other devices.
+- `it_behaves_like 'a publishable recipe'` is what a recipe should hold before it is published: every view draws without overflow or page errors on the TRMNL OG (1-bit, and 2-bit in landscape and portrait) and the TRMNL X (landscape and portrait), and the transform runs without error within TRMNL's limits. It uses the group's `mocks` and `custom_fields` when the group defines them; `screens: [{ device: 'kobo_libra_2' }, ...]` draws on other devices.
 - `match_snapshot` stores a missing snapshot under `tests/snapshots/<os>/` and fails one on CI; `trmnlp test --update` rewrites them. Fonts render differently per operating system, so run tests in the Docker image when CI should share your snapshots. `fit_image_size_limit` checks the PNG against the model's limit.
 
 ## Development

data/lib/trmnlp/firefox_driver.rb

@@ -15,7 +15,7 @@ module TRMNLP
     end
 
     def options
-      Selenium::WebDriver::Firefox::Options.new.tap do |opts|
+      Selenium::WebDriver::Firefox::Options.new(web_socket_url: true).tap do |opts|
         opts.add_argument('--headless')
         opts.add_argument('--disable-web-security')
       end

data/lib/trmnlp/screenshot.rb

@@ -68,44 +68,29 @@ module TRMNLP
     end
 
     def resize(driver, width, height)
-      apply_window_size(driver, width, height)
+      set_viewport(driver, width, height)
       wait_for_viewport(driver, width, height)
     end
 
-    def apply_window_size(driver, width, height)
-      borders = driver.execute_script(<<~JS)
-        return {
-          width: window.outerWidth - window.innerWidth,
-          height: window.outerHeight - window.innerHeight
-        }
-      JS
-
-      dim = Selenium::WebDriver::Dimension.new(width + borders['width'], height + borders['height'])
-      driver.manage.window.size = dim
+    # The page's viewport, not the window: Firefox will not size a window under about 500px, as an OG in portrait.
+    def set_viewport(driver, width, height)
+      driver.bidi.send_cmd('browsingContext.setViewport', context: driver.window_handle, viewport: { width:, height: })
     end
 
     # NOTE: A cold Firefox — e.g. the first render after the container boots —
-    # applies a window resize lazily. The old fixed sleep raced that reflow and
+    # applies a resize lazily. The old fixed sleep raced that reflow and
     # clipped the first screenshot short (800x433 instead of 800x480). Poll the
-    # real viewport instead, re-applying the size until it lands. A width the
-    # browser refuses to honour (below its ~500px window minimum) never settles;
-    # that surfaces as a clear RenderError rather than an opaque, retried timeout.
+    # real viewport instead, re-applying the size until it lands.
     def wait_for_viewport(driver, width, height)
       Selenium::WebDriver::Wait.new(timeout: @viewport_timeout, interval: 0.1).until do
         next true if viewport(driver) == [width, height]
 
-        apply_window_size(driver, width, height)
+        set_viewport(driver, width, height)
         false
       end
     rescue Selenium::WebDriver::Error::TimeoutError
-      raise RenderError, viewport_clamp_message(driver, width, height)
-    end
-
-    def viewport_clamp_message(driver, width, height)
       actual_width, actual_height = viewport(driver)
-      "Could not render at #{width}x#{height}: the browser clamped the viewport " \
-        "to #{actual_width}x#{actual_height}. PNG rendering needs a width of " \
-        'roughly 500px or more — headless Firefox will not size its window narrower.'
+      raise RenderError, "Could not render at #{width}x#{height}: the viewport stayed #{actual_width}x#{actual_height}"
     end
 
     def viewport(driver)

data/lib/trmnlp/testing/publishable_recipe.rb

@@ -2,9 +2,10 @@
 
 module TRMNLP
   module Testing
-    # The screens a recipe is drawn on: TRMNL's own devices, and a portrait screen.
+    # The screens a recipe is drawn on: TRMNL's own devices, in landscape and portrait.
     PUBLISHABLE_RECIPE_SCREENS = [
-      { device: 'og_png' }, { device: 'og_plus' }, { device: 'v2' }, { device: 'v2', orientation: :portrait }
+      { device: 'og_png' }, { device: 'og_plus' }, { device: 'og_plus', orientation: :portrait },
+      { device: 'v2' }, { device: 'v2', orientation: :portrait }
     ].freeze
     PUBLISHABLE_RECIPE_VIEWS = %w[full half_horizontal half_vertical quadrant].freeze
   end

data/lib/trmnlp/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module TRMNLP
-  VERSION = '0.18.0'
+  VERSION = '0.18.1'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: trmnl_preview
 version: !ruby/object:Gem::Version
-  version: 0.18.0
+  version: 0.18.1
 platform: ruby
 authors:
 - Rockwell Schrock