mjml-rb 0.2.4 → 0.2.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4bb6667d87d9376e60199a47e987e10cfb496be78c4bd549f71fa049c2618aff
-  data.tar.gz: 98f63a171ea6a2d5b1dd6212c86a13cdd2eabf71c1ab9f12d15f8c83a04e30d2
+  metadata.gz: f213ff60765d24bebf6bd96742274aa2eed792452c0176b8de29bfbdbc8506cb
+  data.tar.gz: 7859582b4ecf32f4f5018cbea4929169ebf96089fc3ae2c973672c015e190ed3
 SHA512:
-  metadata.gz: 33c9d050ef8ebd560b92a513391957c9fd2ebea43023d3977c5c06fe154bc7eef1bf6646abbf555dcbf6f53d4900461cceb89daaf0e3b2be36d3e08cec445b38
-  data.tar.gz: 839a5ddbe4448b2c86b9b015d59468e8f2553a78432cf6371bdb6d061c850568850bde0316fd9be6ac548a0bc17715900d233cd1538cff03785383f29d960240
+  metadata.gz: 28114bad8d00b9260fb49945b9383b18be09b588ba6474458d20b2b1b4f0986ca44cc8a20487e230931e4985e21f135878cbd938aff5f95fb156cd8af623558c
+  data.tar.gz: 6992e280ea3c63a02b3715ffaf53e0944d123a1a17be5e3237fedc6a4a66baf44863ba14a822a1d931e0beddba8ded6d14d02271957b8afcdee4ab78df9dbda0

data/README.md

@@ -61,14 +61,14 @@ The table below tracks current JS-to-Ruby migration status for MJML components i
 | `mj-navbar` | migrated | Implemented in `navbar.rb`, including `base-url` propagation and breakpoint-aware hamburger CSS. |
 | `mj-navbar-link` | migrated | Implemented in `navbar.rb` as an ending-tag navbar child component. |
 | `mj-raw` | migrated | Implemented in `raw.rb`, including head insertion and top-level `position="file-start"` output before the doctype. |
-| `mj-head` | partial | Core tags such as `mj-title`, `mj-preview`, `mj-style`, `mj-font`, and `mj-attributes` are supported. |
+| `mj-head` | migrated | Implemented in `head.rb` and dispatches supported head children through component handlers. |
 | `mj-attributes` | migrated | Implemented in `attributes.rb`, including npm-style `mj-class` descendant defaults. |
-| `mj-all` | partial | Supported through `mj-attributes`. |
+| `mj-all` | migrated | Implemented through `attributes.rb` with npm-style global default attribute precedence. |
 | `mj-class` | migrated | Supported through `attributes.rb`, including nested per-tag descendant defaults. |
-| `mj-title` | partial | Supported through head context. |
-| `mj-preview` | partial | Supported through head context. |
-| `mj-style` | partial | Supported, including inline CSS application. |
-| `mj-font` | partial | Supported for font link injection. |
+| `mj-title` | migrated | Implemented in `head.rb`. |
+| `mj-preview` | migrated | Implemented in `head.rb`. |
+| `mj-style` | migrated | Implemented in `head.rb`, including inline-style registration. |
+| `mj-font` | migrated | Implemented in `head.rb`. |
 | `mj-carousel` | not migrated | Declared in dependency rules but no renderer implementation yet. |
 | `mj-carousel-image` | not migrated | Declared in dependency rules but no renderer implementation yet. |
 | `mj-breakpoint` | migrated | Supported in `mj-head` and used to control desktop column media-query widths. |
@@ -76,6 +76,6 @@ The table below tracks current JS-to-Ruby migration status for MJML components i
 | `mj-selector` | migrated | Supported as the selector container for `mj-html-attribute` rules. |
 | `mj-html-attribute` | migrated | Supported for injecting custom HTML attributes into matched rendered nodes. |
 
-Remaining top-level migration work is mainly `mj-carousel` and `mj-carousel-image`, plus bringing the remaining renderer-owned partial implementations closer to upstream JS behavior.
+Remaining top-level migration work is mainly `mj-carousel` and `mj-carousel-image`.
 
 TODO: `mj-html-attributes` currently uses `Nokogiri` to parse rendered HTML and apply CSS-selector-based attribute injections. Rewrite this path to avoid `Nokogiri` if we want to keep the runtime dependency surface minimal.

