i18n_proofreading 0.9.0 → 0.9.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 578d960a613d57612657b905fb64b4625dd665e19ef02d90c61493d032058372
-  data.tar.gz: b8653d8cee11804f5765391a55e8b055d415ccb0681d7def0b144c1987f6410f
+  metadata.gz: 442bbc0db3a43634199ae4ad02e0b4c93bfe4b0278f5c4bf4236793acfd9006d
+  data.tar.gz: b5adfadeac507e64888ea36412173daec3ec8b8b0067fe1c056de8b1c328c5b7
 SHA512:
-  metadata.gz: c85beac359797b0eb32321adcce88f0859e4b30943754e9ff4fa106168ccc4891ccf678e31b0f908d80f1b42183a5696ad7371dddd529378df49c11d9d2016f4
-  data.tar.gz: 04e0f42e8a0c1991094fc13b84530cb39876073f1f9e615d6d1c8df2f11e398c867ef78f66f46570bd9e5c51b311f00c2d0df1a180206de533ac25e94a3b0acf
+  metadata.gz: ad016e5b43be45f388e0f0c11956dbcfaed05d2702f1765d598a2c29e4ee2af8a539bb853c8d8ef8d0dd0121273c64e64353611b8acb494348d10e24188aa5d4
+  data.tar.gz: '08506d993c5b691a33ab9dc327fc2ca84814723cd289bf014ff164181f4fa85d882901286ccc0816aa26363aba371893090ee4024845cb7e0431c57bb2483dd4'

data/CHANGELOG.md

@@ -2,6 +2,24 @@
 
 ## [Unreleased]
 
+## [0.9.2]
+
+- Make the suggestion popover **full-screen on mobile** (≤ 480px wide): edge to
+  edge with no rounded corners, instead of a floating dialog — no bottom sheet
+  and no animation. The widget's inputs render at 16px on small screens so iOS
+  Safari no longer zooms the page when one gains focus, the action row pads
+  itself past the iPhone home indicator (`env(safe-area-inset-bottom)`), and the
+  panel's scrolling is contained (`overscroll-behavior`) so reaching its end
+  never scrolls the page behind it. CSS only; desktop is unchanged.
+
+## [0.9.1]
+
+- Add Minitest **system tests** that drive the widget in headless Chrome
+  (Capybara + Selenium): entering suggest mode, marker stripping, opening the
+  popover, submitting a suggestion, and Escape to exit. Run with
+  `rake test:system`; `rake test` stays browser-free. Dev/test only — no runtime
+  change and no new runtime dependencies.
+
 ## [0.9.0]
 
 - **Renamed the gem `i18n_feedback` → `i18n_proofreading`.** The name now says

data/README.md

