view_component 2.52.0 → 2.54.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1eb8c14aae22e2977db7cde77cd64f9ff1d69f14c0f2c95b6b2ebf0fabf67f5a
-  data.tar.gz: fe2c729ae013bf1dd2effb17143acfef756e915ea773dcb7eef84fd329c7e002
+  metadata.gz: f9cab3c05c054f173b248b903912dfb432eeec09b2d3d42f6fd602a7565acd2d
+  data.tar.gz: 78b70e8ea1306fb635978dbe2a86cece2618247bb0079798e5a886f2210dfed1
 SHA512:
-  metadata.gz: 0e72fac02dd38cd39cfa7fe45bc57617a8088d2016add0c769df9075393dc8fde66ea44c96d466019354585da3754b1db2099e9207a46418b011e6ef8765b816
-  data.tar.gz: 6ec82473f583cd484e9ee4042683d7d605131279ef855a199f1a89292967dd971c6ed8a43f8ad7fc9d519bae02fb4f2e5cf8d14f4876ffa1b524ad41ce25ea20
+  metadata.gz: 653d1c627f3fb52677e47516472fec508c562359e3beac3defa43d172d3e793359207df638f352f1c1cc6a876a0f3c3fdc4c7ff4eefdc1907c20a161e9359d83
+  data.tar.gz: 25bc7a21cde84c3311d9482cdb2ccccf821ce2c104492506a999e81f8cd9d9a5b3b3c19f3b417dc3f250787adf18fbe9f9d9ff5589e897fc199e26e50e4b0efa

data/app/controllers/concerns/view_component/preview_actions.rb