data/lib/mjml-rb/components/breakpoint.rb

@@ -20,6 +20,11 @@ module MjmlRb
       def render(tag_name:, node:, context:, attrs:, parent:)
         ""
       end
+
+      def handle_head(node, context)
+        width = node.attributes["width"].to_s.strip
+        context[:breakpoint] = width unless width.empty?
+      end
     end
   end
 end

data/lib/mjml-rb/components/head.rb

@@ -0,0 +1,59 @@
+require_relative "base"
+
+module MjmlRb
+  module Components
+    class Head < Base
+      TAGS = %w[mj-head mj-title mj-preview mj-style mj-font].freeze
+
+      ALLOWED_ATTRIBUTES = {
+        "mj-style" => {
+          "inline" => "string"
+        },
+        "mj-font" => {
+          "name" => "string",
+          "href" => "string"
+        }
+      }.freeze
+
+      class << self
+        def allowed_attributes_for(tag_name)
+          ALLOWED_ATTRIBUTES[tag_name] || {}
+        end
+
+        def allowed_attributes
+          {}
+        end
+      end
+
+      def tags
+        TAGS
+      end
+
+      def render(tag_name:, node:, context:, attrs:, parent:)
+        ""
+      end
+
+      def handle_head(node, context)
+        case node.tag_name
+        when "mj-head"
+          node.element_children.each do |child|
+            component = renderer.send(:component_for, child.tag_name)
+            component.handle_head(child, context) if component&.respond_to?(:handle_head)
+          end
+        when "mj-title"
+          context[:title] = raw_inner(node).strip
+        when "mj-preview"
+          context[:preview] = raw_inner(node).strip
+        when "mj-style"
+          css = raw_inner(node)
+          context[:head_styles] << css
+          context[:inline_styles] << css if node.attributes["inline"] == "inline"
+        when "mj-font"
+          name = node.attributes["name"]
+          href = node.attributes["href"]
+          context[:fonts][name] = href if name && href
+        end
+      end
+    end
+  end
+end

data/lib/mjml-rb/components/html_attributes.rb

@@ -3,7 +3,7 @@ require_relative "base"
 module MjmlRb
   module Components
     class HtmlAttributes < Base
