polaris_view_components 0.7.0 → 0.8.0

data/app/components/polaris/exception_list_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class ExceptionListComponent < Polaris::NewComponent
+  class ExceptionListComponent < Polaris::Component
     renders_many :items, Polaris::ExceptionList::ItemComponent
 
     def initialize(**system_arguments)

data/app/components/polaris/filters_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class FiltersComponent < Polaris::NewComponent
+  class FiltersComponent < Polaris::Component
     renders_one :query, ->(**system_arguments) do
       QueryComponent.new(disabled: @disabled, **system_arguments)
     end
@@ -34,7 +34,7 @@ module Polaris
       )
     end
 
-    class QueryComponent < Polaris::NewComponent
+    class QueryComponent < Polaris::Component
       def initialize(clear_button: true, disabled: false, **system_arguments)
         @disabled = disabled
         @system_arguments = system_arguments.merge(
@@ -52,7 +52,7 @@ module Polaris
       end
     end
 
-    class ItemComponent < Polaris::NewComponent
+    class ItemComponent < Polaris::Component
       def initialize(label:, sectioned: true, width: nil, disabled: false, **system_arguments)
         @label = label
         @sectioned = sectioned

data/app/components/polaris/footer_help_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class FooterHelpComponent < Polaris::NewComponent
+  class FooterHelpComponent < Polaris::Component
     def initialize(**system_arguments)
       @system_arguments = system_arguments
       @system_arguments[:classes] = class_names(

data/app/components/polaris/form_layout/group_component.rb

@@ -1,4 +1,4 @@
-class Polaris::FormLayout::GroupComponent < Polaris::NewComponent
+class Polaris::FormLayout::GroupComponent < Polaris::Component
   attr_reader :position
 
   renders_many :items, "GroupItemComponent"
@@ -22,7 +22,7 @@ class Polaris::FormLayout::GroupComponent < Polaris::NewComponent
     )
   end
 
-  class GroupItemComponent < Polaris::NewComponent
+  class GroupItemComponent < Polaris::Component
     def initialize(**system_arguments)
       @system_arguments = system_arguments
       @system_arguments[:tag] = "div"

data/app/components/polaris/form_layout/item_component.rb

@@ -1,4 +1,4 @@
-class Polaris::FormLayout::ItemComponent < Polaris::NewComponent
+class Polaris::FormLayout::ItemComponent < Polaris::Component
   attr_reader :position
 
   def initialize(position:, **system_arguments)

data/app/components/polaris/form_layout_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class FormLayoutComponent < Polaris::NewComponent
+  class FormLayoutComponent < Polaris::Component
     renders_many :items, ->(**system_arguments) do
       @counter += 1
       Polaris::FormLayout::ItemComponent.new(position: @counter, **system_arguments)

data/app/components/polaris/frame/save_bar_component.rb

@@ -1,4 +1,4 @@
-class Polaris::Frame::SaveBarComponent < Polaris::NewComponent
+class Polaris::Frame::SaveBarComponent < Polaris::Component
   renders_one :save_action, ->(**system_arguments) do
     Polaris::ButtonComponent.new(primary: true, **system_arguments)
   end

data/app/components/polaris/frame/top_bar_component.rb

@@ -1,4 +1,4 @@
-class Polaris::Frame::TopBarComponent < Polaris::NewComponent
+class Polaris::Frame::TopBarComponent < Polaris::Component
   renders_one :user_menu, Polaris::TopBar::UserMenuComponent
 
   def initialize(logo:, **system_arguments)

data/app/components/polaris/frame_component.rb

@@ -1,5 +1,5 @@
 module Polaris
-  class FrameComponent < Polaris::NewComponent
+  class FrameComponent < Polaris::Component
     renders_one :top_bar, ->(**system_arguments) do
       Polaris::Frame::TopBarComponent.new(logo: @logo, **system_arguments)
     end

data/app/components/polaris/heading_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class HeadingComponent < Polaris::NewComponent
+  class HeadingComponent < Polaris::Component
     ELEMENT_DEFAULT = :h2
     ELEMENT_OPTIONS = %i[p h1 h2 h3 h4 h5 h6]
 

data/app/components/polaris/headless_button.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class HeadlessButton < Polaris::NewComponent
+  class HeadlessButton < Polaris::Component
     SIZE_DEFAULT = :medium
     SIZE_MAPPINGS = {
       SIZE_DEFAULT => "",

data/app/components/polaris/icon_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class IconComponent < Polaris::NewComponent
+  class IconComponent < Polaris::Component
     COLOR_DEFAULT = :default
     COLOR_MAPPINGS = {
       COLOR_DEFAULT => "",

data/app/components/polaris/index_table/cell_component.rb

@@ -1,4 +1,4 @@
-class Polaris::IndexTable::CellComponent < Polaris::NewComponent
+class Polaris::IndexTable::CellComponent < Polaris::Component
   def initialize(flush: false, **system_arguments)
     @flush = flush
     @system_arguments = system_arguments

data/app/components/polaris/index_table/column_component.rb

@@ -1,4 +1,4 @@
-class Polaris::IndexTable::ColumnComponent < Polaris::NewComponent
+class Polaris::IndexTable::ColumnComponent < Polaris::Component
   attr_reader :title, :flush
 
   def initialize(title, flush: false, **system_arguments, &block)

data/app/components/polaris/index_table_component.rb

@@ -1,5 +1,5 @@
 module Polaris
-  class IndexTableComponent < Polaris::NewComponent
+  class IndexTableComponent < Polaris::Component
     renders_many :columns, ->(title, **system_arguments, &block) do
       IndexTable::ColumnComponent.new(title, **system_arguments, &block)
     end

data/app/components/polaris/inline_error_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class InlineErrorComponent < Polaris::NewComponent
+  class InlineErrorComponent < Polaris::Component
     def initialize(**system_arguments)
       @system_arguments = system_arguments
       @system_arguments[:tag] = "div"

data/app/components/polaris/label_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class LabelComponent < Polaris::NewComponent
+  class LabelComponent < Polaris::Component
     def initialize(
       form: nil,
       attribute: nil,

data/app/components/polaris/labelled_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class LabelledComponent < Polaris::NewComponent
+  class LabelledComponent < Polaris::Component
     renders_one :label_action, ->(**system_arguments) do
       Polaris::ButtonComponent.new(plain: true, **system_arguments)
     end

data/app/components/polaris/layout/annotated_section.rb

@@ -2,7 +2,7 @@
 
 module Polaris
   module Layout
-    class AnnotatedSection < Polaris::NewComponent
+    class AnnotatedSection < Polaris::Component
       attr_reader :position
 
       def initialize(position:, title:, description: nil, **system_arguments)

data/app/components/polaris/layout/section.rb

@@ -2,7 +2,7 @@
 
 module Polaris
   module Layout
-    class Section < Polaris::NewComponent
+    class Section < Polaris::Component
       attr_reader :position
 
       def initialize(

data/app/components/polaris/layout_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class LayoutComponent < Polaris::NewComponent
+  class LayoutComponent < Polaris::Component
     # A list of sections
     #
     # @param secondary [Boolean] The section will act like a sidebar

data/app/components/polaris/link_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class LinkComponent < Polaris::NewComponent
+  class LinkComponent < Polaris::Component
     def initialize(
       url:,
       external: false,

data/app/components/polaris/list_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class ListComponent < Polaris::NewComponent
+  class ListComponent < Polaris::Component
     renders_many :items, "ListItemComponent"
 
     TYPE_DEFAULT = :bullet
@@ -28,7 +28,7 @@ module Polaris
       )
     end
 
-    class ListItemComponent < Polaris::NewComponent
+    class ListItemComponent < Polaris::Component
       def initialize(**system_arguments)
         @system_arguments = system_arguments
         @system_arguments[:tag] = :li

data/app/components/polaris/modal/section_component.rb

@@ -1,4 +1,4 @@
-class Polaris::Modal::SectionComponent < Polaris::NewComponent
+class Polaris::Modal::SectionComponent < Polaris::Component
   def initialize(**system_arguments)
     @system_arguments = system_arguments
   end

data/app/components/polaris/modal_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class ModalComponent < Polaris::NewComponent
+  class ModalComponent < Polaris::Component
     renders_one :close_button, ->(**system_arguments) do
       button_arguments = @close_button_arguments.deep_merge(system_arguments)
       render(Polaris::BaseComponent.new(**button_arguments)) do

data/app/components/polaris/navigation/item_component.rb

@@ -1,4 +1,4 @@
-class Polaris::Navigation::ItemComponent < Polaris::NewComponent
+class Polaris::Navigation::ItemComponent < Polaris::Component
   renders_many :sub_items, Polaris::Navigation::ItemComponent
   renders_one :secondary_action, "SecondaryActionComponent"
 
@@ -47,7 +47,7 @@ class Polaris::Navigation::ItemComponent < Polaris::NewComponent
     sub_items.any?(&:selected)
   end
 
-  class SecondaryActionComponent < Polaris::NewComponent
+  class SecondaryActionComponent < Polaris::Component
     def initialize(url: nil, external: false, icon: nil, **system_arguments)
       @url = url
       @external = external

data/app/components/polaris/navigation/section_component.rb

@@ -1,4 +1,4 @@
-class Polaris::Navigation::SectionComponent < Polaris::NewComponent
+class Polaris::Navigation::SectionComponent < Polaris::Component
   renders_many :items, Polaris::Navigation::ItemComponent
   renders_one :action, "ActionComponent"
 
@@ -26,7 +26,7 @@ class Polaris::Navigation::SectionComponent < Polaris::NewComponent
     end
   end
 
-  class ActionComponent < Polaris::NewComponent
+  class ActionComponent < Polaris::Component
     def initialize(url: nil, external: false, icon: nil, **system_arguments)
       @url = url
       @external = external

data/app/components/polaris/navigation_component.rb

@@ -1,5 +1,5 @@
 module Polaris
-  class NavigationComponent < Polaris::NewComponent
+  class NavigationComponent < Polaris::Component
     renders_many :sections, Polaris::Navigation::SectionComponent
     renders_many :items, Polaris::Navigation::ItemComponent
 

data/app/components/polaris/option_list/checkbox_component.rb

@@ -1,4 +1,4 @@
-class Polaris::OptionList::CheckboxComponent < Polaris::NewComponent
+class Polaris::OptionList::CheckboxComponent < Polaris::Component
   def initialize(
     label:,
     value:,

data/app/components/polaris/option_list/option_component.rb

@@ -1,4 +1,4 @@
-class Polaris::OptionList::OptionComponent < Polaris::NewComponent
+class Polaris::OptionList::OptionComponent < Polaris::Component
   def initialize(**system_arguments)
     @system_arguments = system_arguments
   end

data/app/components/polaris/option_list/radio_button_component.rb

@@ -1,4 +1,4 @@
-class Polaris::OptionList::RadioButtonComponent < Polaris::NewComponent
+class Polaris::OptionList::RadioButtonComponent < Polaris::Component
   def initialize(
     label:,
     value:,

data/app/components/polaris/option_list/section_component.rb

@@ -1,4 +1,4 @@
-class Polaris::OptionList::SectionComponent < Polaris::NewComponent
+class Polaris::OptionList::SectionComponent < Polaris::Component
   renders_many :options, Polaris::OptionList::OptionComponent
   renders_many :radio_buttons, ->(value:, **system_arguments) do
     Polaris::OptionList::RadioButtonComponent.new(

data/app/components/polaris/option_list_component.rb

@@ -1,5 +1,5 @@
 module Polaris
-  class OptionListComponent < Polaris::NewComponent
+  class OptionListComponent < Polaris::Component
     renders_many :sections, ->(**system_arguments) do
       Polaris::OptionList::SectionComponent.new(
         form: @form,

data/app/components/polaris/page_actions_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class PageActionsComponent < Polaris::NewComponent
+  class PageActionsComponent < Polaris::Component
     DISTRIBUTION_DEFAULT = nil
     DISTRIBUTION_OPTIONS = [
       DISTRIBUTION_DEFAULT,

data/app/components/polaris/page_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class PageComponent < Polaris::NewComponent
+  class PageComponent < Polaris::Component
     renders_one :primary_action, ->(primary: true, **system_arguments) do
       Polaris::ButtonComponent.new(primary: primary, **system_arguments)
     end

data/app/components/polaris/pagination_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class PaginationComponent < Polaris::NewComponent
+  class PaginationComponent < Polaris::Component
     def initialize(
       previous_url: nil,
       next_url: nil,

data/app/components/polaris/popover/pane_component.rb

@@ -1,4 +1,4 @@
-class Polaris::Popover::PaneComponent < Polaris::NewComponent
+class Polaris::Popover::PaneComponent < Polaris::Component
   renders_many :sections, Polaris::Popover::SectionComponent
 
   def initialize(fixed: false, sectioned: false, **system_arguments)

data/app/components/polaris/popover/section_component.rb

@@ -1,4 +1,4 @@
-class Polaris::Popover::SectionComponent < Polaris::NewComponent
+class Polaris::Popover::SectionComponent < Polaris::Component
   def initialize(**system_arguments)
     @system_arguments = system_arguments
   end

data/app/components/polaris/popover_component.rb

@@ -1,10 +1,10 @@
 module Polaris
-  class PopoverComponent < Polaris::NewComponent
+  class PopoverComponent < Polaris::Component
     ALIGNMENT_DEFAULT = :center
     ALIGNMENT_OPTIONS = [:left, :right, :center]
 
-    POSITION_DEFAULT = :below
-    POSITION_OPTIONS = [:above, :below]
+    POSITION_DEFAULT = :auto
+    POSITION_OPTIONS = [:auto, :above, :below]
 
     renders_one :button, ->(**system_arguments) do
       system_arguments[:data] ||= {}
@@ -98,7 +98,13 @@ module Polaris
     end
 
     def popperjs_placement
-      placement = @position == :above ? "top" : "bottom"
+      placement =
+        case @position
+        when :above then "top"
+        when :below then "bottom"
+        else
+          "auto"
+        end
       placement += "-start" if @alignment == :left
       placement += "-end" if @alignment == :right
       placement

data/app/components/polaris/progress_bar_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class ProgressBarComponent < Polaris::NewComponent
+  class ProgressBarComponent < Polaris::Component
     COLOR_DEFAULT = :highlight
     COLOR_MAPPINGS = {
       COLOR_DEFAULT => "Polaris-ProgressBar--colorHighlight",

data/app/components/polaris/radio_button_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class RadioButtonComponent < Polaris::NewComponent
+  class RadioButtonComponent < Polaris::Component
     def initialize(
       form: nil,
       attribute: nil,

data/app/components/polaris/resource_item_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class ResourceItemComponent < Polaris::NewComponent
+  class ResourceItemComponent < Polaris::Component
     CURSOR_DEFAULT = :default
     CURSOR_OPTIONS = %i[default pointer]
 

data/app/components/polaris/resource_list_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class ResourceListComponent < Polaris::NewComponent
+  class ResourceListComponent < Polaris::Component
     renders_one :filters, Polaris::FiltersComponent
 
     def initialize(

data/app/components/polaris/scrollable_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class ScrollableComponent < Polaris::NewComponent
+  class ScrollableComponent < Polaris::Component
     def initialize(
       height: nil,
       width: nil,

data/app/components/polaris/select_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class SelectComponent < Polaris::NewComponent
+  class SelectComponent < Polaris::Component
     def initialize(
       options:,
       form: nil,

data/app/components/polaris/setting_toggle_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class SettingToggleComponent < Polaris::NewComponent
+  class SettingToggleComponent < Polaris::Component
     renders_one :action, ->(**system_arguments) do
       Polaris::ButtonComponent.new(primary: !@enabled, **system_arguments)
     end

data/app/components/polaris/shopify_navigation_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class ShopifyNavigationComponent < Polaris::NewComponent
+  class ShopifyNavigationComponent < Polaris::Component
     renders_many :links, ->(**system_arguments) do
       ShopifyNavigationLinkComponent.new(auto_detect_active: @auto_detect_active, **system_arguments)
     end
@@ -21,7 +21,7 @@ module Polaris
       links.any?
     end
 
-    class ShopifyNavigationLinkComponent < Polaris::NewComponent
+    class ShopifyNavigationLinkComponent < Polaris::Component
       def initialize(url:, auto_detect_active:, active: false, **system_arguments)
         @url = url
         @auto_detect_active = auto_detect_active

data/app/components/polaris/skeleton_body_text_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class SkeletonBodyTextComponent < Polaris::NewComponent
+  class SkeletonBodyTextComponent < Polaris::Component
     def initialize(lines: 3, **system_arguments)
       @lines = lines
       @system_arguments = system_arguments

data/app/components/polaris/spacer_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class SpacerComponent < Polaris::NewComponent
+  class SpacerComponent < Polaris::Component
     VERTICAL_SPACING_DEFAULT = :default
     VERTICAL_SPACING_MAPPINGS = {
       VERTICAL_SPACING_DEFAULT => "",

data/app/components/polaris/spinner_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class SpinnerComponent < Polaris::NewComponent
+  class SpinnerComponent < Polaris::Component
     SIZE_DEFAULT = :large
     SIZE_MAPPINGS = {
       small: "Polaris-Spinner--sizeSmall",

data/app/components/polaris/stack/item_component.rb

@@ -0,0 +1,15 @@
+class Polaris::Stack::ItemComponent < Polaris::Component
+  def initialize(fill: false, **system_arguments)
+    @system_arguments = system_arguments
+    @system_arguments[:tag] = "div"
+    @system_arguments[:classes] = class_names(
+      @system_arguments[:classes],
+      "Polaris-Stack__Item",
+      "Polaris-Stack__Item--fill": fill
+    )
+  end
+
+  def call
+    render(Polaris::BaseComponent.new(**@system_arguments)) { content }
+  end
+end

data/app/components/polaris/stack_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class StackComponent < Polaris::NewComponent
+  class StackComponent < Polaris::Component
     ALIGNMENT_DEFAULT = :default
     ALIGNMENT_MAPPINGS = {
       ALIGNMENT_DEFAULT => "",
@@ -37,7 +37,7 @@ module Polaris
     }
     SPACING_OPTIONS = SPACING_MAPPINGS.keys
 
-    renders_many :items, "StackItemComponent"
+    renders_many :items, Polaris::Stack::ItemComponent
 
     def initialize(
       alignment: ALIGNMENT_DEFAULT,
@@ -59,21 +59,5 @@ module Polaris
         "Polaris-Stack--noWrap": !wrap
       )
     end
-
-    class StackItemComponent < Polaris::NewComponent
-      def initialize(fill: false, **system_arguments)
-        @system_arguments = system_arguments
-        @system_arguments[:tag] = "div"
-        @system_arguments[:classes] = class_names(
-          @system_arguments[:classes],
-          "Polaris-Stack__Item",
-          "Polaris-Stack__Item--fill": fill
-        )
-      end
-
-      def call
-        render(Polaris::BaseComponent.new(**@system_arguments)) { content }
-      end
-    end
   end
 end

data/app/components/polaris/subheading_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class SubheadingComponent < Polaris::NewComponent
+  class SubheadingComponent < Polaris::Component
     ELEMENT_DEFAULT = :h3
     ELEMENT_OPTIONS = %i[p h1 h2 h3 h4 h5 h6]
 

data/app/components/polaris/tabs/tab_component.rb

@@ -1,4 +1,4 @@
-class Polaris::Tabs::TabComponent < Polaris::NewComponent
+class Polaris::Tabs::TabComponent < Polaris::Component
   renders_one :badge, Polaris::BadgeComponent
 
   def initialize(

data/app/components/polaris/tabs_component.rb

@@ -1,5 +1,5 @@
 module Polaris
-  class TabsComponent < Polaris::NewComponent
+  class TabsComponent < Polaris::Component
     renders_many :tabs, Polaris::Tabs::TabComponent
 
     def initialize(fitted: false, wrapper_arguments: {}, **system_arguments)

data/app/components/polaris/tag_component.rb

@@ -1,11 +1,12 @@
 # frozen_string_literal: true
 
 module Polaris
-  class TagComponent < Polaris::NewComponent
-    renders_one :remove_button, ->(**system_arguments) do
+  class TagComponent < Polaris::Component
+    renders_one :remove_button, ->(type: :button, **system_arguments) do
       render Polaris::BaseButton.new(
         classes: "Polaris-Tag__Button",
         disabled: @disabled,
+        type: type,
         **system_arguments
       ) do |button|
         polaris_icon(name: "CancelSmallMinor")

data/app/components/polaris/text_container_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Polaris
-  class TextContainerComponent < Polaris::NewComponent
+  class TextContainerComponent < Polaris::Component
     SPACING_DEFAULT = :default
     SPACING_MAPPINGS = {
       SPACING_DEFAULT => "",