@@ -0,0 +1,97 @@
+# frozen_string_literal: true
+
+module ViewComponent
+  module PreviewActions
+    extend ActiveSupport::Concern
+
+    included do
+      prepend_view_path File.expand_path("../../../views", __dir__)
+
+      around_action :set_locale, only: :previews
+      before_action :find_preview, only: :previews
+      before_action :require_local!, unless: :show_previews?
+
+      if respond_to?(:content_security_policy)
+        content_security_policy(false)
+      end
+    end
+
+    def index
+      @previews = ViewComponent::Preview.all
+      @page_title = "Component Previews"
+      render "view_components/index", **determine_layout
+    end
+
+    def previews
+      if params[:path] == @preview.preview_name
+        @page_title = "Component Previews for #{@preview.preview_name}"
+        render "view_components/previews", **determine_layout
+      else
+        prepend_application_view_paths
+        prepend_preview_examples_view_path
+        @example_name = File.basename(params[:path])
+        @render_args = @preview.render_args(@example_name, params: params.permit!)
+        layout = determine_layout(@render_args[:layout], prepend_views: false)[:layout]
+        locals = @render_args[:locals]
+        opts = {}
+        opts[:layout] = layout if layout.present? || layout == false
+        opts[:locals] = locals if locals.present?
+        render "view_components/preview", opts # rubocop:disable GitHub/RailsControllerRenderLiteral
+      end
+    end
+
+    private
+
+    def default_preview_layout # :doc:
+      ViewComponent::Base.default_preview_layout
+    end
+
+    def show_previews? # :doc:
+      ViewComponent::Base.show_previews
+    end
+
+    def find_preview # :doc:
+      candidates = []
+      params[:path].to_s.scan(%r{/|$}) { candidates << $` }
+      preview = candidates.detect { |candidate| ViewComponent::Preview.exists?(candidate) }
+
+      if preview
+        @preview = ViewComponent::Preview.find(preview)
+      else
+        raise AbstractController::ActionNotFound, "Component preview '#{params[:path]}' not found."
+      end
+    end
+
+    def set_locale
+      I18n.with_locale(params[:locale] || I18n.default_locale) do
+        yield
+      end
+    end
+
+    # Returns either {} or {layout: value} depending on configuration
+    def determine_layout(layout_override = nil, prepend_views: true)
+      return {} unless defined?(Rails.root)
+
+      layout_declaration = {}
+
+      if !layout_override.nil?
+        # Allow component-level override, even if false (thus no layout rendered)
+        layout_declaration[:layout] = layout_override
+      elsif default_preview_layout.present?
+        layout_declaration[:layout] = default_preview_layout
+      end
+
+      prepend_application_view_paths if layout_declaration[:layout].present? && prepend_views
+
+      layout_declaration
+    end
+
+    def prepend_application_view_paths
+      prepend_view_path Rails.root.join("app/views") if defined?(Rails.root)
+    end
+
+    def prepend_preview_examples_view_path
+      prepend_view_path(ViewComponent::Base.preview_paths)
+    end
+  end
+end

data/app/controllers/view_components_controller.rb

@@ -3,91 +3,5 @@
 require "rails/application_controller"
 
 class ViewComponentsController < Rails::ApplicationController # :nodoc:
-  prepend_view_path File.expand_path("../views", __dir__)
-
-  around_action :set_locale, only: :previews
-  before_action :find_preview, only: :previews
-  before_action :require_local!, unless: :show_previews?
-
-  if respond_to?(:content_security_policy)
-    content_security_policy(false)
-  end
-
-  def index
-    @previews = ViewComponent::Preview.all
-    @page_title = "Component Previews"
-    render "view_components/index", **determine_layout
-  end
-
-  def previews
-    if params[:path] == @preview.preview_name
-      @page_title = "Component Previews for #{@preview.preview_name}"
-      render "view_components/previews", **determine_layout
-    else
-      prepend_application_view_paths
-      prepend_preview_examples_view_path
-      @example_name = File.basename(params[:path])
-      @render_args = @preview.render_args(@example_name, params: params.permit!)
-      layout = determine_layout(@render_args[:layout], prepend_views: false)[:layout]
-      locals = @render_args[:locals]
-      opts = {}
-      opts[:layout] = layout if layout.present? || layout == false
-      opts[:locals] = locals if locals.present?
-      render "view_components/preview", opts # rubocop:disable GitHub/RailsControllerRenderLiteral
-    end
-  end
-
-  private
-
-  def default_preview_layout # :doc:
-    ViewComponent::Base.default_preview_layout
-  end
-
-  def show_previews? # :doc:
-    ViewComponent::Base.show_previews
-  end
-
-  def find_preview # :doc:
-    candidates = []
-    params[:path].to_s.scan(%r{/|$}) { candidates << $` }
-    preview = candidates.detect { |candidate| ViewComponent::Preview.exists?(candidate) }
-
-    if preview
-      @preview = ViewComponent::Preview.find(preview)
-    else
-      raise AbstractController::ActionNotFound, "Component preview '#{params[:path]}' not found."
-    end
-  end
-
-  def set_locale
-    I18n.with_locale(params[:locale] || I18n.default_locale) do
-      yield
-    end
-  end
-
-  # Returns either {} or {layout: value} depending on configuration
-  def determine_layout(layout_override = nil, prepend_views: true)
-    return {} unless defined?(Rails.root)
-
-    layout_declaration = {}
-
-    if !layout_override.nil?
-      # Allow component-level override, even if false (thus no layout rendered)
-      layout_declaration[:layout] = layout_override
-    elsif default_preview_layout.present?
-      layout_declaration[:layout] = default_preview_layout
-    end
-
-    prepend_application_view_paths if layout_declaration[:layout].present? && prepend_views
-
-    layout_declaration
-  end
-
-  def prepend_application_view_paths
-    prepend_view_path Rails.root.join("app/views") if defined?(Rails.root)
-  end
-
-  def prepend_preview_examples_view_path
-    prepend_view_path(ViewComponent::Base.preview_paths)
-  end
+  include ViewComponent::PreviewActions
 end

data/docs/CHANGELOG.md

@@ -3,10 +3,71 @@ layout: default
 title: Changelog
 ---
 
+<!-- Add unreleased changes under the "main" heading. -->
+
 # Changelog
 
 ## main
 