-      TAGS = %w[mj-selector mj-html-attribute].freeze
+      TAGS = %w[mj-html-attributes mj-selector mj-html-attribute].freeze
 
       ALLOWED_ATTRIBUTES = {
         "mj-selector" => {
@@ -27,6 +27,28 @@ module MjmlRb
       def render(tag_name:, node:, context:, attrs:, parent:)
         render_children(node, context, parent: parent)
       end
+
+      def handle_head(node, context)
+        node.element_children.each do |selector|
+          next unless selector.tag_name == "mj-selector"
+
+          path = selector.attributes["path"].to_s.strip
+          next if path.empty?
+
+          custom_attrs = selector.element_children.each_with_object({}) do |child, memo|
+            next unless child.tag_name == "mj-html-attribute"
+
+            name = child.attributes["name"].to_s.strip
+            next if name.empty?
+
+            memo[name] = child.text_content
+          end
+          next if custom_attrs.empty?
+
+          context[:html_attributes][path] ||= {}
+          context[:html_attributes][path].merge!(custom_attrs)
+        end
+      end
     end
   end
 end

data/lib/mjml-rb/components/raw.rb

@@ -16,6 +16,10 @@ module MjmlRb
       def render(tag_name:, node:, context:, attrs:, parent:)
         raw_inner(node)
       end
+
+      def handle_head(node, context)
+        context[:head_raw] << raw_inner(node)
+      end
     end
   end
 end

data/lib/mjml-rb/renderer.rb

@@ -5,6 +5,7 @@ require_relative "components/attributes"
 require_relative "components/body"
 require_relative "components/breakpoint"
 require_relative "components/button"
+require_relative "components/head"
 require_relative "components/hero"
 require_relative "components/image"
 require_relative "components/navbar"
@@ -71,76 +72,13 @@ module MjmlRb
       return context unless head
 
       head.element_children.each do |node|
-        case node.tag_name
-        when "mj-title"
-          context[:title] = node.text_content.strip
-        when "mj-preview"
-          context[:preview] = node.text_content.strip
-        when "mj-style"
-          context[:head_styles] << node.text_content
-          context[:inline_styles] << node.text_content if node.attributes["inline"] == "inline"
-        when "mj-font"
-          name = node.attributes["name"]
-          href = node.attributes["href"]
-          context[:fonts][name] = href if name && href
-        when "mj-breakpoint"
-          width = node.attributes["width"].to_s.strip
-          context[:breakpoint] = width unless width.empty?
-        when "mj-attributes"
-          if (component = component_for(node.tag_name)) && component.respond_to?(:handle_head)
-            component.handle_head(node, context)
-          else
-            absorb_attribute_node(node, context)
-          end
-        when "mj-html-attributes"
-          absorb_html_attributes_node(node, context)
-        when "mj-raw"
-          context[:head_raw] << raw_inner(node)
-        end
+        component = component_for(node.tag_name)
+        component.handle_head(node, context) if component&.respond_to?(:handle_head)
       end
 
       context
     end
 
-    def absorb_attribute_node(attributes_node, context)
-      attributes_node.element_children.each do |child|
-        case child.tag_name
-        when "mj-all"
-          context[:global_defaults].merge!(child.attributes)
-        when "mj-class"
-          name = child.attributes["name"]
-          next unless name
-
-          context[:classes][name] = child.attributes.reject { |k, _| k == "name" }
-        else
-          context[:tag_defaults][child.tag_name] ||= {}
-          context[:tag_defaults][child.tag_name].merge!(child.attributes)
-        end
-      end
-    end
-
-    def absorb_html_attributes_node(html_attributes_node, context)
-      html_attributes_node.element_children.each do |selector|
-        next unless selector.tag_name == "mj-selector"
-
-        path = selector.attributes["path"].to_s.strip
-        next if path.empty?
-
-        custom_attrs = selector.element_children.each_with_object({}) do |child, memo|
-          next unless child.tag_name == "mj-html-attribute"
-
-          name = child.attributes["name"].to_s.strip
-          next if name.empty?
-
-          memo[name] = child.text_content
-        end
-        next if custom_attrs.empty?
-
-        context[:html_attributes][path] ||= {}
-        context[:html_attributes][path].merge!(custom_attrs)
-      end
-    end
-
     def build_html_document(content, context)
       title = context[:title].to_s
       preview = context[:preview]
@@ -433,6 +371,7 @@ module MjmlRb
         registry = {}
         # Register component classes here as they are implemented.
         register_component(registry, Components::Body.new(self))
+        register_component(registry, Components::Head.new(self))
         register_component(registry, Components::Attributes.new(self))
         register_component(registry, Components::Breakpoint.new(self))
         register_component(registry, Components::Accordion.new(self))
@@ -443,6 +382,7 @@ module MjmlRb
         register_component(registry, Components::Raw.new(self))
         register_component(registry, Components::Text.new(self))
         register_component(registry, Components::Divider.new(self))
+        register_component(registry, Components::HtmlAttributes.new(self))
         register_component(registry, Components::Table.new(self))
         register_component(registry, Components::Social.new(self))
         register_component(registry, Components::Section.new(self))

data/lib/mjml-rb/version.rb

@@ -1,3 +1,3 @@
 module MjmlRb
-  VERSION = "0.2.4".freeze
+  VERSION = "0.2.6".freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mjml-rb
 version: !ruby/object:Gem::Version
-  version: 0.2.4
+  version: 0.2.6
 platform: ruby
 authors:
 - Andrei Andriichuk
@@ -61,6 +61,7 @@ files:
 - lib/mjml-rb/components/button.rb
 - lib/mjml-rb/components/column.rb
 - lib/mjml-rb/components/divider.rb
+- lib/mjml-rb/components/head.rb
 - lib/mjml-rb/components/hero.rb
 - lib/mjml-rb/components/html_attributes.rb
 - lib/mjml-rb/components/image.rb