@@ -2,7 +2,7 @@
 
 [![Gem Version](https://img.shields.io/gem/v/i18n_proofreading)](https://rubygems.org/gems/i18n_proofreading)
 [![Downloads](https://img.shields.io/gem/dt/i18n_proofreading)](https://rubygems.org/gems/i18n_proofreading)
-[![CI](https://github.com/yshmarov/i18n-proofreading/actions/workflows/ci.yml/badge.svg)](https://github.com/yshmarov/i18n-proofreading/actions/workflows/ci.yml)
+[![CI](https://github.com/yshmarov/i18n_proofreading/actions/workflows/ci.yml/badge.svg)](https://github.com/yshmarov/i18n_proofreading/actions/workflows/ci.yml)
 [![License: MIT](https://img.shields.io/badge/license-MIT-blue)](MIT-LICENSE)
 
 In-context translation proofreading for Rails.
@@ -14,9 +14,9 @@ It is meant for development and staging, never production.
 
 ![Click any string in your running app and suggest a better wording](i18n-proofreading-demo.gif)
 
-<video src="https://github.com/yshmarov/i18n-proofreading/raw/main/i18n-proofreading-demo-640-high.mp4" controls muted playsinline width="640">
+<video src="https://github.com/yshmarov/i18n_proofreading/raw/main/i18n-proofreading-demo-640-high.mp4" controls muted playsinline width="640">
   Your browser can't play this video —
-  <a href="https://github.com/yshmarov/i18n-proofreading/raw/main/i18n-proofreading-demo-640-high.mp4">download it here</a>.
+  <a href="https://github.com/yshmarov/i18n_proofreading/raw/main/i18n-proofreading-demo-640-high.mp4">download it here</a>.
 </video>
 
 - **Zero UI dependencies.** The widget is plain JavaScript and styles itself. No

data/Rakefile

@@ -6,9 +6,19 @@ require 'bundler/setup'
 require 'bundler/gem_tasks'
 
 require 'rake/testtask'
+
+# Unit/integration tests — no browser, runs on every Ruby x Rails combo.
 Rake::TestTask.new(:test) do |t|
   t.libs << 'test'
-  t.pattern = 'test/**/*_test.rb'
+  t.test_files = FileList['test/**/*_test.rb'].exclude('test/system/**/*')
+  t.warning = false
+  t.verbose = false
+end
+
+# System tests — drive the widget in headless Chrome (needs a browser).
+Rake::TestTask.new('test:system') do |t|
+  t.libs << 'test'
+  t.test_files = FileList['test/system/**/*_test.rb']
   t.warning = false
   t.verbose = false
 end

data/lib/i18n_proofreading/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module I18nProofreading
-  VERSION = '0.9.0'
+  VERSION = '0.9.2'
 end

data/lib/i18n_proofreading/widget.js

@@ -394,6 +394,7 @@
       "  align-items: center; justify-content: center; padding: 16px;",
       "  background: rgba(0,0,0,.45); cursor: default; }",
       ".i18np-panel { width: 28rem; max-width: 100%; max-height: 90vh; overflow: auto;",
+      "  overscroll-behavior: contain;",
       "  background: #fff; color: #111; border-radius: 12px; padding: 20px;",
       "  box-shadow: 0 20px 60px rgba(0,0,0,.35); font: 14px/1.5 system-ui, sans-serif; }",
       ".i18np-panel > * + * { margin-top: 14px; }",
@@ -437,6 +438,21 @@
       "  .i18np-error { color: #f87171; }",
       "  .i18np-btn-ghost { color: #f4f4f5; }",
       "}",
+
+      // Full-screen popover on small screens — no bottom sheet, no animation.
+      // Selectors are at least as specific as the base rules and this block
+      // comes last, so the mobile values win.
+      "@media (max-width: 480px) {",
+      "  .i18np-overlay { padding: 0; }",
+      "  .i18np-overlay .i18np-panel { position: fixed; top: 0; right: 0; bottom: 0; left: 0;",
+      "    width: 100%; height: 100dvh; max-height: 100dvh; border-radius: 0; margin: 0; }",
+      // 16px inputs stop iOS Safari from zooming the page on focus.
+      "  .i18np-panel input.i18np-input, .i18np-panel textarea.i18np-input,",
+      "  .i18np-panel select { font-size: 16px; }",
+      // Keep the action row clear of the iPhone home indicator (its own
+      // padding-bottom is 0; the panel padding sits outside the row).
+      "  .i18np-panel .i18np-actions { padding-bottom: calc(0px + env(safe-area-inset-bottom)); }",
+      "}",
     ].join("\n");
     document.head.appendChild(style);
   }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: i18n_proofreading
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.9.2
 platform: ruby
 authors:
 - Yaroslav Shmarov
@@ -83,12 +83,12 @@ files:
 - lib/i18n_proofreading/version.rb
 - lib/i18n_proofreading/widget.js
 - lib/i18n_proofreading/widget.rb
-homepage: https://github.com/yshmarov/i18n-proofreading
+homepage: https://github.com/yshmarov/i18n_proofreading
 licenses:
 - MIT
 metadata:
-  source_code_uri: https://github.com/yshmarov/i18n-proofreading
-  changelog_uri: https://github.com/yshmarov/i18n-proofreading/blob/main/CHANGELOG.md
+  source_code_uri: https://github.com/yshmarov/i18n_proofreading
+  changelog_uri: https://github.com/yshmarov/i18n_proofreading/blob/main/CHANGELOG.md
   rubygems_mfa_required: 'true'
 rdoc_options: []
 require_paths:
@@ -104,7 +104,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.6
+rubygems_version: 3.6.9
 specification_version: 4
 summary: 'In-context i18n proofreading for Rails: click any translated string and
   suggest a fix.'