+## 2.54.1
+
+* Update docs dependencies.
+
+    *Joel Hawksley*
+
+* Resolve warning in slots API.
+* Raise in the test environment when ViewComponent code emits a warning.
+
+    *Blake Williams*
+
+## 2.54.0
+
+* Add `with_*` slot API for defining slots. Note: we plan to deprecate the non `with_*` API for slots in an upcoming release.
+
+    *Blake Williams*
+
+* Add QuickNode to list of companies that heavily rely on ViewComponent.
+
+    *Luc Castera*
+
+* Include the `Translatable` module by default.
+
+    *Elia Schito*
+
+* Update docs dependencies.
+
+    *Joel Hawksley*
+
+## 2.53.0
+
+* Add support for relative I18n scopes to translations.
+
+    *Elia Schito*
+
+* Update CI configuration to use latest Rails 7.0.
+
+    *Hans Lemuet*
+
+* Document how to use blocks with lambda slots.
+
+    *Sam Partington*
+
+* Skip Rails 5.2 in local test environment if using incompatible Ruby version.
+
+    *Cameron Dutro*, *Blake Williams*, *Joel Hawksley*
+
+* Improve landing page documentation.
+
+    *Jason Swett*
+
+* Add Bearer to list of companies that heavily rely on ViewComponent.
+
+    *Yaroslav Shmarov*
+
+* Add articles to resources page.
+
+    *Joel Hawksley*
+
 ## 2.52.0
 
 * Add ADR for separate slot getter/setter API.
@@ -124,10 +185,14 @@ title: Changelog
 
     *Joel Hawksley*
 
-* Add Ruby 3.1 and Rails 7.0 to CI
+* Add Ruby 3.1 and Rails 7.0 to CI.
 
     *Peter Goldstein*
 
+* Move preview logic to module for easier app integration.
+
+    *Sammy Henningsson*
+
 ## 2.48.0
 
 * Correct path in example test command in Contributing docs.

data/lib/view_component/base.rb

@@ -17,6 +17,7 @@ module ViewComponent
     include ViewComponent::ContentAreas
     include ViewComponent::Previewable
     include ViewComponent::SlotableV2
+    include ViewComponent::Translatable
     include ViewComponent::WithContentHelper
 
     ViewContextCalledBeforeRenderError = Class.new(StandardError)
@@ -69,7 +70,7 @@ module ViewComponent
       @view_context = view_context
       self.__vc_original_view_context ||= view_context
 
-      @output_buffer = ActionView::OutputBuffer.new unless @global_buffer_in_use
+      @output_buffer = ActionView::OutputBuffer.new unless defined?(@global_buffer_in_use) && @global_buffer_in_use
 
       @lookup_context ||= view_context.lookup_context
 

data/lib/view_component/compiler.rb

@@ -77,6 +77,7 @@ module ViewComponent
 
         define_render_template_for
 
+        component_class.build_i18n_backend
         component_class._after_compile
 
         CompileCache.register(component_class)

data/lib/view_component/polymorphic_slots.rb

@@ -45,10 +45,16 @@ module ViewComponent
               "#{slot_name}_#{poly_type}"
             end
 
+          # Deprecated: Will be removed in 3.0
           define_method(setter_name) do |*args, &block|
             set_polymorphic_slot(slot_name, poly_type, *args, &block)
           end
           ruby2_keywords(setter_name.to_sym) if respond_to?(:ruby2_keywords, true)
+
+          define_method("with_#{setter_name}") do |*args, &block|
+            set_polymorphic_slot(slot_name, poly_type, *args, &block)
+          end
+          ruby2_keywords(:"with_#{setter_name}") if respond_to?(:ruby2_keywords, true)
         end
 
         self.registered_slots[slot_name] = {

data/lib/view_component/slotable_v2.rb

@@ -61,20 +61,26 @@ module ViewComponent
       # = Setting sub-component content
       #
       # Consumers of the component can render a sub-component by calling a
-      # helper method with the same name as the slot.
+      # helper method with the same name as the slot prefixed with `with_`.
       #
       #   <%= render_inline(MyComponent.new) do |component| %>
-      #     <% component.header(classes: "Foo") do %>
+      #     <% component.with_header(classes: "Foo") do %>
       #       <p>Bar</p>
       #     <% end %>
       #   <% end %>
       def renders_one(slot_name, callable = nil)
         validate_singular_slot_name(slot_name)
 
+        define_method :"with_#{slot_name}" do |*args, &block|
+          set_slot(slot_name, nil, *args, &block)
+        end
+        ruby2_keywords(:"with_#{slot_name}") if respond_to?(:ruby2_keywords, true)
+
         define_method slot_name do |*args, &block|
           if args.empty? && block.nil?
             get_slot(slot_name)
           else
+            # Deprecated: Will remove in 3.0
             set_slot(slot_name, nil, *args, &block)
           end
         end
@@ -112,15 +118,15 @@ module ViewComponent
       # = Setting sub-component content
       #
       # Consumers of the component can set the content of a slot by calling a
-      # helper method with the same name as the slot. The method can be
-      # called multiple times to append to the slot.
+      # helper method with the same name as the slot prefixed with `with_`. The
+      # method can be called multiple times to append to the slot.
       #
       #   <%= render_inline(MyComponent.new) do |component| %>
-      #     <% component.item(name: "Foo") do %>
+      #     <% component.with_item(name: "Foo") do %>
       #       <p>One</p>
       #     <% end %>
       #
-      #     <% component.item(name: "Bar") do %>
+      #     <% component.with_item(name: "Bar") do %>
       #       <p>two</p>
       #     <% end %>
       #   <% end %>
@@ -132,17 +138,31 @@ module ViewComponent
         # Define setter for singular names
         # for example `renders_many :items` allows fetching all tabs with
         # `component.tabs` and setting a tab with `component.tab`
+        #
+        # Deprecated: Will remove in 3.0
         define_method singular_name do |*args, &block|
           set_slot(slot_name, nil, *args, &block)
         end
         ruby2_keywords(singular_name.to_sym) if respond_to?(:ruby2_keywords, true)
 
+        define_method :"with_#{singular_name}" do |*args, &block|
+          set_slot(slot_name, nil, *args, &block)
+        end
+        ruby2_keywords(:"with_#{singular_name}") if respond_to?(:ruby2_keywords, true)
+
+        define_method :"with_#{slot_name}" do |collection_args = nil, &block|
+          collection_args.map do |args|
+            set_slot(slot_name, nil, **args, &block)
+          end
+        end
+
         # Instantiates and and adds multiple slots forwarding the first
         # argument to each slot constructor
         define_method slot_name do |collection_args = nil, &block|
           if collection_args.nil? && block.nil?
             get_slot(slot_name)
           else
+            # Deprecated: Will remove in 3.0
             collection_args.map do |args|
               set_slot(slot_name, nil, **args, &block)
             end

data/lib/view_component/translatable.rb

@@ -3,7 +3,6 @@
 require "erb"
 require "set"
 require "i18n"
-require "action_view/helpers/translation_helper"
 require "active_support/concern"
 
 module ViewComponent
@@ -21,9 +20,7 @@ module ViewComponent
         @i18n_scope ||= virtual_path.sub(%r{^/}, "").gsub(%r{/_?}, ".")
       end
 
-      def _after_compile
-        super
-
+      def build_i18n_backend
         return if CompileCache.compiled? self
 
         if (translation_files = _sidecar_files(%w[yml yaml])).any?
@@ -68,7 +65,10 @@ module ViewComponent
       return key.map { |k| translate(k, **options) } if key.is_a?(Array)
 
       locale = options.delete(:locale) || ::I18n.locale
+      scope = options.delete(:scope)
+      scope = scope.join(".") if scope.is_a? Array
       key = key&.to_s unless key.is_a?(String)
+      key = "#{scope}.#{key}" if scope
       key = "#{i18n_scope}#{key}" if key.start_with?(".")
 
       if HTML_SAFE_TRANSLATION_KEY.match?(key)

data/lib/view_component/version.rb

@@ -3,8 +3,8 @@
 module ViewComponent
   module VERSION
     MAJOR = 2
-    MINOR = 52
-    PATCH = 0
+    MINOR = 54
+    PATCH = 1
 
     STRING = [MAJOR, MINOR, PATCH].join(".")
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: view_component
 version: !ruby/object:Gem::Version
-  version: 2.52.0
+  version: 2.54.1
 platform: ruby
 authors:
 - GitHub Open Source
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-03-25 00:00:00.000000000 Z
+date: 2022-05-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -293,6 +293,7 @@ files:
 - README.md
 - app/assets/vendor/prism.css
 - app/assets/vendor/prism.min.js
+- app/controllers/concerns/view_component/preview_actions.rb
 - app/controllers/view_components_controller.rb
 - app/helpers/preview_helper.rb
 - app/views/test_mailer/test_email.html.erb